*** xingchao has joined #puppet-openstack | 00:00 | |
openstackgerrit | Paul Belanger proposed openstack/puppet-keystone: Do not merge https://review.openstack.org/214901 | 00:03 |
---|---|---|
openstackgerrit | Paul Belanger proposed openstack/puppet-keystone: Do not merge https://review.openstack.org/214901 | 00:03 |
*** xingchao has quit IRC | 00:04 | |
*** morazi has joined #puppet-openstack | 00:38 | |
*** xingchao has joined #puppet-openstack | 00:39 | |
*** saneax has quit IRC | 00:42 | |
*** tiswanso has joined #puppet-openstack | 00:48 | |
*** tiswanso has quit IRC | 00:50 | |
*** tiswanso has joined #puppet-openstack | 00:51 | |
*** xingchao has quit IRC | 00:58 | |
openstackgerrit | Merged openstack/puppet-cinder: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/189863 | 01:07 |
openstackgerrit | Merged openstack/puppet-keystone: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/189871 | 01:07 |
openstackgerrit | Merged openstack/puppet-designate: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/189864 | 01:11 |
openstackgerrit | Merged openstack/puppet-vswitch: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/189883 | 01:11 |
*** xingchao has joined #puppet-openstack | 01:12 | |
*** chem has quit IRC | 01:13 | |
openstackgerrit | Merged openstack/puppet-ironic: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/189870 | 01:13 |
*** xarses has quit IRC | 01:15 | |
gildub | Please core members review https://review.openstack.org/213598 | 01:15 |
openstackgerrit | Merged openstack/puppet-openstack_extras: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/189876 | 01:15 |
gildub | Please core members also review https://review.openstack.org/213603 | 01:16 |
openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] And example commit to keep beaker-jobs and integration happy https://review.openstack.org/214897 | 01:19 |
openstackgerrit | Merged stackforge/puppet-ceph: use full length APT key https://review.openstack.org/209981 | 01:23 |
openstackgerrit | Merged openstack/puppet-heat: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/189868 | 01:24 |
openstackgerrit | Merged openstack/puppet-neutron: lint: Add and fix missing parameter documentation https://review.openstack.org/210059 | 01:24 |
openstackgerrit | Merged openstack/puppet-neutron: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/209122 | 01:27 |
openstackgerrit | Merged openstack/puppet-ceilometer: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/209121 | 01:29 |
*** saneax has joined #puppet-openstack | 01:32 | |
*** leanderthal has joined #puppet-openstack | 01:36 | |
*** xarses has joined #puppet-openstack | 01:41 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_trust for Keystone V3 API https://review.openstack.org/200996 | 01:48 |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_group for Keystone V3 API https://review.openstack.org/202409 | 01:48 |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_group_role for Keystone V3 API https://review.openstack.org/207755 | 01:49 |
*** saneax has quit IRC | 02:00 | |
*** TC01 has quit IRC | 02:22 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Keystone Role/Group resource provider https://review.openstack.org/211444 | 02:30 |
*** vinsh has quit IRC | 02:42 | |
*** vinsh has joined #puppet-openstack | 02:42 | |
*** markvoelker has quit IRC | 02:46 | |
*** vinsh has quit IRC | 02:46 | |
*** vinsh has joined #puppet-openstack | 03:00 | |
*** vinsh has quit IRC | 03:05 | |
*** gildub has quit IRC | 03:07 | |
*** TC01 has joined #puppet-openstack | 03:31 | |
*** TC01 has quit IRC | 03:35 | |
*** TC01 has joined #puppet-openstack | 03:41 | |
*** morazi has quit IRC | 03:44 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_group for Keystone V3 API https://review.openstack.org/202409 | 03:45 |
*** markvoelker has joined #puppet-openstack | 03:46 | |
*** gildub has joined #puppet-openstack | 03:46 | |
*** markvoelker has quit IRC | 03:51 | |
*** zacdev has quit IRC | 03:53 | |
*** tiswanso has quit IRC | 04:05 | |
*** TC01 has quit IRC | 04:16 | |
gildub | Any core members around? | 04:52 |
gildub | bodepd, ^ | 04:53 |
* gildub thinking it would be good to have core members in TZ around the clock | 04:55 | |
* gildub then we'll be almost a 24/7 shop :) | 04:56 | |
*** TC01 has joined #puppet-openstack | 05:04 | |
*** danieru has quit IRC | 05:10 | |
*** saneax has joined #puppet-openstack | 05:13 | |
*** saneax has quit IRC | 05:39 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_group for Keystone V3 API https://review.openstack.org/202409 | 05:51 |
*** danieru has joined #puppet-openstack | 05:52 | |
*** TC01 has quit IRC | 06:08 | |
gildub | michchap_, could you please review https://review.openstack.org/213598 and https://review.openstack.org/213603 ? | 06:18 |
*** leanderthal has quit IRC | 06:27 | |
gildub | michchap_, seems like this one is not up to date: https://launchpad.net/~puppet-manager-core/+members#active | 06:30 |
michchap_ | gildub: definitely not. | 06:31 |
gildub | michchap_, you, xingchao and EmilienM | 06:32 |
michchap_ | gildub: there's 12 core members at the moment | 06:33 |
gildub | michchap_, is any list available somewhere? | 06:33 |
xingchao | gildub: hi, what's up? | 06:33 |
michchap_ | gildub: the group in gerrit called puppet-manager-core is the definitive list. | 06:34 |
gildub | xingchao, hi, was trying to find an exhaustive list of core member (puppet-manager-core) | 06:34 |
barthalion | I think gerrit might be better place to look for it | 06:35 |
gildub | Fantastic I didn't know the group was there | 06:35 |
xingchao | gildub: https://review.openstack.org/#/admin/groups/134,members FYI | 06:35 |
gildub | Ok, so since we need +2 times too, would it be appropriate to directly add the group to the reviewer list? I mean easier than trying to find two individuals, right? | 06:37 |
gildub | michchap_, xingchao ^ | 06:37 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-designate: Add designate::mdns class https://review.openstack.org/214976 | 06:37 |
michchap_ | gildub: from my end I was away from the project for ~6 months but when I'm reviewing I usually use the watched projects tab, and have all the puppet projects in my watch list. | 06:38 |
xingchao | gildub: yes, you can add puppet-manager-core group as the reviewer. | 06:38 |
michchap_ | I'm not sure that adding every core to every review is a good idea since we'll end up with a torrent of email. | 06:39 |
*** stamak has joined #puppet-openstack | 06:39 | |
*** leanderthal has joined #puppet-openstack | 06:39 | |
gildub | michchap_, do you mean all the core member have always the full list in their tabs without necessarily being in the reviewer list? | 06:39 |
michchap_ | gildub: to be honest I have no idea how others do it - I just copied bodepd when I started :) | 06:40 |
gildub | michchap_, xingchao, from my experience, this is too busy, so unless poking around for reviews.... | 06:40 |
gildub | michchap_, fair enough! ^^ | 06:41 |
gildub | xingchao, could you please review https://review.openstack.org/213603 ? | 06:41 |
* gildub I hope I would never become a core member otherwise I'd be tempted to take an alias to get my code moving! Come-on, I'm just kidding!!! | 06:43 | |
xingchao | gildub: ok, I will review your patch soon. | 06:44 |
gildub | xingchao, thank you | 06:46 |
*** stamak has quit IRC | 06:52 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Added resource ordering for acceptance tests https://review.openstack.org/214984 | 07:07 |
*** jtomasek has joined #puppet-openstack | 07:09 | |
*** paramite has joined #puppet-openstack | 07:10 | |
*** linkedinyou has quit IRC | 07:27 | |
*** sanjayu_ has joined #puppet-openstack | 07:33 | |
*** sanjayu has quit IRC | 07:36 | |
openstackgerrit | Xingchao Yu proposed openstack/puppet-designate: Use generic_service to manage services https://review.openstack.org/215000 | 07:42 |
*** jpena has joined #puppet-openstack | 07:48 | |
*** markvoelker has joined #puppet-openstack | 07:48 | |
*** fedexo has quit IRC | 07:48 | |
openstackgerrit | Merged openstack/puppet-keystone: Purely cosmetic change on keystone provider https://review.openstack.org/213598 | 07:52 |
*** markvoelker has quit IRC | 07:53 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Authentication URLs and endpoint clarity re-factor https://review.openstack.org/213603 | 07:54 |
openstackgerrit | Vasyl Saienko proposed openstack/puppet-keystone: add --domain option when calling user list https://review.openstack.org/213906 | 07:55 |
*** stamak has joined #puppet-openstack | 07:58 | |
*** LimorStotland has quit IRC | 08:03 | |
*** derekh has joined #puppet-openstack | 08:08 | |
*** LimorStotland has joined #puppet-openstack | 08:09 | |
*** rochaporto has joined #puppet-openstack | 08:10 | |
*** paramite is now known as paramite|afk | 08:11 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_group for Keystone V3 API https://review.openstack.org/202409 | 08:13 |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Added resource ordering for acceptance tests https://review.openstack.org/214984 | 08:15 |
*** paramite|afk is now known as paramite | 08:17 | |
*** chem has joined #puppet-openstack | 08:21 | |
*** dgurtner has joined #puppet-openstack | 08:22 | |
*** dgurtner has quit IRC | 08:22 | |
*** dgurtner has joined #puppet-openstack | 08:22 | |
openstackgerrit | Xingchao Yu proposed openstack/puppet-keystone: Endpoints are APIs version-less https://review.openstack.org/213957 | 08:23 |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_group for Keystone V3 API https://review.openstack.org/202409 | 08:23 |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Drop no longer supported releases https://review.openstack.org/214104 | 08:26 |
*** gildub has quit IRC | 08:26 | |
*** xingchao has quit IRC | 08:28 | |
*** jistr has joined #puppet-openstack | 08:28 | |
*** c0m0 has joined #puppet-openstack | 08:49 | |
*** jpena_ has joined #puppet-openstack | 08:55 | |
*** jpena has quit IRC | 08:56 | |
*** v1k0d3n has quit IRC | 09:05 | |
*** jpena_ is now known as jpena | 09:06 | |
*** stamak has quit IRC | 09:19 | |
*** jistr has quit IRC | 09:19 | |
*** jistr has joined #puppet-openstack | 09:20 | |
*** ddmitriev has quit IRC | 09:24 | |
*** stamak has joined #puppet-openstack | 09:27 | |
*** stamak has quit IRC | 09:28 | |
*** stamak has joined #puppet-openstack | 09:29 | |
*** stamak has quit IRC | 09:30 | |
*** stamak has joined #puppet-openstack | 09:31 | |
*** jpena has quit IRC | 09:36 | |
*** degorenko has joined #puppet-openstack | 09:39 | |
degorenko | hello folks! Take a look on https://review.openstack.org/#/c/205988/ and https://review.openstack.org/#/c/198744/ please :) | 09:47 |
*** jpena has joined #puppet-openstack | 09:49 | |
*** markvoelker has joined #puppet-openstack | 09:49 | |
openstackgerrit | Vasyl Saienko proposed openstack/puppet-keystone: [DNOTMERGE] https://review.openstack.org/215044 | 09:50 |
*** markvoelker has quit IRC | 09:54 | |
*** chem has quit IRC | 10:08 | |
*** linkedinyou has joined #puppet-openstack | 10:20 | |
*** dgurtner_ has joined #puppet-openstack | 10:48 | |
*** dgurtner_ has quit IRC | 10:48 | |
*** dgurtner_ has joined #puppet-openstack | 10:48 | |
openstackgerrit | Lukas Bezdicka proposed openstack/puppet-neutron: Run neutron db sync also for each neutron module https://review.openstack.org/200877 | 10:50 |
*** dgurtner has quit IRC | 10:51 | |
*** xingchao has joined #puppet-openstack | 10:51 | |
*** dgurtner_ has quit IRC | 10:52 | |
openstackgerrit | Xingchao Yu proposed openstack/puppet-designate: Use generic_service to manage services https://review.openstack.org/215000 | 10:54 |
*** dgurtner has joined #puppet-openstack | 10:54 | |
*** xingchao has quit IRC | 10:55 | |
*** ddmitriev has joined #puppet-openstack | 10:57 | |
*** v1k0d3n has joined #puppet-openstack | 11:06 | |
*** dgurtner_ has joined #puppet-openstack | 11:06 | |
*** dgurtner has quit IRC | 11:10 | |
*** v1k0d3n has quit IRC | 11:10 | |
*** dgurtner_ has quit IRC | 11:11 | |
*** dgurtner has joined #puppet-openstack | 11:13 | |
openstackgerrit | Merged openstack/puppet-keystone: Clarify the origin of provider warning messages https://review.openstack.org/212290 | 11:15 |
*** dprince has joined #puppet-openstack | 11:35 | |
*** markvoelker has joined #puppet-openstack | 11:35 | |
*** markvoelker has quit IRC | 11:40 | |
openstackgerrit | Vasyl Saienko proposed openstack/puppet-horizon: add api_versions parameter https://review.openstack.org/211650 | 11:43 |
*** xingchao has joined #puppet-openstack | 11:45 | |
*** morazi has joined #puppet-openstack | 11:46 | |
*** xingchao has quit IRC | 11:50 | |
*** v1k0d3n has joined #puppet-openstack | 12:03 | |
*** derekh is now known as derekh_afk | 12:03 | |
*** jayg|g0n3 is now known as jayg | 12:04 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Authentication URLs and endpoint clarity re-factor https://review.openstack.org/213603 | 12:04 |
EmilienM | hello | 12:09 |
*** jpena is now known as jpena|lunch | 12:10 | |
iurygregory | good morning | 12:11 |
iurygregory | thanks crinkle for the +A ^^ | 12:11 |
EmilienM | iurygregory: http://specs.openstack.org/openstack/puppet-openstack-specs/specs/liberty/enabling-federation.html | 12:13 |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Endpoints are APIs version-less https://review.openstack.org/213957 | 12:13 |
*** gildub has joined #puppet-openstack | 12:15 | |
gildub | EmilienM, This backport to stable/kilo is clear to go, but since I didn't do the -x properly I need a core member to look at it, if necessary I'm happy to abandon and re-cherry pick it: https://review.openstack.org/#/c/213938/ | 12:17 |
*** ducttape_ has joined #puppet-openstack | 12:18 | |
EmilienM | gildub: why do you need a core member to look at it? | 12:18 |
gildub | EmilienM, the only justification for it is because of clarity for the 'real' patch coming next, to be backported too: https://review.openstack.org/213603 | 12:18 |
gildub | EmilienM, that's my understanding from the https://wiki.openstack.org/wiki/StableBranch pages, but I'm still new to the process | 12:19 |
*** ducttape_ has quit IRC | 12:19 | |
EmilienM | gildub: is the final goal, to backport https://review.openstack.org/#/c/213957 once it's merged? | 12:19 |
*** dgurtner_ has joined #puppet-openstack | 12:19 | |
*** dgurtner_ has quit IRC | 12:19 | |
*** dgurtner_ has joined #puppet-openstack | 12:19 | |
EmilienM | gildub: go ahead with the -x and people will review | 12:19 |
*** ducttape_ has joined #puppet-openstack | 12:19 | |
iurygregory | \o/ thanks EmilienM =D | 12:20 |
gildub | EmilienM, no the final goal is https://review.openstack.org/213603 | 12:20 |
gildub | EmilienM, https://review.openstack.org/#/c/213957 can't be merged in kilo, it's not supported unless OSC gets bumped up to 1.6.x, which won't happen | 12:21 |
EmilienM | gildub: why do you need better readability in stable branch? | 12:21 |
EmilienM | we backports bugs and features usually | 12:21 |
EmilienM | it looks cosmetic | 12:21 |
gildub | EmilienM, I know, as I said the goal is https://review.openstack.org/213603, coming next, that should help with v2/v3 issues | 12:22 |
*** dgurtner has quit IRC | 12:22 | |
EmilienM | gildub: anyway, please do the patches with -x and we're good | 12:23 |
gildub | EmilienM, but the first one, as you commented wasn't done with it (-x), so it's in kilo/branch, is that all right? | 12:23 |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Be more specific with CentOS7 package exclude https://review.openstack.org/214093 | 12:24 |
*** dgurtner_ has quit IRC | 12:24 | |
gildub | EmilienM, also since the second one depends on the first, I can cherry pick it until first one has been merged, correct? | 12:24 |
gildub | EmilienM, sorry, typo ^, I meant, I cannot cherry pick.... | 12:25 |
*** paramite is now known as paramite|afk | 12:25 | |
*** dgurtner has joined #puppet-openstack | 12:26 | |
gildub | EmilienM, ok, I got it, I'm adding the -x to the first. And I'll wait for the second 'cause the deps. | 12:26 |
EmilienM | gildub: to cherry-pick patches, you should be patient and wait they are merged in 'master'. Once it's done, you create a branch from stable/kilo and do the cherry-pick -x and you'll have Gerrit dependencies. You run git review stable/kilo and the magic happens | 12:27 |
iurygregory | i think marek got angry with the spec =X | 12:28 |
gildub | EmilienM, Thanks, I got that now. My question is because I initially did not wait for the merge in master, is the review clean? | 12:30 |
*** vinsh has joined #puppet-openstack | 12:35 | |
*** zacdev has joined #puppet-openstack | 12:39 | |
*** markvoelker has joined #puppet-openstack | 12:39 | |
*** ducttape_ has quit IRC | 12:40 | |
openstackgerrit | Merged stackforge/puppet-ceph: Drop no longer supported releases https://review.openstack.org/214104 | 12:41 |
*** xingchao has joined #puppet-openstack | 12:43 | |
gildub | EmilienM, https://review.openstack.org/213598 looks good, I leave it with the core to confirm | 12:45 |
EmilienM | iurygregory: what? | 12:47 |
*** jpena|lunch is now known as jpena | 12:47 | |
iurygregory | marek in openstack-keystone told | 12:49 |
iurygregory | marek rodrigo: https://review.openstack.org/#/c/190361/28 so i am really looking forward to see code and capabilities of those puppet module. It's still full of inconsistencies... "rodrigo" they talk about supporting saml and oidc and then don't give a ** about oidc and want to do this from one module? | 12:50 |
iurygregory | I'm trying to explain to him that there is no inconsistencies | 12:50 |
*** jistr is now known as jistr|mtg | 12:53 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Authentication URLs and endpoint clarity re-factor https://review.openstack.org/213603 | 12:55 |
*** paramite|afk is now known as paramite | 12:55 | |
*** xingchao has quit IRC | 12:56 | |
*** dachary_ has joined #puppet-openstack | 12:56 | |
dachary_ | Hey | 12:56 |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Endpoints are APIs version-less https://review.openstack.org/213957 | 12:57 |
*** dachary_ has left #puppet-openstack | 13:02 | |
*** richm has joined #puppet-openstack | 13:02 | |
*** Atmesh has joined #puppet-openstack | 13:08 | |
iurygregory | EmilienM, when running bundle exec rspec spec/acceptance for first time everything is ok | 13:14 |
iurygregory | (68 examples, 0 failures) then if i run only basic_keystone_spec.rb (34 examples, 2 failures) | 13:14 |
iurygregory | Failed examples: basic keystone server with resources default parameters Port "5000"/"35357" should be listening. Keystone log says Address already in use (http://paste.openstack.org/show/422669/) | 13:14 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Purely cosmetic change on keystone provider https://review.openstack.org/213938 | 13:16 |
iurygregory | hey richm, about the SP implementation you think we need only one patch or one for each possible choice? (openid, saml shib/mellon) | 13:21 |
openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Add an ability to set up drop_flows_on_start option https://review.openstack.org/211986 | 13:22 |
*** Atmesh has quit IRC | 13:23 | |
*** bogdando has quit IRC | 13:25 | |
*** bogdando has joined #puppet-openstack | 13:25 | |
*** ducttape_ has joined #puppet-openstack | 13:27 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Added resource ordering for acceptance tests https://review.openstack.org/214984 | 13:30 |
*** xingchao has joined #puppet-openstack | 13:34 | |
*** prad_ has joined #puppet-openstack | 13:36 | |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Added resource ordering for acceptance tests https://review.openstack.org/214984 | 13:37 |
openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Keystone Role/Group resource provider https://review.openstack.org/211444 | 13:38 |
*** tiswanso has joined #puppet-openstack | 13:40 | |
*** ducttape_ has quit IRC | 13:40 | |
*** gildub has quit IRC | 13:44 | |
*** prad_ is now known as pradk | 13:47 | |
*** degorenko has quit IRC | 13:58 | |
*** degorenko has joined #puppet-openstack | 13:59 | |
*** jistr|mtg is now known as jistr | 14:08 | |
*** ducttape_ has joined #puppet-openstack | 14:09 | |
*** cmh has joined #puppet-openstack | 14:11 | |
*** dfisher has quit IRC | 14:16 | |
*** dfisher has joined #puppet-openstack | 14:17 | |
openstackgerrit | Javier Peña proposed openstack/puppet-swift: Add support for swift-object-expirer service https://review.openstack.org/211549 | 14:18 |
cmh | Hello. Does anyone here have experience using the puppet keystone module? I'm trying to use the v3_basic.pp example but I'm encountering an error with the openstack client (you can view the error here: https://gist.github.com/Chase-H/911e7e256045828d827a). I've tried updating the client to the latest version using Pip but it still fails. Is there a different rdo repo I should be using to get the latest version of the openstackcl | 14:23 |
*** saneax has joined #puppet-openstack | 14:25 | |
*** paramite is now known as paramite|afk | 14:26 | |
*** finite has joined #puppet-openstack | 14:26 | |
*** djc_ has joined #puppet-openstack | 14:27 | |
openstackgerrit | Merged openstack/puppet-swift: Add tempurl middleware options https://review.openstack.org/206655 | 14:28 |
finite | cmh do you know if the liberty repositories have the v3 client you're searching for? | 14:28 |
*** paramite|afk is now known as paramite | 14:29 | |
cmh | finite I haven't been able to find the rdo liberty repos. I think rdo release is still at kilo. | 14:29 |
jistr | EmilienM: hi, could you please explain what you meant here? https://review.openstack.org/210350 Was the comment meant to be made on a different review maybe? | 14:30 |
jistr | EmilienM: oh i see | 14:31 |
EmilienM | jistr: :) | 14:31 |
EmilienM | it can be done in a different patch | 14:31 |
EmilienM | it's up to you | 14:31 |
EmilienM | I just wanted to take the opportunity to update it | 14:31 |
EmilienM | I've notice a lot of deprecated parameters in Liberty, we might need to fix that during our midcycle | 14:32 |
EmilienM | jistr: it's only a -1, people can't merge it I don't mind | 14:32 |
EmilienM | I just rise the issue | 14:32 |
jistr | ack | 14:32 |
EmilienM | raise* | 14:32 |
jistr | though what's in master right now is broken | 14:32 |
EmilienM | jistr: yeah | 14:32 |
jistr | yeah i think fixing bugs vs. moving config to liberty are two different things | 14:33 |
EmilienM | jistr++ | 14:34 |
EmilienM | jistr: I can merge it | 14:34 |
EmilienM | jistr: done | 14:34 |
EmilienM | jistr: if you could also take care of the moving config thing ;) | 14:34 |
jistr | EmilienM: thanks :) we have puppet-cinder pinned down to an old version because of that | 14:35 |
finite | Does anyone have a minute to help out cmh and myself with this v3 issue? | 14:35 |
EmilienM | finite: https://github.com/openstack/puppet-keystone/blob/master/spec/acceptance/keystone_wsgi_apache_spec.rb#L11-L43 | 14:35 |
EmilienM | finite: this is how to make Liberty working | 14:36 |
jistr | EmilienM: yeah, let's add it to the backlog though because i'm now trying to move forward on partner stuff | 14:36 |
EmilienM | jistr: please do :) | 14:36 |
EmilienM | jistr: please use https://trello.com/b/4X3zxWRZ/on-going-effort | 14:36 |
jistr | EmilienM: can you give me edit rights? this is me https://trello.com/jistr | 14:38 |
finite | Thanks, EmilienM | 14:38 |
EmilienM | sure | 14:38 |
EmilienM | jistr: done | 14:39 |
jistr | EmilienM: thx | 14:41 |
vinsh | EmilienM: What do you think about Guilherme's change here: https://review.openstack.org/#/c/205243/ | 14:42 |
vinsh | It makes sense to me. I am considering pulling it into my current patch before it is merged. It looks to be helpful. | 14:43 |
*** Atmesh has joined #puppet-openstack | 14:44 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Initial run_tests.sh script https://review.openstack.org/210412 | 14:44 |
jistr | EmilienM: added https://trello.com/c/zzPhu8s1/61-move-lock-path-config-option-in-puppet-cinder-to-follow-liberty-oslo | 14:45 |
EmilienM | jistr: thx | 14:45 |
EmilienM | vinsh: looking now | 14:45 |
vinsh | TY | 14:45 |
EmilienM | jistr: I would renaming it to "update config for liberty" | 14:45 |
EmilienM | jistr: so we can put the work down to tasks/patches | 14:45 |
EmilienM | vinsh: well, this is something we might consider indeed | 14:47 |
EmilienM | vinsh: I'm just worried about backward compatibility | 14:47 |
EmilienM | vinsh: because we are changing the ordering right now | 14:47 |
EmilienM | so I guess some params (maybe) will change | 14:47 |
degorenko | folks, who have free time take a look on https://review.openstack.org/#/c/205988/ and https://review.openstack.org/#/c/198744/ please :) | 14:48 |
*** Atmesh has quit IRC | 14:48 | |
openstackgerrit | Vasyl Saienko proposed openstack/puppet-keystone: [DNOTMERGE] https://review.openstack.org/215044 | 14:48 |
vinsh | EmilienM: Do you mean we might see different results because the template is now applied in the correct order? | 14:50 |
EmilienM | yeah | 14:50 |
*** derekh_afk is now known as derekh | 14:50 | |
vinsh | I would suggest its better to take the pain of using it correctly now then later :) | 14:51 |
EmilienM | vinsh: we still have to verify backward compatiblity | 14:52 |
vinsh | The beaker upgrade test sort of does that | 14:52 |
vinsh | I will pull this patch in locally in my dev env and check before/after of conf files. | 14:53 |
*** dgurtner_ has joined #puppet-openstack | 14:53 | |
EmilienM | vinsh: no it does not test it | 14:54 |
EmilienM | vinsh: but almost | 14:54 |
EmilienM | we need to compare swift config before and after | 14:55 |
EmilienM | with default values | 14:55 |
vinsh | Yeah, it could be a subtle config change that test wouldn't point out. | 14:55 |
EmilienM | and see the changes and how they affect swift serviecs | 14:55 |
openstackgerrit | Merged openstack/puppet-cinder: Fix "Add lock_path to cinder config" https://review.openstack.org/210350 | 14:55 |
*** dgurtner has quit IRC | 14:57 | |
*** dgurtner_ has quit IRC | 14:58 | |
EmilienM | vinsh: I dropped a comment | 15:01 |
vinsh | Thanks, good learning for me too. | 15:01 |
*** dgurtner has joined #puppet-openstack | 15:06 | |
*** cmh_ has joined #puppet-openstack | 15:08 | |
*** cmh has quit IRC | 15:09 | |
finite | EmilienM We're still getting `Execution of '/bin/openstack domain list --quiet --format csv' returned 2: openstack: 'domain' is not an openstack command. See 'openstack --help’` We're using python-openstackclient-1.6.1-dev8.el7.centos.noarch which should have that, correct? | 15:10 |
*** cmh_ has quit IRC | 15:11 | |
*** cmh has joined #puppet-openstack | 15:11 | |
*** dprince has quit IRC | 15:12 | |
* cmh slaps finite around a bit with a large fishbot | 15:12 | |
*** jistr is now known as jistr|mtg | 15:22 | |
finite | bump for previous question | 15:26 |
cmh | @mfisch: Had a colleague tell me you might be able to help finite and I with our puppet keystone issue. Do you have a minute? | 15:33 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Initial run_tests.sh script https://review.openstack.org/210412 | 15:34 |
vinsh | cmh: He and a bunch of others are on airplanes returning from midcycle meetup. | 15:37 |
vinsh | Ping him again tonight or tomorrow, I bet he can help. | 15:37 |
*** cmh has quit IRC | 15:39 | |
vinsh | EmilienM: Do we have any policy in puppet-openstack about naming conventions for services? | 15:44 |
vinsh | Meaning.. MUST they start with the service name? or does that matter? | 15:44 |
EmilienM | vinsh: can you give an example ? | 15:44 |
vinsh | For example, in swift (my favorite ;) ) there is service "swift-proxy" and maybe "swift-object-replicator" | 15:45 |
vinsh | could those instead be "proxy-sever" and "object-replicator" | 15:45 |
vinsh | *server | 15:45 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-manila: Add rabbit_ha_queues option https://review.openstack.org/215201 | 15:45 |
vinsh | They each have the tag of "swift-service" as well. | 15:46 |
vinsh | Where I am leading to, is it simplifies things when using swift-init if we drop the "swift-" prefix. | 15:46 |
EmilienM | I think you should keep swift- | 15:46 |
*** stamak has quit IRC | 15:47 | |
*** paramite has quit IRC | 15:47 | |
vinsh | Ok, so sort of a policy on it. | 15:47 |
*** LimorStotland has quit IRC | 15:47 | |
*** cmh has joined #puppet-openstack | 15:47 | |
*** angdraug has joined #puppet-openstack | 15:51 | |
*** hogepodge has quit IRC | 15:53 | |
*** sergmelikyan has joined #puppet-openstack | 15:54 | |
*** AlexeyElagin has joined #puppet-openstack | 15:57 | |
xingchao | EmilienM: hi, for now, what's the process of submiting a new puppet module ? any link for specify? | 15:59 |
EmilienM | yes it's documented | 15:59 |
EmilienM | a sec | 15:59 |
*** Andreyy has joined #puppet-openstack | 15:59 | |
EmilienM | xingchao: https://wiki.openstack.org/wiki/Puppet/New_module | 16:00 |
EmilienM | kudos to spredzy & sbadia | 16:00 |
xingchao | EmilienM: aha, thx :) | 16:00 |
Andreyy | Guys, anyone can please review https://review.openstack.org/#/c/209997/ ? | 16:01 |
*** Andreyy has quit IRC | 16:03 | |
*** cmh has quit IRC | 16:04 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Initial run_tests.sh script https://review.openstack.org/210412 | 16:04 |
*** skolekonov has joined #puppet-openstack | 16:05 | |
*** jistr|mtg is now known as jistr | 16:10 | |
*** djc_ has quit IRC | 16:10 | |
*** finite has quit IRC | 16:17 | |
openstackgerrit | Lukas Bezdicka proposed openstack/puppet-neutron: Run neutron db sync also for each neutron module https://review.openstack.org/200877 | 16:20 |
openstackgerrit | Merged openstack/puppet-gnocchi: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/189866 | 16:20 |
*** xingchao has quit IRC | 16:21 | |
*** hogepodge has joined #puppet-openstack | 16:23 | |
openstackgerrit | Merged openstack/puppet-openstacklib: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/189875 | 16:25 |
openstackgerrit | Merged openstack/puppet-tempest: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/189879 | 16:30 |
*** pradk has quit IRC | 16:31 | |
ukitanuki | Is anyone around that has used the puppet-ceph module? Not sure if its best to follow the examples folder or the USECAE.md method | 16:32 |
openstackgerrit | Merged openstack/puppet-nova: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/189874 | 16:34 |
*** sac has quit IRC | 16:35 | |
*** rochaporto has quit IRC | 16:35 | |
openstackgerrit | Merged openstack/puppet-ceilometer: Reflect provider change in puppet-openstacklib https://review.openstack.org/209868 | 16:36 |
*** sac has joined #puppet-openstack | 16:36 | |
openstackgerrit | Merged openstack/puppet-heat: Reflect provider change in puppet-openstacklib https://review.openstack.org/209882 | 16:36 |
*** dprince has joined #puppet-openstack | 16:39 | |
openstackgerrit | Merged openstack/puppet-sahara: Reflect provider change in puppet-openstacklib https://review.openstack.org/209877 | 16:41 |
*** Guest20238 has joined #puppet-openstack | 16:44 | |
*** c0m0 has quit IRC | 16:46 | |
*** jistr has quit IRC | 16:46 | |
*** aimon has quit IRC | 16:46 | |
*** Guest20238 has quit IRC | 16:48 | |
*** IBerezovskiy has quit IRC | 16:53 | |
*** derekh has quit IRC | 17:00 | |
*** TC01_ has joined #puppet-openstack | 17:04 | |
*** sergmelikyan has quit IRC | 17:13 | |
*** sergmelikyan has joined #puppet-openstack | 17:13 | |
*** jpena has quit IRC | 17:15 | |
*** sergmelikyan has quit IRC | 17:15 | |
*** sergmelikyan has joined #puppet-openstack | 17:15 | |
*** aimon has joined #puppet-openstack | 17:23 | |
*** xarses has quit IRC | 17:30 | |
*** dgurtner has quit IRC | 17:34 | |
*** sergmelikyan has quit IRC | 17:41 | |
*** sergmelikyan has joined #puppet-openstack | 17:44 | |
*** xarses has joined #puppet-openstack | 17:49 | |
*** prad_ has joined #puppet-openstack | 17:49 | |
*** sergmelikyan has quit IRC | 17:57 | |
*** ducttape_ has quit IRC | 18:00 | |
*** prad_ is now known as pradk | 18:04 | |
*** tiswanso has quit IRC | 18:06 | |
*** tiswanso has joined #puppet-openstack | 18:11 | |
*** angdraug has quit IRC | 18:14 | |
*** tiswanso has quit IRC | 18:32 | |
*** sergmelikyan has joined #puppet-openstack | 18:34 | |
*** angdraug has joined #puppet-openstack | 18:39 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as IdentityProvider https://review.openstack.org/208054 | 18:57 |
richm | iurygregory: re: one patch per SP - yes, I think so | 19:05 |
*** TC01_ has quit IRC | 19:06 | |
*** tiswanso has joined #puppet-openstack | 19:07 | |
*** ducttape_ has joined #puppet-openstack | 19:10 | |
*** sergmelikyan has quit IRC | 19:13 | |
EmilienM | iurygregory: I have a few suggestions to make us win some time | 19:14 |
EmilienM | iurygregory: richm is right, iterations and git dependencies are our best friends - separate patches when it makes sense | 19:15 |
*** sergmelikyan has joined #puppet-openstack | 19:15 | |
EmilienM | iurygregory: don't focus on unit tests except if (like me) you code in TDD | 19:15 |
*** stamak has joined #puppet-openstack | 19:19 | |
xarses | EmilienM: pabelanger, dgurtener, did any thing come of that ceph integration conversation you started to have, I tried to catch up but ended up involved in something else | 19:20 |
EmilienM | xarses: for now, we are still building basic bits | 19:20 |
EmilienM | without ceph | 19:20 |
EmilienM | but one day will come we will have ceph in the party | 19:20 |
iurygregory | ok richm | 19:21 |
iurygregory | EmilienM, go i try to code in TDD =X but i'm not familiar with acceptance | 19:22 |
EmilienM | iurygregory: I use TDD only for unit test | 19:22 |
EmilienM | not for acceptance | 19:22 |
*** sergmelikyan has quit IRC | 19:22 | |
iurygregory | yeah i mean for classes | 19:22 |
EmilienM | iurygregory: acceptance is not really complicated, it's just manifests + serverspec | 19:22 |
EmilienM | oh, I suggest to learn by reading code | 19:23 |
EmilienM | and read doc maybe | 19:23 |
iurygregory | i'm doing this =) | 19:23 |
EmilienM | I guess you're not going to do providers so unit tests will be 'easy' | 19:23 |
iurygregory | i've learned to use bundle rake =) | 19:23 |
*** tiswanso has quit IRC | 19:23 | |
*** sergmelikyan has joined #puppet-openstack | 19:24 | |
iurygregory | the unit tests are just to ensure configurations i think | 19:24 |
EmilienM | yeah | 19:24 |
EmilienM | and different use cases | 19:24 |
EmilienM | if you have some logic in manifests, we need to validate the logic | 19:24 |
EmilienM | that's contexts | 19:24 |
EmilienM | iurygregory: look https://review.openstack.org/213315 | 19:25 |
iurygregory | ok | 19:25 |
*** sergmelikyan has quit IRC | 19:27 | |
*** sergmelikyan has joined #puppet-openstack | 19:27 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Initial run_tests.sh script https://review.openstack.org/210412 | 19:28 |
*** tiswanso has joined #puppet-openstack | 19:28 | |
*** stamak has quit IRC | 19:35 | |
*** richm has quit IRC | 19:38 | |
*** sergmelikyan has quit IRC | 19:42 | |
*** sergmelikyan has joined #puppet-openstack | 19:44 | |
*** cmh has joined #puppet-openstack | 19:46 | |
*** sergmelikyan has quit IRC | 19:47 | |
*** amg_ has joined #puppet-openstack | 19:51 | |
*** sergmelikyan has joined #puppet-openstack | 19:53 | |
*** richm has joined #puppet-openstack | 19:54 | |
*** sergmelikyan has quit IRC | 19:58 | |
*** sergmelikyan has joined #puppet-openstack | 19:59 | |
cmh | @mfisch: Are you there? amg_ and I are still looking at our puppet-keystone issue. | 19:59 |
*** stamak has joined #puppet-openstack | 20:04 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Initial run_tests.sh script https://review.openstack.org/210412 | 20:06 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Create puppet folder layout https://review.openstack.org/201298 | 20:12 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy MySQL and Keystone (WSGI) https://review.openstack.org/207070 | 20:12 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: configure and run tempest https://review.openstack.org/207078 | 20:13 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: CI - do not merge - testing https://review.openstack.org/#/c/210412/ https://review.openstack.org/214793 | 20:13 |
*** saneax has quit IRC | 20:18 | |
*** sergmelikyan has quit IRC | 20:21 | |
*** stamak has quit IRC | 20:31 | |
*** sergmelikyan has joined #puppet-openstack | 20:39 | |
iurygregory | richm, +1? O.o | 20:42 |
*** jtomasek has quit IRC | 20:42 | |
richm | iurygregory: ? | 20:46 |
iurygregory | https://review.openstack.org/#/c/208054/ | 20:46 |
richm | iurygregory: yes - looks good to me | 20:51 |
iurygregory | i'm discovering the -1 with jenkis =D | 20:52 |
iurygregory | tomorrow i'll send other patch ^^ | 20:52 |
*** tiswanso has quit IRC | 21:05 | |
*** dgurtner has joined #puppet-openstack | 21:13 | |
openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 21:13 |
*** ddmitriev has quit IRC | 21:15 | |
*** sergmelikyan has quit IRC | 21:16 | |
*** dprince has quit IRC | 21:16 | |
*** derekh has joined #puppet-openstack | 21:17 | |
openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 21:19 |
*** cmh has quit IRC | 21:19 | |
*** ddmitriev has joined #puppet-openstack | 21:21 | |
bodepd | dfisher: it's code to be able to specify based on role, how many of each role should proceed, as well as what roles need to wait for other roles. We want to eventually add the ability to be able to roll commands out to a fleet as well. | 21:23 |
dfisher | ooh. need some context, if you have it. I think I pinged you a few days ago and my tiny brain has forgotten what I sent | 21:24 |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Be more specific with CentOS7 package exclude https://review.openstack.org/214093 | 21:25 |
openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 21:26 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: [WIP] Support for Keystone as Identity Provider https://review.openstack.org/208054 | 21:27 |
openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 21:28 |
bodepd | dfisher: yeah, I mentioned something about orchestration code on teh channel, and responded to your request for comment :) | 21:31 |
dfisher | i'm … not sure that was me. | 21:31 |
dfisher | maybe. | 21:31 |
dfisher | i don't know :) | 21:31 |
bodepd | dfisher: no worries, it's probably from days/weeks ago, b/c I hardly exist on the internet anymore | 21:31 |
dfisher | but the internet is full of nice things. | 21:32 |
*** rochaporto has joined #puppet-openstack | 21:37 | |
*** ddmitriev has quit IRC | 21:38 | |
nibalizer | EmilienM: what are the upgrade jobs for | 21:40 |
nibalizer | how do I make them pass | 21:40 |
*** amg_ has quit IRC | 21:45 | |
openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 21:51 |
*** sergmelikyan has joined #puppet-openstack | 21:57 | |
*** aimon has quit IRC | 22:02 | |
*** sergmelikyan has quit IRC | 22:04 | |
EmilienM | nibalizer: you can't now with infra modules | 22:09 |
EmilienM | nibalizer: I'm back now | 22:09 |
EmilienM | they are very dumb now | 22:10 |
EmilienM | but the goal in the future is to deploy stable/kilo, run tempest, javelin, switch to the patchset in master (liberty) and see if openstack is upgraded | 22:10 |
EmilienM | and if resources survived | 22:10 |
nibalizer | kay | 22:10 |
EmilienM | nibalizer: so I guess we might need collaboration if you wanna use it | 22:10 |
nibalizer | I think we should do some project-config munging | 22:10 |
nibalizer | so that we don't get the upgrade jobs | 22:10 |
EmilienM | yeah! | 22:10 |
nibalizer | once you get them working | 22:10 |
nibalizer | we can maybe add them to infra | 22:10 |
EmilienM | nibalizer: have you seen how they work now? | 22:10 |
EmilienM | nibalizer: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/puppet-module-jobs.yaml#L66-L103 | 22:11 |
EmilienM | if you run it against infra modules *right now* they will fail because of https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/puppet-module-jobs.yaml#L76 | 22:11 |
EmilienM | you don't have stable branches while we do | 22:11 |
EmilienM | we might need to adjust that | 22:11 |
nibalizer | github link? | 22:12 |
* nibalizer slaps EmilienM on the wrist | 22:12 | |
nibalizer | git.openstack.org/cgit man | 22:12 |
EmilienM | lol | 22:13 |
EmilienM | nibalizer: http://goo.gl/dV2rb8 | 22:13 |
*** morazi has quit IRC | 22:25 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Initial run_tests.sh script https://review.openstack.org/210412 | 22:34 |
*** aimon has joined #puppet-openstack | 22:35 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Create puppet folder layout https://review.openstack.org/201298 | 22:37 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy MySQL and Keystone (WSGI) https://review.openstack.org/207070 | 22:43 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: configure and run tempest https://review.openstack.org/207078 | 22:43 |
*** aimon has quit IRC | 22:47 | |
*** aimon has joined #puppet-openstack | 22:48 | |
*** saneax has joined #puppet-openstack | 22:49 | |
*** aimon_ has joined #puppet-openstack | 22:54 | |
*** aimon has quit IRC | 22:57 | |
*** dfisher has quit IRC | 23:02 | |
*** sergmelikyan has joined #puppet-openstack | 23:03 | |
*** ducttape_ has quit IRC | 23:12 | |
*** zacdev has quit IRC | 23:23 | |
*** angdraug has quit IRC | 23:45 | |
*** benh57 has joined #puppet-openstack | 23:55 | |
*** gildub has joined #puppet-openstack | 23:55 | |
*** sergmelikyan has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!