Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-06-12

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:10 michchap_ joined #puppet-openstack
00:12 markvoelker joined #puppet-openstack
00:36 michchap_ joined #puppet-openstack
00:41 aimon joined #puppet-openstack
00:42 stevemar joined #puppet-openstack
00:43 gildub joined #puppet-openstack
00:54 ducttape_ joined #puppet-openstack
00:55 EmilienM richm: hopefully that story is finished soon
00:59 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: keystone_service resource using API version 3  https://review.openstack.org/184195
00:59 xingchao joined #puppet-openstack
01:06 michchap_ joined #puppet-openstack
01:08 saneax joined #puppet-openstack
01:14 gildub EmilienM, Hi - Do we have a convention for header files, I'm talking about Licence, Author and Copyrights?
01:15 gildub EmilienM, Sorry I meant License(s)
01:15 gildub EmilienM, we probably inherit from umbrella standards
01:15 EmilienM yes
01:16 EmilienM gildub: you can look at how we do
01:16 gildub EmilienM, This is quite inconsistent at the moment, it doesn't makes sense to me to have every file containing those information
01:17 EmilienM gildub: I'm off a bit for diner, bbl
01:17 gildub EmilienM, normally a top file for License and Copyright does the job
01:17 gildub EmilienM, np
01:18 gildub EmilienM, bon appetit!
01:21 saneax joined #puppet-openstack
02:01 markvoelker joined #puppet-openstack
02:05 michchap_ joined #puppet-openstack
02:08 xingchao joined #puppet-openstack
02:27 britthouser joined #puppet-openstack
02:33 gildub EmilienM, crinkle: Any chance to get https://review.openstack.org/187654/ merged?
02:33 EmilienM looking
02:33 EmilienM gildub: already +2, no?
02:33 gildub EmilienM, yes
02:34 crinkle merged
02:35 gildub crinkle, EmilienM, thanks!
02:35 openstackgerrit Cody Herriges proposed stackforge/puppet-neutron: Sync up module with latest msync commit.  https://review.openstack.org/189873
02:39 openstackgerrit Cody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x  https://review.openstack.org/184646
02:40 gildub EmilienM, I know it's late for you, and I hate forcing others outside of normal business hours, meanwhile I would appreciate a quick feedback on the license/header question. From https://wiki.openstack.org/wiki/LegalIssuesFAQ#Copyright_Headers it seems things are not 100% settled but I understand license/headers could be helpful at times but not necessarily needed in every file either, I also understand there are legacy heade
02:40 gildub rs.
02:40 openstackgerrit Merged stackforge/puppet-openstacklib: support for Keystone v3 authentication  https://review.openstack.org/187654
02:41 openstackgerrit Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - add domains to service_identity  https://review.openstack.org/180586
02:42 gildub EmilienM, so basically what I'm trying to clarify, is the fact every project has an ASF Liencse file on top tree level, so we don't need (agains unless special cases) to have a license header in some files, is that a correct approach?
02:46 openstackgerrit Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - beaker tests  https://review.openstack.org/178828
03:01 michchap_ joined #puppet-openstack
03:04 openstackgerrit Richard Megginson proposed stackforge/puppet-openstack-specs: Support Keystone v3 API in openstack puppet modules  https://review.openstack.org/150108
03:09 stevemar joined #puppet-openstack
03:33 openstackgerrit Sam Morrison proposed stackforge/puppet-neutron: Don't specify a nova region by default  https://review.openstack.org/180378
03:41 stevemar2 joined #puppet-openstack
03:50 markvoelker joined #puppet-openstack
03:51 saneax joined #puppet-openstack
03:53 EmilienM gildub: ok, it seems fair
03:55 gildub EmilienM, ok, thanks for the feedback
04:00 EmilienM gildub: also, it would be awesome if you could follow richm patches title convention
04:00 EmilienM gildub: https://review.openstack.org/#/c/184195/
04:00 EmilienM but it's not a big deal!
04:01 EmilienM gildub: I'm off, cheers
04:01 gildub EmilienM, that's okay, just that I thought I was following openstack commit standards
04:01 EmilienM gildub: just easier for review
04:02 gildub EmilienM, no pb I'll
04:02 gildub EmilienM, have a good one!
04:03 fedexo joined #puppet-openstack
04:04 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: keystone_endpoint resource using API version 3  https://review.openstack.org/178456
04:12 cwolferh joined #puppet-openstack
04:22 ozialien joined #puppet-openstack
04:29 openstackgerrit Cody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x  https://review.openstack.org/184646
04:29 openstackgerrit Cody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x  https://review.openstack.org/184646
04:40 openstackgerrit Matt Fischer proposed stackforge/puppet-trove: Sync amqp support with other openstack modules  https://review.openstack.org/169808
05:10 bkero joined #puppet-openstack
05:35 michchap_ joined #puppet-openstack
05:39 markvoelker joined #puppet-openstack
05:53 gfidente joined #puppet-openstack
06:00 jtomasek joined #puppet-openstack
06:04 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use zuul-cloner and clone map to clone dependencies  https://review.openstack.org/190839
06:12 michchap_ joined #puppet-openstack
06:18 gildub crinkle, we're all green for bp/api-version-support but a couple of those http://logs.openstack.org/76/174976/17/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-centos7/9d85196/console.html
06:19 crinkle gildub: i think keystone_user isn't setting credentials.username in the password check method
06:21 gildub crinkle, yeah richm chanded it for user_id
06:21 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use zuul-cloner and clone map to clone dependencies  https://review.openstack.org/190839
06:23 gildub crinkle, ok, I can see it now, thanks
06:24 gildub crinkle, catch such errors is the goal of acceptance tests, sorry but not used to have them working yet! :D)
06:25 crinkle :)
06:34 alex_bh joined #puppet-openstack
06:50 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: support for keystone v3 api - the user resource  https://review.openstack.org/174976
06:57 gchamoul morning
06:59 derekh joined #puppet-openstack
07:03 tfz joined #puppet-openstack
07:03 xarses joined #puppet-openstack
07:03 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use zuul-cloner and clone map to clone dependencies  https://review.openstack.org/190839
07:09 ianw joined #puppet-openstack
07:13 arnaud_orange joined #puppet-openstack
07:15 ianw joined #puppet-openstack
07:16 fedexo joined #puppet-openstack
07:25 jpena joined #puppet-openstack
07:27 dgurtner joined #puppet-openstack
07:28 markvoelker joined #puppet-openstack
07:29 alex_bh left #puppet-openstack
08:20 jistr joined #puppet-openstack
08:31 gildub joined #puppet-openstack
08:35 cdent joined #puppet-openstack
08:51 social joined #puppet-openstack
09:01 jgrassler joined #puppet-openstack
09:16 markvoelker joined #puppet-openstack
09:25 jgrassler joined #puppet-openstack
10:04 openstackgerrit vsaienko proposed stackforge/puppet-horizon: Add cache_backend and cache_options params  https://review.openstack.org/190548
10:42 ianw joined #puppet-openstack
10:59 crinkle joined #puppet-openstack
11:50 markvoelker joined #puppet-openstack
11:58 ducttape_ joined #puppet-openstack
12:06 dprince joined #puppet-openstack
12:17 EmilienM good morning folks
12:30 trash hey EmilienM, can you have a look at the simple backport of https://review.openstack.org/#/c/188396/? this is quiet annoying with the 120s default or as we do, have a fork just because of that timeout
12:36 iurygregory good morning
12:37 iurygregory if anyone have time https://review.openstack.org/#/c/190361/ =)
12:59 openstackgerrit Merged stackforge/puppet-neutron: Don't specify a nova region by default  https://review.openstack.org/180378
13:05 kkirkpatrick joined #puppet-openstack
13:08 ducttape_ joined #puppet-openstack
13:10 morazi joined #puppet-openstack
13:11 tristanC joined #puppet-openstack
13:16 tfz joined #puppet-openstack
13:17 richm joined #puppet-openstack
13:20 ducttape_ joined #puppet-openstack
13:25 EmilienM trash: looking now
13:26 ferest joined #puppet-openstack
13:27 trash EmilienM: thanks!
14:03 saneax joined #puppet-openstack
14:06 pasquier-s joined #puppet-openstack
14:54 _ody https://review.openstack.org/#/c/184646/ and https://review.openstack.org/#/c/189873/, Can someone please review them at the same time this time so they'll merge.  They are dependent on each other.
14:57 sbadia hi!
14:57 sbadia _ody: sorry for the last pings :-/
14:57 sbadia _ody: I take a look, thanks!
14:58 sbadia and btw, no msync was never launched on our modules…
14:59 xingchao joined #puppet-openstack
15:00 sbadia EmilienM: no, I didn't manage all repos, but it's on my todo, btw the backport wasn't merged yet (for syntax/ rspec-puppet 2.x)
15:01 _ody sbadia: Ah.  Had another thing to add to the msync thread that came from review of puppet-neutron.
15:01 * _ody will go comment
15:13 xarses joined #puppet-openstack
15:19 chem joined #puppet-openstack
15:26 michchap joined #puppet-openstack
15:48 openstackgerrit Alex Schultz proposed stackforge/puppet-neutron: Create file /etc/default/neutron-server  https://review.openstack.org/130937
15:55 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use zuul-cloner and clone map to clone dependencies  https://review.openstack.org/190839
16:18 xingchao joined #puppet-openstack
16:22 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use zuul-cloner and clone map to clone dependencies  https://review.openstack.org/190839
16:28 dprince joined #puppet-openstack
16:34 openstackgerrit Rajini Ram proposed stackforge/puppet-cinder: Added support for DellStorageCenter ISCSI cinder driver  https://review.openstack.org/189372
16:38 openstackgerrit Rajini Ram proposed stackforge/puppet-cinder: Added support for DellStorageCenter ISCSI cinder driver  https://review.openstack.org/189372
16:40 openstackgerrit Rajini Ram proposed stackforge/puppet-cinder: Added support for DellStorageCenter ISCSI cinder driver  https://review.openstack.org/189372
16:41 openstackgerrit Jiri Stransky proposed stackforge/puppet-tripleo: Configure fencing devices  https://review.openstack.org/188412
16:49 cdent_ joined #puppet-openstack
18:12 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use zuul-cloner and clone map to clone dependencies  https://review.openstack.org/190839
18:13 EmilienM information for everyone, infra team is currently working on migration from stackforge to openstack
18:13 EmilienM of our modules
18:20 britthouser joined #puppet-openstack
18:21 britthouser left #puppet-openstack
18:28 openstackgerrit Emilien Macchi proposed stackforge/puppet-tripleo: Implement Advanced Firewalling support  https://review.openstack.org/189484
18:29 aimon_ joined #puppet-openstack
18:36 marvin0815 joined #puppet-openstack
18:44 EmilienM richm: hey, let me introduce you chem, who is willing to work more in upstream modules. I was wondering if he could help in v3 work with updating providers
18:44 EmilienM richm: maybe in puppet-neutron?
18:47 EmilienM chem: you might want to read http://specs.openstack.org/stackforge/puppet-openstack-specs/specs/kilo/auth-restructure.html and https://review.openstack.org/#/c/150108/ which are the two main blueprints
18:48 EmilienM chem: also the code: https://review.openstack.org/#/q/topic:bp/api-v3-support,n,z and https://review.openstack.org/#/q/topic:bp/auth_restructure,n,z
18:49 richm chem: Hello!
18:49 EmilienM chem: also our trello board with a lot of tasks: https://trello.com/b/4X3zxWRZ/on-going-effort - feel free to take a card, assign it to you and work on it. We try to keep richm updated about the work, he's mainly coordinating that work
18:53 openstackgerrit Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - the user resource  https://review.openstack.org/174976
19:09 dontalton joined #puppet-openstack
19:15 chem richm: Hi :)
19:15 chem EmilienM: thanks
19:21 vinsh_ joined #puppet-openstack
19:51 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use zuul-cloner and clone map to clone dependencies  https://review.openstack.org/190839
20:07 social joined #puppet-openstack
20:07 dmellado joined #puppet-openstack
20:12 openstackgerrit Richard Megginson proposed stackforge/puppet-openstack-specs: Support Keystone v3 API in openstack puppet modules  https://review.openstack.org/150108
20:15 openstackgerrit Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api - the user_role resource  https://review.openstack.org/174999
20:20 richm crinkle: ping - re: https://review.openstack.org/#/c/175060/23/manifests/init.pp
20:22 crinkle richm: pong
20:22 richm crinkle: You gave an example of a file which does not have a source or contents - /etc/keystone/keystone.conf
20:23 richm This is different because a) /etc/keystone/keystone.conf is always provided by the package and b) it is always directly modifiable
20:23 richm /etc/keystone/policy.json is different a) not always provided b) not always the right version
20:24 richm How can I have a file resource for /etc/keystone/policy.json that ensures it exists and is based on the v3 policy source?
20:24 richm That I can then modify to add the stuff that puppet needs?
20:24 crinkle richm: i'm open to suggestions but do you see the problem with how it is in that patch?
20:24 richm crinkle: yes
20:25 crinkle richm: i would want to identify the parts of the sample keystonev3 policy file that are really critical and manage them with openstacklib::policy::base
20:26 crinkle or convince the packagers to do it like they do keystone.conf
20:26 richm I could use an exec resource . . .
20:27 crinkle richm: that's not the worst idea
20:27 crinkle richm: that might be worth pursuing
20:28 richm crinkle: ok - I'll give it a shot
20:44 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use zuul-cloner and clone map to clone dependencies  https://review.openstack.org/190839
20:44 ozialien joined #puppet-openstack
20:48 xarses olleh puppeters, looking at keystone, it looks like all of the providers (ie keystone_enpoint, keystone_user...) expect that keystone is installed on the host we are running on https://github.com/stackforge/puppet-keystone/blob/master/lib/puppet/provider/keystone.rb#L89-90. How should we go about making it so we don't have to read these from a config file in the event keystone wasn't installed on this node?
20:49 iurygregory Hi richm , i have make the spec for K2K https://review.openstack.org/#/c/190361/ =)
20:49 xarses for now I think I will just write out the bits into a /etc/keystone/keystone.conf and erase it later.
20:50 richm xarses: I think that is a common thing to do.  From what I've been able to understand from most deployers, the puppet composition layer is arranged such that keystone resources are only instantiated on the keystone node
20:50 richm xarses: that is, you arrange your manifests so that keystone_endpoint, keystone_user, etc. are only ever used on the keystone node
20:51 crinkle xarses: you should be able to set environment variables or have it look in /root/openrc http://git.openstack.org/cgit/stackforge/puppet-openstack-specs/tree/specs/kilo/auth-restructure.rst
20:51 xarses richm: yes, but it no longer makes any sense when we are talking about deploying multiple regions where keystone is shared accross them
20:51 crinkle xarses: the providers now have a lot of their auth logic in puppet-openstacklib
20:52 crinkle so what you see in lib/puppet/provider/keystone.rb isn't the whole story anymore
20:52 openstackgerrit David Moreau Simard proposed stackforge/puppet-keystone: Synchronize Keystone WSGI script with upstream  https://review.openstack.org/191212
20:54 richm iurygregory: nice!  I just sent an email to openstack-dev with some information about using puppet to set up Keystone federated auth using ipsilon/freeipa
20:55 iurygregory i'll take a look =)
20:56 xarses crinkle: so I'm miss reading that https://github.com/stackforge/puppet-keystone/blob/master/lib/puppet/provider/keystone.rb#L84-92 is going to set the token to the admin_token or nil and the url to the admin endpoint or http://127.0.0.1:35357/v(whatever was in credver in openstacklib)
20:57 xarses it seems that it would over-write what ever was in /root/openrc|ENV from the get_admin_endpoint or get_admin_token function
20:58 richm xarses: only if super in self.request throws an exception
20:59 richm xarses: That is, if you have set up authentication using ENV or openrc, and it works, it will not call request_by_service_token
20:59 crinkle yep ^
21:01 xarses ok, so that confusion out of the way
21:02 xarses We need to have ENV | /root/openrc | /etc/keystone/keystone.conf ? Or can we initalize openstack lib somehow?
21:04 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use zuul-cloner and clone map to clone dependencies  https://review.openstack.org/190839
21:05 richm xarses: not sure what you mean by "initalize openstack lib"
21:06 crinkle xarses: it will figure it out automagically if you either have OS_* variables set in your environment, or you have a /root/openrc, or you have it in keystone.conf in that order
21:06 crinkle hmm i guess it's saturday for gildub, he probably won't be online to answer questions
21:10 xarses richm: I need to initalize whatever openstack lib is going to attempt to use for auth, I'd prefer to do it in a way that I don't have to clean up later, but at the same time, ENV AFAIK would have to be set before puppet starts, which I don't have a mech to do. Can we set the ENV and re-use it in the same puppet run?
21:12 richm ENV would have to be set before puppet starts
21:13 richm I suppose you could create /root/openrc or /etc/keystone/keystone.conf very early, before any keystone resources are used
21:13 crinkle the openstack_extras module can be used to create /root/openrc
21:13 richm I'm not sure what you mean by "set the ENV and re-use it"
21:21 openstackgerrit Colleen Murphy proposed stackforge/puppet-openstacklib: Test dependency for https://review.openstack.org/#/c/190839/  https://review.openstack.org/191223
21:22 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use zuul-cloner and clone map to clone dependencies  https://review.openstack.org/190839
21:37 xarses richm: i mean that I don't think I can set up ENV items in the expect them to take affect in the same puppet run, I think they'd have to be set before the start
21:51 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use zuul-cloner and clone map to clone dependencies  https://review.openstack.org/190839
21:57 richm xarses: you can set them and they will apply if you can set them in the same process or in the parent process
22:18 chem joined #puppet-openstack
22:43 aimon joined #puppet-openstack
22:56 aimon_ joined #puppet-openstack
23:22 xarses richm: crinkle thanks
23:22 openstackgerrit joined #puppet-openstack
23:50 openstackgerrit Jeremy Stanley proposed openstack/puppet-ceilometer: Update .gitreview file for project rename  https://review.openstack.org/191285
23:50 openstackgerrit Jeremy Stanley proposed openstack/puppet-cinder: Update .gitreview file for project rename  https://review.openstack.org/191286
23:50 openstackgerrit Jeremy Stanley proposed openstack/puppet-designate: Update .gitreview file for project rename  https://review.openstack.org/191287
23:50 openstackgerrit Jeremy Stanley proposed openstack/puppet-glance: Update .gitreview file for project rename  https://review.openstack.org/191288
23:50 openstackgerrit Jeremy Stanley proposed openstack/puppet-gnocchi: Update .gitreview file for project rename  https://review.openstack.org/191289
23:50 openstackgerrit Jeremy Stanley proposed openstack/puppet-heat: Update .gitreview file for project rename  https://review.openstack.org/191290
23:50 openstackgerrit Jeremy Stanley proposed openstack/puppet-horizon: Update .gitreview file for project rename  https://review.openstack.org/191291
23:50 openstackgerrit Jeremy Stanley proposed openstack/puppet-ironic: Update .gitreview file for project rename  https://review.openstack.org/191292
23:50 openstackgerrit Jeremy Stanley proposed openstack/puppet-keystone: Update .gitreview file for project rename  https://review.openstack.org/191293
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-manila: Update .gitreview file for project rename  https://review.openstack.org/191294
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-modulesync-configs: Update .gitreview file for project rename  https://review.openstack.org/191295
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-monasca: Update .gitreview file for project rename  https://review.openstack.org/191296
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-neutron: Update .gitreview file for project rename  https://review.openstack.org/191297
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-nova: Update .gitreview file for project rename  https://review.openstack.org/191298
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-openstack-specs: Update .gitreview file for project rename  https://review.openstack.org/191299
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-openstack_extras: Update .gitreview file for project rename  https://review.openstack.org/191300
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-openstacklib: Update .gitreview file for project rename  https://review.openstack.org/191301
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-sahara: Update .gitreview file for project rename  https://review.openstack.org/191302
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-swift: Update .gitreview file for project rename  https://review.openstack.org/191303
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-tempest: Update .gitreview file for project rename  https://review.openstack.org/191304
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-tripleo: Update .gitreview file for project rename  https://review.openstack.org/191305
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-trove: Update .gitreview file for project rename  https://review.openstack.org/191306
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-tuskar: Update .gitreview file for project rename  https://review.openstack.org/191307
23:51 openstackgerrit Jeremy Stanley proposed openstack/puppet-vswitch: Update .gitreview file for project rename  https://review.openstack.org/191308
23:53 openstackgerrit Richard Megginson proposed openstack/puppet-keystone: support for keystone v3 api - keystone and keystone::roles::admin  https://review.openstack.org/175060
23:54 openstackgerrit Richard Megginson proposed openstack/puppet-keystone: support for keystone v3 api - add domains to service_identity  https://review.openstack.org/180586
23:54 openstackgerrit Richard Megginson proposed openstack/puppet-keystone: support for keystone v3 api - beaker tests  https://review.openstack.org/178828
23:55 xarses joined #puppet-openstack
23:56 xarses joined #puppet-openstack

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary