| *** derekh has quit IRC | 00:04 | |
| *** sergmelikyan has joined #puppet-openstack | 00:07 | |
| *** sergmelikyan has quit IRC | 00:09 | |
| *** jayg is now known as jayg|g0n3 | 00:18 | |
| *** saneax has quit IRC | 00:19 | |
| openstackgerrit | willy lin proposed openstack/puppet-designate: Use .fixtures.yaml instead of .fixtures.yml https://review.openstack.org/215384 | 00:28 |
|---|---|---|
| EmilienM | gildub: I'm commenting in the review, but I can change my mind if mfisch or mgagne think we can drop the param | 00:28 |
| gildub | EmilienM, reading.... | 00:29 |
| EmilienM | gildub: not yet sent :) | 00:29 |
| gildub | EmilienM, ah, well I think we can't drop it because that might break existing manifest overriding the value. | 00:30 |
| gildub | EmilienM, so even if the value is not used anymore, I think we have to have a deprecation period (with the warning I forgot) | 00:30 |
| gildub | EmilienM, if that makes sense | 00:30 |
| EmilienM | gildub: sent | 00:31 |
| gildub | EmilienM, k | 00:31 |
| EmilienM | gildub: you are totally breaking backward compatibility | 00:31 |
| gildub | EmilienM, I'm not, keystone is! | 00:31 |
| EmilienM | gildub: you need to carefully keep our manifests compatible | 00:31 |
| EmilienM | gildub: this is one of our challenges, to keep the interface not changing every day | 00:32 |
| EmilienM | gildub: so our end users can see warnings in the catalog that tell them to update their manifests | 00:32 |
| EmilienM | here you're just dropping the parameter BOOM | 00:32 |
| gildub | EmilienM, not BOOM, I'm suggesting to not drop the version number so no catalog breaks but it's ignored because the version number is useless anyway with keystone v3, it will determine the API version from the context | 00:33 |
| EmilienM | yeah | 00:34 |
| EmilienM | again I'm happy if mfisch or mgagne say it's fine | 00:34 |
| gildub | EmilienM, okie dokie | 00:34 |
| EmilienM | gildub: though, please do not '' -> this is not puppetish | 00:34 |
| EmilienM | undef is cool man | 00:34 |
| EmilienM | I still wonder why people use '' | 00:34 |
| gildub | EmilienM, undef is not cool when it makes the parameter required if the value is not set, BTW the previous value is not undef, it's 'v2.0' | 00:35 |
| EmilienM | yeah but when you deprecate, undef is fine I guess | 00:35 |
| EmilienM | gildub: reading https://review.openstack.org/#/c/214984/4/spec/acceptance/basic_keystone_spec.rb,cm | 00:35 |
| EmilienM | gildub: git blame what? | 00:35 |
| gildub | EmilienM, in this case I wanted clear there is no value set but we can put $version = 'EmilienM said undef is better but up to you' and it will work ;) | 00:36 |
| EmilienM | yeah no value is fine | 00:37 |
| EmilienM | but we need a WARNING | 00:37 |
| EmilienM | operators NEED to see that kind of change | 00:37 |
| gildub | EmilienM, ack | 00:37 |
| EmilienM | it will really break their catalog otherwise | 00:37 |
| EmilienM | so now, ordering | 00:37 |
| EmilienM | https://review.openstack.org/#/c/214984/4/spec/acceptance/basic_keystone_spec.rb,cm | 00:37 |
| EmilienM | definitely not | 00:37 |
| EmilienM | "Well I agree that on the long run use autorequire is more robust, but in the meantime that fixes the issue." | 00:37 |
| EmilienM | seriously? ^ | 00:38 |
| gildub | EmilienM, well my issue is acceptance test break from time to time because the ordering, that would explain why jenkins is [RED] inconsistently, I think | 00:38 |
| EmilienM | from time to time? | 00:38 |
| EmilienM | wow, so we have a bug | 00:38 |
| EmilienM | let me check the code | 00:38 |
| gildub | EmilienM, I think | 00:38 |
| EmilienM | so | 00:39 |
| EmilienM | 1/ domain needs keystone started https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/type/keystone_domain.rb#L49-L51 | 00:39 |
| gildub | EmilienM, oh, think it's my code | 00:39 |
| EmilienM | 2/ tenant needs default domain (so domains) and keystone started https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/type/keystone_tenant.rb#L56-L58 | 00:39 |
| gildub | EmilienM, you're right it's the autorequire missing (I left it undone) in my patch | 00:40 |
| EmilienM | 3/ user needs tenant, domain https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/type/keystone_user.rb#L92-L105 | 00:40 |
| EmilienM | ... | 00:40 |
| gildub | EmilienM, forget about this one then (abandon) | 00:40 |
| EmilienM | you can abandon the patch | 00:40 |
| EmilienM | ++ | 00:40 |
| gildub | EmilienM, thanks | 00:40 |
| EmilienM | gildub: anything else before I go diner? | 00:40 |
| gildub | EmilienM, apperitif? | 00:40 |
| EmilienM | lol | 00:40 |
| EmilienM | I wish | 00:41 |
| gildub | EmilienM, no I'm good, thanks again for your time | 00:41 |
| EmilienM | cool np | 00:41 |
| *** ducttape_ has joined #puppet-openstack | 00:46 | |
| *** tiswanso has joined #puppet-openstack | 00:52 | |
| *** tiswanso has quit IRC | 00:53 | |
| *** tiswanso has joined #puppet-openstack | 00:53 | |
| *** ducttape_ has quit IRC | 00:54 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: if running eventlet, send deprecation warning https://review.openstack.org/215390 | 00:55 |
| *** tiswanso_ has joined #puppet-openstack | 00:58 | |
| *** tiswanso has quit IRC | 00:59 | |
| *** xingchao has joined #puppet-openstack | 01:00 | |
| *** richm has quit IRC | 01:01 | |
| *** dgurtner has quit IRC | 01:06 | |
| *** ducttape_ has joined #puppet-openstack | 01:08 | |
| *** sergmelikyan has joined #puppet-openstack | 01:30 | |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_group for Keystone V3 API https://review.openstack.org/202409 | 01:36 |
| *** xingchao_ has joined #puppet-openstack | 01:47 | |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_group for Keystone V3 API https://review.openstack.org/202409 | 01:49 |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Keystone Role/Group resource provider https://review.openstack.org/211444 | 01:49 |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_group_role for Keystone V3 API https://review.openstack.org/207755 | 01:49 |
| *** xingchao has quit IRC | 01:50 | |
| *** xingchao_ has quit IRC | 01:59 | |
| *** v1k0d3n has quit IRC | 01:59 | |
| *** ducttape_ has quit IRC | 02:04 | |
| *** xingchao has joined #puppet-openstack | 02:04 | |
| *** xarses has quit IRC | 02:07 | |
| *** sergmelikyan has quit IRC | 02:08 | |
| openstackgerrit | zhangfeng proposed openstack/puppet-sahara: Use .fixtures.yaml instead of .fixtures.yml https://review.openstack.org/215405 | 02:22 |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Authentication URLs and endpoint clarity re-factor https://review.openstack.org/213603 | 02:24 |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-designate: Use generic_service to manage services https://review.openstack.org/215000 | 02:26 |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-designate: Add enabled_notification_handlers option https://review.openstack.org/214465 | 02:26 |
| gildub | xingchao, michchap_, please review https://review.openstack.org/213938/ backport | 02:33 |
| *** xarses has joined #puppet-openstack | 02:36 | |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-designate: Add designate::mdns class https://review.openstack.org/214976 | 02:40 |
| xingchao | gildub: ok, reviewing now | 02:40 |
| openstackgerrit | Merged openstack/puppet-keystone: Purely cosmetic change on keystone provider https://review.openstack.org/213938 | 03:03 |
| openstackgerrit | Merged openstack/puppet-heat: Create Heat Domain with Keystone_domain resource https://review.openstack.org/204541 | 03:03 |
| *** ducttape_ has joined #puppet-openstack | 03:14 | |
| *** xingchao_ has joined #puppet-openstack | 03:18 | |
| *** xingchao has quit IRC | 03:18 | |
| *** ducttape_ has quit IRC | 03:20 | |
| *** v1k0d3n has joined #puppet-openstack | 03:23 | |
| *** ducttape_ has joined #puppet-openstack | 03:27 | |
| *** ducttape_ has quit IRC | 03:45 | |
| *** tiswanso_ has quit IRC | 03:57 | |
| *** fedexo has joined #puppet-openstack | 04:11 | |
| *** fedexo has quit IRC | 04:35 | |
| gildub | xingchao_, thanks | 04:44 |
| gildub | xingchao_, michchap_, I have this one, again, per minors changes from comments suggestions: https://review.openstack.org/213603 | 04:53 |
| *** cgfbee has joined #puppet-openstack | 05:00 | |
| *** topshare has joined #puppet-openstack | 05:31 | |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Endpoints are APIs version-less https://review.openstack.org/213957 | 05:37 |
| *** topshare has quit IRC | 05:40 | |
| *** Zhang has joined #puppet-openstack | 05:41 | |
| *** xingchao_ has quit IRC | 05:56 | |
| *** xingchao has joined #puppet-openstack | 05:59 | |
| gildub | xingchao, could you please re-review https://review.openstack.org/213603 ? That would be the last one for today! | 06:36 |
| xingchao | gildub: I'm a little busy now, I will review it ASAP. | 06:40 |
| gildub | xingchao, no worries, whenever you can is already great :) | 06:41 |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Endpoints are APIs version-less https://review.openstack.org/213957 | 06:42 |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_trust for Keystone V3 API https://review.openstack.org/200996 | 06:43 |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-trove: Introduce use_guestagent_template option https://review.openstack.org/215476 | 06:45 |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-manila: Add rabbit_ha_queues option https://review.openstack.org/215201 | 06:55 |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-cinder: Add rabbit_ha_queues option https://review.openstack.org/215482 | 07:02 |
| *** jtomasek has joined #puppet-openstack | 07:04 | |
| *** rochaporto has joined #puppet-openstack | 07:05 | |
| *** jpena has joined #puppet-openstack | 07:20 | |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-heat: Add rabbit_ha_queues option https://review.openstack.org/215496 | 07:41 |
| openstackgerrit | Benedikt von St. Vieth proposed openstack/puppet-nova: Allow to disable openstackocci installation via pip https://review.openstack.org/194977 | 07:46 |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-trove: Introduce use_guestagent_template option https://review.openstack.org/215476 | 07:46 |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-glance: Add rabbit_ha_queues option https://review.openstack.org/215500 | 07:50 |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_group for Keystone V3 API https://review.openstack.org/202409 | 07:55 |
| *** vinsh has quit IRC | 08:03 | |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_group for Keystone V3 API https://review.openstack.org/202409 | 08:05 |
| *** derekh has joined #puppet-openstack | 08:08 | |
| *** stamak has joined #puppet-openstack | 08:10 | |
| *** xingchao has quit IRC | 08:23 | |
| *** gildub has quit IRC | 08:23 | |
| *** dgurtner has joined #puppet-openstack | 08:26 | |
| *** dgurtner has quit IRC | 08:26 | |
| *** dgurtner has joined #puppet-openstack | 08:26 | |
| *** Zhang has quit IRC | 08:27 | |
| *** jistr has joined #puppet-openstack | 08:30 | |
| *** v1k0d3n has quit IRC | 08:31 | |
| *** c0m0 has joined #puppet-openstack | 08:52 | |
| openstackgerrit | Benedikt von St. Vieth proposed openstack/puppet-nova: Allow to disable openstackocci installation via pip https://review.openstack.org/194977 | 08:53 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/puppet-nova: Allow to disable openstackocci installation via pip https://review.openstack.org/194977 | 08:56 |
| openstackgerrit | Michael Polenchuk proposed openstack/puppet-nova: Handle libvirt/cpu_model option https://review.openstack.org/214542 | 09:01 |
| degorenko | spredzy, hello! o/ Are you here? | 09:02 |
| spredzy | degorenko, Hey, yes I am here :) | 09:03 |
| degorenko | spredzy, i want ask you about your comments here: https://review.openstack.org/#/c/198744/ | 09:03 |
| degorenko | it seems you right | 09:03 |
| *** vsaienko has quit IRC | 09:03 | |
| degorenko | but include doesn't provide possibility to override default options | 09:04 |
| degorenko | how it will work when we want to override default values? | 09:04 |
| spredzy | degorenko, correct. Most people uses hiera to deploy those modules. So one will be available to deploy by overriding sahara::rpc::rpc_backend in a hiera file | 09:05 |
| spredzy | yet, from the ENC perspective only the sahara class will need to be declared | 09:05 |
| spredzy | degorenko, does that make sense ? | 09:06 |
| degorenko | spredzy, well, looks good, but what about simple deploy without hiera? | 09:07 |
| spredzy | erf ... that a recurrent issue... I'll say people would have to deal with it on their composition layer ... but that might not bring me friends | 09:08 |
| degorenko | spredzy, all another modules have rpc options in main class :) | 09:08 |
| spredzy | Yes, and all modules have a main class of hundreads of lines | 09:09 |
| spredzy | the winner I think being keystone with 996 | 09:09 |
| degorenko | spredzy, yees, it's hard to understand, i agree | 09:09 |
| spredzy | my point here is that init classes are becoming bigger and bigger, when al they do can be separated in small logical class that can be included | 09:10 |
| spredzy | ending up with only includes (or mainly) | 09:10 |
| degorenko | spredzy, totally agree with you | 09:10 |
| spredzy | as I said in the comment its not a blocker -1, just a would like to see thing | 09:10 |
| degorenko | but the issue with overriding parameters... | 09:10 |
| spredzy | Is fuel relying on hiera ? | 09:11 |
| degorenko | yes :) | 09:11 |
| degorenko | but there is we have some layer which will call all openstack modules | 09:11 |
| degorenko | with proper values | 09:11 |
| spredzy | degorenko, The only solution I have found to remain compatible with non-hiera installer is to use pick https://review.openstack.org/#/c/208394/5/manifests/db.pp,cm | 09:12 |
| spredzy | degorenko, this is an example ^ | 09:12 |
| degorenko | spredzy, looking | 09:12 |
| spredzy | but I don't really like it, yet I can't find a smarter way | 09:12 |
| spredzy | That will end up in leaving the parameter in manifest/init.pp include sahara::rpc and create a pick call for each parameter declared in manifest/init.pp | 09:13 |
| degorenko | spredzy, yep, i know this way | 09:13 |
| degorenko | dirty way :) | 09:13 |
| spredzy | degorenko, agree, but I can't find another solution if we want to remain non-hiera way compatible | 09:14 |
| spredzy | if it was up to me it will be hiera only :) | 09:14 |
| degorenko | spredzy, anyway, we need hundred of lines with parameters in init.pp | 09:14 |
| spredzy | hence the full hiera way :) | 09:15 |
| degorenko | for this way) | 09:15 |
| spredzy | :D | 09:15 |
| degorenko | :D hard way | 09:15 |
| spredzy | Na I'd call it the easy way :p | 09:15 |
| spredzy | But to be back on the review, post any of your thought, see what people have to say abut it | 09:16 |
| degorenko | spredzy, can you provide some example with hiera? How it will override default parameters? | 09:16 |
| degorenko | spredzy, oh, most people see -1 and think: oh, let's wait for update | 09:16 |
| spredzy | sahara::notify::enable_notifications: true | 09:16 |
| degorenko | in hiera right? | 09:17 |
| spredzy | yes | 09:17 |
| degorenko | oh, of course | 09:17 |
| spredzy | this will override the default value for enable_notifications | 09:17 |
| degorenko | yeah | 09:17 |
| degorenko | sorry for stupid question... | 09:17 |
| spredzy | ahah no stupid question :) For the -1 behavior ... hmmm that's wrong but nothing I can do for that | 09:18 |
| spredzy | Let's chat back if you prefer this afternoon when US people will be awake | 09:18 |
| degorenko | yep, we can wait for EmilienM opinion | 09:18 |
| spredzy | This way we can have more feebacks | 09:18 |
| degorenko | bth, spredzy where are you from? :) | 09:19 |
| degorenko | btw* | 09:19 |
| spredzy | Bordeaux, France | 09:19 |
| spredzy | you ? | 09:19 |
| degorenko | Saratov, Russia :) | 09:19 |
| degorenko | Moskow time zone | 09:19 |
| spredzy | ok +1:00 compared to me | 09:20 |
| degorenko | yes :) | 09:20 |
| *** vsaienko has joined #puppet-openstack | 09:20 | |
| degorenko | so, let's wait then for US people | 09:21 |
| spredzy | degorenko, feel free to raise the case of non-hiera way in a comment, so people can have both side of the story | 09:21 |
| degorenko | spredzy, ok :) | 09:21 |
| myatsenko | michchap: hello, michchap ! I tried to respond to your comments in CR: https://review.openstack.org/#/c/209412 | 09:26 |
| myatsenko | Did you have a chance/time to look at it? maybe I should collet/provide more info for one of the questions? | 09:26 |
| openstackgerrit | Javier Peña proposed openstack/puppet-swift: Add support for swift-object-expirer service https://review.openstack.org/211549 | 09:28 |
| openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-nova: Fixed issue with rabbit_hosts parameter https://review.openstack.org/197013 | 09:34 |
| openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-nova: Fixed issue with rabbit_hosts parameter https://review.openstack.org/197013 | 09:35 |
| degorenko | spredzy, hey, can you help with chery-pick for this one https://review.openstack.org/#/c/205988/ | 09:43 |
| degorenko | i have error: error: Commit 8c4357fde242cc5a628806009fa582858b148443 is a merge but no -m option was given. | 09:43 |
| degorenko | fatal: cherry-pick failed | 09:43 |
| *** rcarrillocruz has joined #puppet-openstack | 09:44 | |
| rcarrillocruz | so, crinkle, i'm going to push a change on ironic::bifrost | 09:44 |
| rcarrillocruz | instead of deploying baremetal.json an groupvars/all on the bifrost git repo | 09:45 |
| rcarrillocruz | i'll make the manifest to create /etc/bifrost (and parameterize it, /etc/bifrost just default) and put those config files in there | 09:45 |
| rcarrillocruz | that way we don't have config files in the bifrost cloned repo, leaving it dirty and causing issues if it's updated | 09:46 |
| rcarrillocruz | will push in a bit | 09:46 |
| rcarrillocruz | that /etc/bifrost/ vars file can later be passed to ansible with -e "@/etc/bifrost/vars' | 09:47 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Update parameters for Sahara https://review.openstack.org/205988 | 09:53 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-sahara: Update parameters for Sahara https://review.openstack.org/205988 | 09:56 |
| spredzy | degorenko, Oops I think I hijacked your review :/ | 09:57 |
| spredzy | That wasn't intended ... I just wanted to try on my side to let you know how to fix it :/ | 09:57 |
| degorenko | spredzy, thanks for try :D but i'm already find out what was wrong... I tried to cherry-pick git merge commit :D | 09:58 |
| spredzy | erf ... indeed :p | 09:59 |
| spredzy | well please go ahead and resend a review | 09:59 |
| degorenko | i return back my 4 patch set :) | 09:59 |
| spredzy | Mine didnt't take care well of the conflict just wanted to have something working | 09:59 |
| spredzy | yes please send it back | 09:59 |
| degorenko | because you forgot remove sync_db from master branchj :) | 09:59 |
| degorenko | yes :) | 09:59 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Update parameters for Sahara https://review.openstack.org/205988 | 10:01 |
| spredzy | yeah ... I really didn't do it cleanly - just keeping not HEAD diff - trying to see if something was broken in the process. | 10:02 |
| *** xingchao has joined #puppet-openstack | 10:02 | |
| degorenko | spredzy, no problem :) my bad | 10:02 |
| spredzy | degorenko, looks good now :) will review later | 10:02 |
| degorenko | spredzy, thanks) | 10:02 |
| *** paramite has joined #puppet-openstack | 10:26 | |
| *** v1k0d3n has joined #puppet-openstack | 10:32 | |
| *** paramite has quit IRC | 10:37 | |
| *** v1k0d3n has quit IRC | 10:37 | |
| *** chandankumar has joined #puppet-openstack | 10:38 | |
| openstackgerrit | Ricardo Carrillo Cruz proposed openstack/puppet-ironic: Put configuration files under configurable folder https://review.openstack.org/215575 | 10:48 |
| rcarrillocruz | crinkle: ^, this should address your concerns you had at some point | 10:49 |
| *** paramite has joined #puppet-openstack | 10:52 | |
| *** paramite has quit IRC | 11:04 | |
| rcarrillocruz | s/your/the | 11:10 |
| *** xingchao has quit IRC | 11:14 | |
| openstackgerrit | Jiri Stransky proposed openstack/puppet-neutron: Configure Big Switch ML2 plugin https://review.openstack.org/214686 | 11:14 |
| openstackgerrit | Merged openstack/puppet-horizon: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/189869 | 11:16 |
| *** paramite has joined #puppet-openstack | 11:18 | |
| openstackgerrit | Merged openstack/puppet-sahara: Initial msync run for all Puppet OpenStack modules https://review.openstack.org/189877 | 11:20 |
| *** dprince has joined #puppet-openstack | 11:35 | |
| EmilienM | good morning | 11:44 |
| *** blentz has quit IRC | 11:44 | |
| Madkiss | hello EmilienM | 11:44 |
| *** blentz has joined #puppet-openstack | 11:45 | |
| iurygregory | good morning =) | 11:46 |
| Madkiss | Error 400 on SERVER: Invalid parameter domain on Keystone_tenant[services] at /opt/puppet-modules/keystone/manifests/roles/admin.pp:106 -- what is this trying to tell me? | 11:48 |
| *** paramite is now known as paramite|afk | 11:49 | |
| *** jpena is now known as jpena|lunch | 11:56 | |
| *** dgurtner has quit IRC | 11:57 | |
| *** paramite|afk is now known as paramite | 12:00 | |
| Madkiss | EmilienM: hello? | 12:02 |
| Madkiss | What do I need to do to work around this | 12:03 |
| openstackgerrit | Lukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes https://review.openstack.org/200877 | 12:04 |
| *** pradk has quit IRC | 12:04 | |
| EmilienM | Madkiss: hello | 12:06 |
| EmilienM | Madkiss: if you need our help, please provide your manifests, logs and 'puppet module list' result | 12:07 |
| Madkiss | what? | 12:08 |
| Madkiss | i'm updating frm juno to kilo. The juno setup was working fine. I checked out all modules to 6.0.0, and now, when I try to run Puppet, all I get is the error message mentioned above. | 12:09 |
| *** coffee-- has joined #puppet-openstack | 12:09 | |
| EmilienM | https://github.com/openstack/puppet-keystone/blob/stable/kilo/manifests/roles/admin.pp#L111 | 12:10 |
| EmilienM | your provider should have been updated if you're running kilo | 12:10 |
| EmilienM | maybe provider is in cache on agent nodes | 12:11 |
| EmilienM | are you running puppet master? | 12:11 |
| Madkiss | yes. | 12:11 |
| EmilienM | oh that's why probably | 12:11 |
| Madkiss | And I have that version of the provider | 12:11 |
| Madkiss | EmilienM: restarting | 12:11 |
| Madkiss | EmilienM: that's way better. thanks man! | 12:13 |
| EmilienM | Madkiss: cool | 12:13 |
| Madkiss | EmilienM: how much beer do I owe you by now? :) | 12:13 |
| EmilienM | Madkiss: I stopped counting | 12:13 |
| EmilienM | :) | 12:13 |
| Madkiss | might as well buy you a brewery ... | 12:13 |
| *** coffee-- has quit IRC | 12:15 | |
| *** prad_ has joined #puppet-openstack | 12:18 | |
| iurygregory | hey EmilienM, little question about rspec tests | 12:20 |
| *** ducttape_ has joined #puppet-openstack | 12:20 | |
| EmilienM | spredzy: I gave a bit more context on https://review.openstack.org/#/c/209997/9/manifests/agents/ml2/mlnx.pp,cm | 12:21 |
| spredzy | EmilienM, ack thanks. | 12:22 |
| spredzy | Make sense | 12:22 |
| *** dgurtner has joined #puppet-openstack | 12:22 | |
| *** sergmelikyan has joined #puppet-openstack | 12:25 | |
| *** vinsh has joined #puppet-openstack | 12:26 | |
| *** linkedinyou has quit IRC | 12:28 | |
| Madkiss | yikes | 12:29 |
| Madkiss | Error: /Stage[main]/Neutron::Keystone::Auth/Keystone::Resource::Service_identity[neutron]/Keystone_user[neutron]: Could not evaluate: Execution of '/usr/bin/openstack token issue --format value' returned 1: ERROR: openstack The resource could not be found. (HTTP 404) (Request-ID: req-3650a428-0afd-4f5d-8f81-376bc9b867cb) | 12:29 |
| Madkiss | this seems to be related to /root/openrc or something? | 12:29 |
| *** sergmelikyan has quit IRC | 12:29 | |
| *** linkedinyou has joined #puppet-openstack | 12:32 | |
| *** ducttape_ has quit IRC | 12:34 | |
| *** v1k0d3n has joined #puppet-openstack | 12:35 | |
| *** stamak has quit IRC | 12:36 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: CI test - do not merge https://review.openstack.org/215603 | 12:37 |
| *** jayg|g0n3 is now known as jayg | 12:38 | |
| *** beddari has quit IRC | 12:39 | |
| *** beddari has joined #puppet-openstack | 12:39 | |
| openstackgerrit | Merged openstack/puppet-modulesync-configs: spec: Fix nodesets errors (x64 for el and amd64 for debian) https://review.openstack.org/209127 | 12:42 |
| openstackgerrit | Merged openstack/puppet-modulesync-configs: sample config (update namespace) https://review.openstack.org/209128 | 12:42 |
| EmilienM | jistr: last comment on https://review.openstack.org/#/c/214686 | 12:47 |
| * jistr looking | 12:47 | |
| jistr | EmilienM: ack i'm on it | 12:48 |
| *** tiswanso has joined #puppet-openstack | 12:51 | |
| *** prad_ is now known as pradk | 12:53 | |
| *** tiswanso has quit IRC | 12:55 | |
| jistr | btw every time i run 'rake spec', it fetches test fixtures from the internet. Is there a way to make it use "cached" ones? | 12:55 |
| *** tiswanso has joined #puppet-openstack | 12:55 | |
| degorenko | EmilienM, hello. Take a look on https://review.openstack.org/198744 please. Me and spredzy have some discussion, so we need suggestions/thoughts from another people | 12:56 |
| *** tiswanso has quit IRC | 12:57 | |
| *** tiswanso has joined #puppet-openstack | 12:57 | |
| EmilienM | degorenko: ok, looking todayy | 12:57 |
| openstackgerrit | Jiri Stransky proposed openstack/puppet-neutron: Configure Big Switch ML2 plugin https://review.openstack.org/214686 | 12:59 |
| jistr | EmilienM: ^ updated. thanks for the review :) | 12:59 |
| EmilienM | degorenko: yanis's comments are good, i'll let him reply | 13:00 |
| *** sergmelikyan has joined #puppet-openstack | 13:01 | |
| EmilienM | degorenko: your patches are good otherwise thx for your work on designate | 13:01 |
| *** serg_melikyan has joined #puppet-openstack | 13:03 | |
| *** sergmelikyan has quit IRC | 13:03 | |
| *** jpena|lunch is now known as jpena | 13:09 | |
| *** tiswanso has quit IRC | 13:12 | |
| *** vinsh has quit IRC | 13:12 | |
| spredzy | EmilienM, re https://review.openstack.org/198744. The main question here (I think) is are we supposed to manager non hiera deployment also ? | 13:17 |
| EmilienM | OF COURSE !! | 13:17 |
| EmilienM | a lot of people are still not using Hiera | 13:18 |
| *** richm has joined #puppet-openstack | 13:18 | |
| spredzy | degorenko, ^ | 13:18 |
| spredzy | Then let me remove my -1 | 13:19 |
| openstackgerrit | Merged openstack/puppet-trove: Fix default value of guestagent_config_file option https://review.openstack.org/208431 | 13:19 |
| *** iurygregory has left #puppet-openstack | 13:21 | |
| *** iurygregory has joined #puppet-openstack | 13:21 | |
| openstackgerrit | Javier Peña proposed openstack/puppet-swift: Add support for swift-object-expirer service https://review.openstack.org/211549 | 13:22 |
| Madkiss | Execution of '/usr/bin/openstack endpoint create --format shell keystone --region RegionOne --os-token foobar --os-url http://foobar.com/v2.0/' returned 2: usage: openstack endpoint create [-h] [-f {html,json,shell,table,value,yaml}] | 13:23 |
| Madkiss | openstack endpoint create: error: argument --publicurl is required | 13:23 |
| Madkiss | what's wrong here? | 13:25 |
| degorenko | spredzy, that was i'm talking about :) | 13:32 |
| spredzy | degorenko, yeah I know. I just think we need to find a solution against those several hundreds line init.pp | 13:32 |
| spredzy | :) | 13:33 |
| degorenko | EmilienM, spredzy also please https://review.openstack.org/205988 it's cherri-pick from master :) | 13:33 |
| degorenko | spredzy, i agree | 13:33 |
| degorenko | as workaround, we can just create separate modules | 13:33 |
| degorenko | for rpc | 13:33 |
| *** vinsh has joined #puppet-openstack | 13:34 | |
| *** tiswanso has joined #puppet-openstack | 13:35 | |
| *** vinsh has quit IRC | 13:35 | |
| *** vinsh_ has joined #puppet-openstack | 13:35 | |
| *** ducttape_ has joined #puppet-openstack | 13:36 | |
| spredzy | degorenko, well the problem remain in the sense one will then need to add it in their ENC | 13:37 |
| degorenko | spredzy, what is ENC? :D | 13:37 |
| spredzy | degorenko, External Network Classifier https://docs.puppetlabs.com/guides/external_nodes.html | 13:38 |
| openstackgerrit | Merged openstack/puppet-designate: Add enabled_notification_handlers option https://review.openstack.org/214465 | 13:38 |
| spredzy | the code that tells puppet which node should be provisioned with which class | 13:38 |
| degorenko | spredzy, oh, thanks! I've tried to google it, but so many possible and crazy results :D | 13:38 |
| *** dgurtner has quit IRC | 13:40 | |
| *** dgurtner has joined #puppet-openstack | 13:45 | |
| *** dgurtner has quit IRC | 13:45 | |
| *** dgurtner has joined #puppet-openstack | 13:45 | |
| openstackgerrit | Vasyl Saienko proposed openstack/puppet-horizon: remove hardcoded 'hypervisor_options' list https://review.openstack.org/215639 | 13:46 |
| spredzy | crinkle, mfisch clayton michchap_ if I can get feedbacks on this series please https://review.openstack.org/#/q/status:open+branch:master+topic:inifile_proxy_provider,n,z | 13:46 |
| openstackgerrit | Michael Polenchuk proposed openstack/puppet-nova: Handle libvirt/cpu_model option https://review.openstack.org/214542 | 14:00 |
| openstackgerrit | Lukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes https://review.openstack.org/200877 | 14:04 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: ci test - do not merge https://review.openstack.org/215648 | 14:05 |
| openstackgerrit | Vasyl Saienko proposed openstack/puppet-horizon: remove hardcoded 'hypervisor_options' list https://review.openstack.org/215639 | 14:08 |
| openstackgerrit | Vasyl Saienko proposed openstack/puppet-horizon: remove hardcoded 'cinder_options' list https://review.openstack.org/215651 | 14:08 |
| *** dfisher has joined #puppet-openstack | 14:11 | |
| openstackgerrit | Vasyl Saienko proposed openstack/puppet-horizon: remove hardcoded 'hypervisor_options' list https://review.openstack.org/215639 | 14:15 |
| openstackgerrit | Vasyl Saienko proposed openstack/puppet-horizon: remove hardcoded 'cinder_options' list https://review.openstack.org/215651 | 14:15 |
| openstackgerrit | Merged openstack/puppet-keystone: Validate service_identity resources https://review.openstack.org/190016 | 14:18 |
| openstackgerrit | Vasyl Saienko proposed openstack/puppet-horizon: remove hardcoded 'hypervisor_options' list https://review.openstack.org/215639 | 14:21 |
| openstackgerrit | Vasyl Saienko proposed openstack/puppet-horizon: remove hardcoded 'cinder_options' list https://review.openstack.org/215651 | 14:21 |
| *** pradk has quit IRC | 14:21 | |
| *** prad_ has joined #puppet-openstack | 14:22 | |
| *** prad_ is now known as pradk | 14:23 | |
| ukitanuki | Could anyone give me a hint as to what should be in my ENC when following the puppet-ceph/example? | 14:28 |
| *** pradk has quit IRC | 14:31 | |
| ukitanuki | I've attempted using ceph::profile::mon and ceph::profile::osd but get complaints about "Class[Ceph::Keys] is already declared in .. profile/client.pp ... profile/mon.pp" | 14:32 |
| *** aimon_ has quit IRC | 14:45 | |
| *** mjblack has quit IRC | 14:51 | |
| *** paramite is now known as paramite|afk | 14:51 | |
| *** aimon has joined #puppet-openstack | 14:51 | |
| *** TC01 has joined #puppet-openstack | 14:52 | |
| *** stamak has joined #puppet-openstack | 14:53 | |
| *** serg_melikyan has quit IRC | 15:00 | |
| *** skolekonov has quit IRC | 15:02 | |
| *** dgurtner has quit IRC | 15:03 | |
| *** paramite|afk is now known as paramite | 15:10 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: acceptance: install latest packages https://review.openstack.org/215681 | 15:12 |
| openstackgerrit | Vasyl Saienko proposed openstack/puppet-horizon: remove hardcoded 'neutron_options' list https://review.openstack.org/215682 | 15:12 |
| *** xingchao has joined #puppet-openstack | 15:14 | |
| *** sergmelikyan has joined #puppet-openstack | 15:18 | |
| *** imcsk8_ is now known as imcsk8 | 15:27 | |
| *** rochaporto has quit IRC | 15:32 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: acceptance: install latest packages https://review.openstack.org/215681 | 15:37 |
| *** paramite is now known as paramite|afk | 15:46 | |
| *** paramite|afk is now known as paramite | 15:48 | |
| dfisher | quick question. in neutron/manifests/init.pp, root_helper defaults to "sudo neutron-rootwrap /etc/neutron/rootwrap.conf". If I want to "unset" that value and just have | 15:54 |
| dfisher | root_helper = | 15:54 |
| dfisher | in neutron.conf, do I pass undef or just an empty string? | 15:54 |
| *** samuelBartel has quit IRC | 15:56 | |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-designate: Use generic_service to manage services https://review.openstack.org/215000 | 15:59 |
| *** jpena has quit IRC | 15:59 | |
| *** sergmelikyan has quit IRC | 16:03 | |
| *** sergmelikyan has joined #puppet-openstack | 16:05 | |
| *** jistr has quit IRC | 16:05 | |
| xingchao | dfisher: an empty str | 16:05 |
| dfisher | thank you! | 16:10 |
| *** chandankumar has quit IRC | 16:17 | |
| *** ducttape_ has quit IRC | 16:21 | |
| *** aimon has quit IRC | 16:28 | |
| EmilienM | richm: xingchao merged https://review.openstack.org/#/c/213603/ | 16:29 |
| EmilienM | richm: would you wanted to wait? | 16:29 |
| *** degorenko has quit IRC | 16:30 | |
| openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 16:30 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: acceptance: install latest packages https://review.openstack.org/215681 | 16:31 |
| EmilienM | richm: https://bugs.launchpad.net/puppet-keystone/+bug/1485508 is very interesting | 16:32 |
| openstack | Launchpad bug 1485508 in puppet-keystone "when domain_specific_drivers_enabled=True keystone_user provider fails " [Undecided,In progress] - Assigned to Vasyl Saienko (vsaienko) | 16:32 |
| *** xingchao has quit IRC | 16:37 | |
| openstackgerrit | Merged openstack/puppet-keystone: Authentication URLs and endpoint clarity re-factor https://review.openstack.org/213603 | 16:37 |
| *** c0m0 has quit IRC | 16:38 | |
| *** ducttape_ has joined #puppet-openstack | 16:40 | |
| *** AlexeyElagin has quit IRC | 16:41 | |
| *** derekh has quit IRC | 16:43 | |
| *** Zhang has joined #puppet-openstack | 16:44 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Remove httpchk option from haproxy listeners https://review.openstack.org/197904 | 16:48 |
| *** aimon has joined #puppet-openstack | 16:54 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: acceptance: install latest packages https://review.openstack.org/215681 | 17:02 |
| openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 17:06 |
| *** xingchao has joined #puppet-openstack | 17:06 | |
| *** xingchao has quit IRC | 17:06 | |
| richm | EmilienM: yeah, I saw that early in my v3 work - openstack user commands required the --domain, but other commands e.g. project did not | 17:08 |
| richm | EmilienM: Then, at some point after switching to kilo, openstack user commands did not require --domain | 17:08 |
| richm | EmilienM: so either this issue is with osc/keystone versions, or some sort of regression? | 17:09 |
| EmilienM | richm: prob osc version | 17:09 |
| EmilienM | but not sue | 17:09 |
| EmilienM | sure* | 17:09 |
| *** Zhang has quit IRC | 17:11 | |
| *** Zhang has joined #puppet-openstack | 17:11 | |
| *** Zhang has quit IRC | 17:12 | |
| richm | EmilienM: I think you are on the right track with https://bugs.launchpad.net/puppet-keystone/+bug/1485508/comments/3 | 17:13 |
| openstack | Launchpad bug 1485508 in puppet-keystone "when domain_specific_drivers_enabled=True keystone_user provider fails " [Undecided,In progress] - Assigned to Vasyl Saienko (vsaienko) | 17:13 |
| EmilienM | richm: yeah, to me it's two different things | 17:14 |
| richm | EmilienM: At any rate, I think the proposed fix is good because then it will work regardless of any setting - it's more network traffic, but the common case is to have few domains | 17:14 |
| *** dh64 has joined #puppet-openstack | 17:20 | |
| openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 17:21 |
| openstackgerrit | Ramkumar Gowrishankar proposed openstack/puppet-neutron: Support for Nuage Neutron plugin and Nuage VRS in puppet-neutron https://review.openstack.org/214798 | 17:26 |
| *** prad_ has joined #puppet-openstack | 17:31 | |
| rgowrishankar | EmilienM: I made changes and uploaded the new files and I also replied to the comments. My replies are right below your review comments in patch 3 | 17:45 |
| EmilienM | rgowrishankar: cool | 17:45 |
| rgowrishankar | EmilienM: I missed the line space fix, I will upload that after you review the current patch. I added comments about why we are setting the controller IP in agents/nuage.pp. If you want to discuss that more let me know.. | 17:47 |
| EmilienM | rgowrishankar: I will | 17:47 |
| *** prad_ is now known as pradk | 17:55 | |
| *** paramite has quit IRC | 17:57 | |
| *** cdelatte has quit IRC | 17:58 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Initial run_tests.sh script https://review.openstack.org/210412 | 18:00 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: CI - do not merge - testing https://review.openstack.org/#/c/210412/ https://review.openstack.org/214793 | 18:01 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Create puppet folder layout https://review.openstack.org/201298 | 18:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy MySQL and Keystone (WSGI) https://review.openstack.org/207070 | 18:03 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: configure and run tempest https://review.openstack.org/207078 | 18:03 |
| EmilienM | is any core around today to review integration testing? | 18:04 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: acceptance: install latest packages https://review.openstack.org/215681 | 18:05 |
| openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 18:08 |
| *** xarses has quit IRC | 18:20 | |
| *** xarses has joined #puppet-openstack | 18:20 | |
| *** xarses has quit IRC | 18:26 | |
| openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 18:41 |
| *** xarses has joined #puppet-openstack | 18:47 | |
| iurygregory | EmilienM, i need to run 'keystone-manage saml_idp_metadata > $idp_metadata_path', can i use Exec or there is a better way? | 18:52 |
| EmilienM | iurygregory: Exec is fine I think | 18:53 |
| iurygregory | ok ^^ | 18:53 |
| EmilienM | iurygregory: the exec has to be idempotent | 18:54 |
| iurygregory | you mean if i run twice i got no erros right? | 18:56 |
| EmilienM | iurygregory: no change, no errors | 18:56 |
| iurygregory | ok | 18:56 |
| *** ducttape_ has quit IRC | 18:58 | |
| *** jtomasek has quit IRC | 19:04 | |
| *** xarses_ has joined #puppet-openstack | 19:15 | |
| openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 19:16 |
| *** ducttape_ has joined #puppet-openstack | 19:18 | |
| *** richm has quit IRC | 19:21 | |
| *** dprince has quit IRC | 19:27 | |
| *** ducttape_ has quit IRC | 19:27 | |
| *** ducttape_ has joined #puppet-openstack | 19:27 | |
| openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 19:29 |
| *** ducttape_ has quit IRC | 19:34 | |
| crinkle | spredzy: re topic:inifile_proxy_provider I'm concerned that it's failing on manila and glance, any idea why? would be nice to know it works for everything before merging it in half the modules | 19:35 |
| spredzy | crinkle, for manila this is a packaging issue. Even this https://review.openstack.org/#/c/211553/ is failing on manila | 19:36 |
| spredzy | Simple renaming stackforge -> openstack | 19:37 |
| spredzy | As for glance you can see acceptance tests passes in CentOS (https://review.openstack.org/#/c/211937/), but fails on ubuntu meaning it is unrelated to the patch itself. | 19:38 |
| spredzy | Reason is : http://logs.openstack.org/37/211937/3/check/gate-puppet-glance-puppet-beaker-rspec-dsvm-trusty/b831ac8/console.html#_2015-08-19_10_52_04_607 | 19:38 |
| crinkle | I don't think that necessarily means its unrelated | 19:39 |
| spredzy | crinkle, I say unrelated because I have the exact same failure on another patchset https://review.openstack.org/#/c/211937/ | 19:40 |
| spredzy | the above still uses the old provider | 19:41 |
| *** ducttape_ has joined #puppet-openstack | 19:44 | |
| crinkle | spredzy: oh ok, maybe want to rebase it then? | 19:44 |
| spredzy | crinkle, doesthe above issue ring a bell to you ? | 19:44 |
| spredzy | crinkle, commit seems to be on top of current master/HEAD | 19:46 |
| openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 19:51 |
| crinkle | spredzy: hmm that's weird | 19:55 |
| spredzy | Unfortunately I can't reproduce locally as I have I guess a wrong version of the image. It complains about Could not find command '/usr/bin/add-apt-repository' | 19:59 |
| openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 20:00 |
| spredzy | crinkle, I think attrs[:is_public] returns null here https://github.com/openstack/puppet-glance/blob/master/lib/puppet/provider/glance_image/openstack.rb#L95 | 20:02 |
| *** iurygregory has quit IRC | 20:03 | |
| *** ducttape_ has quit IRC | 20:03 | |
| *** ducttape_ has joined #puppet-openstack | 20:04 | |
| openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 20:05 |
| spredzy | actually that is a result of the image create that failed :/ | 20:05 |
| openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: [WIP] Initial work for multinode support https://review.openstack.org/214897 | 20:13 |
| spredzy | crinkle, could that be the issue ? https://github.com/openstack/glance/blob/stable/kilo/glance/api/v1/images.py#L930-L935 | 20:23 |
| spredzy | So the firt run it work and load the image fine | 20:24 |
| spredzy | but during the second run, the image is not in queue status anymore yet we still specify --copy-from parameter | 20:24 |
| spredzy | leading to https://github.com/openstack/glance/blob/stable/kilo/glance/api/v1/images.py#L951-L953 | 20:24 |
| *** richm has joined #puppet-openstack | 20:25 | |
| crinkle | the --copy-from should only happen when create is called, and create shouldn't get called the second time | 20:26 |
| crinkle | the problem is the prefetch is failing | 20:26 |
| crinkle | possibly because the is_public value is different than expected in liberty? | 20:26 |
| spredzy | yeah I jsut thought about your first sentence .. this is right | 20:26 |
| spredzy | for liberty output I have no idea ... | 20:27 |
| *** iurygregory has joined #puppet-openstack | 20:27 | |
| spredzy | do you have a liberty platform deployed at reach ? | 20:27 |
| spredzy | crinkle, ^ | 20:27 |
| crinkle | yeah i just now reproduced it | 20:27 |
| crinkle | i'll check what openstack image list outputs | 20:28 |
| spredzy | ack | 20:28 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Initial run_tests.sh script https://review.openstack.org/210412 | 20:29 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Create puppet folder layout https://review.openstack.org/201298 | 20:29 |
| *** sergmelikyan has quit IRC | 20:30 | |
| *** sergmelikyan has joined #puppet-openstack | 20:32 | |
| *** ericpeterson has joined #puppet-openstack | 20:33 | |
| *** ducttape_ has quit IRC | 20:36 | |
| crinkle | spredzy: so it looks like they changed the Is Public to Visibility -_- | 20:37 |
| spredzy | crinkle, :/ erf ... | 20:37 |
| spredzy | so now the provider should support both cases ... kilo + liberty | 20:37 |
| spredzy | and possiblity are : [ 'public', 'private', 'shared', 'community' ] | 20:38 |
| *** stamak has quit IRC | 20:38 | |
| spredzy | source: https://wiki.openstack.org/wiki/Glance-v2-community-image-visibility-design | 20:39 |
| crinkle | hmm can we change back to v1? | 20:39 |
| openstackgerrit | Merged openstack/puppet-ironic: Reflect provider change in puppet-openstacklib https://review.openstack.org/209872 | 20:39 |
| crinkle | maybe the acceptance tests could set it to v1 so we can deal with updating the provider later | 20:39 |
| spredzy | +1 on that | 20:40 |
| *** jayg is now known as jayg|g0n3 | 20:40 | |
| spredzy | crinkle, enable_v2_api = False in glance::configs should do it as a temporary fix | 20:44 |
| spredzy | no ? | 20:44 |
| spredzy | directly in the acceptance test , as soon as we update the provider we could remove it from there | 20:44 |
| spredzy | Or better propose a clean parameter to handle that in glance/api.pp | 20:45 |
| crinkle | lemme test | 20:46 |
| spredzy | sure | 20:46 |
| *** tiswanso has quit IRC | 20:46 | |
| crinkle | hmm that doesn't seem to make it work | 20:47 |
| crinkle | now my keystone is being weird... | 20:48 |
| spredzy | crinkle, hmm maybe we need it on both api and registry | 20:49 |
| spredzy | enable_v2_registry = False | 20:50 |
| spredzy | (just to clarify enable_v2_api = False in glance-api.conf and enable_v2_registry = False in glance-registry.conf) | 20:51 |
| crinkle | nope... | 20:51 |
| openstackgerrit | Merged openstack/puppet-swift: Reflect provider change in puppet-openstacklib https://review.openstack.org/209878 | 20:52 |
| crinkle | what's weird is the database still calls it is_public | 20:53 |
| spredzy | crinkle, hmmm that should be enough http://docs.openstack.org/developer/glance/configuring.html#configuring-glance-apis | 20:54 |
| spredzy | crinkle, maybe for backward compat. Not sure how that works to be honnest. | 20:55 |
| crinkle | it could be an issue with the glance client library or openstackclient | 20:56 |
| spredzy | crinkle, if you run the call with --debug it still cals /v2 ? | 20:58 |
| spredzy | calls* | 20:58 |
| *** xarses_ has quit IRC | 20:59 | |
| openstackgerrit | Merged openstack/puppet-monasca: Reflect provider change in puppet-openstacklib https://review.openstack.org/209874 | 21:01 |
| crinkle | spredzy: nope it calls v1 http://paste.openstack.org/show/423737/ | 21:01 |
| crinkle | if you curl the api directly it does the right thing http://paste.openstack.org/show/423738/ | 21:02 |
| spredzy | hmmm | 21:03 |
| spredzy | crinkle, could you paste the output with '--format csv' please | 21:08 |
| crinkle | spredzy: http://paste.openstack.org/show/423739/ | 21:09 |
| openstackgerrit | Merged openstack/puppet-gnocchi: Reflect provider change in puppet-openstacklib https://review.openstack.org/209871 | 21:09 |
| openstackgerrit | Merged openstack/puppet-keystone: Reflect provider change in puppet-openstacklib https://review.openstack.org/202513 | 21:09 |
| openstackgerrit | Merged openstack/puppet-trove: Reflect provider change in puppet-openstacklib https://review.openstack.org/209880 | 21:10 |
| openstackgerrit | Merged openstack/puppet-designate: Reflect provider change in puppet-openstacklib https://review.openstack.org/209870 | 21:12 |
| spredzy | (ps. crinkle thanks for the +A) | 21:12 |
| crinkle | spredzy: thanks for writing docs! that made me happy | 21:14 |
| spredzy | crinkle, hehe | 21:15 |
| spredzy | crinkle, stupid question. Did you redeploy with only v1 or did you just re fired puppet with that change ? Would that make a difference ? | 21:15 |
| crinkle | spredzy: I restarted the glance-api and glance-registry services | 21:17 |
| crinkle | I'm not sure if redeploying from scratch would change it, I can try | 21:17 |
| crinkle | right now I'm deploying at HEAD^ to get kilo to compare it and then i'll ask the openstack-sdk people if they know what's up | 21:18 |
| spredzy | crinkle, ack | 21:19 |
| openstackgerrit | Merged openstack/puppet-tempest: Reflect provider change in puppet-openstacklib https://review.openstack.org/209879 | 21:22 |
| *** ducttape_ has joined #puppet-openstack | 21:23 | |
| *** ericpeterson has quit IRC | 21:23 | |
| openstackgerrit | Merged openstack/puppet-tuskar: Reflect provider change in puppet-openstacklib https://review.openstack.org/209883 | 21:25 |
| *** dgurtner has joined #puppet-openstack | 21:26 | |
| crinkle | spredzy: oh I'm wrong http://paste.openstack.org/show/423761/ | 21:29 |
| * crinkle goes back to the provider | 21:29 | |
| spredzy | crinkle, ok that means that even with Kilo Visibility was already there as suche | 21:30 |
| spredzy | crinkle, with kilo can you also paste the --format csv output, or will it ouput laso visiblity ? | 21:30 |
| spredzy | IT seems that it takes the output from there https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack.rb#L26 | 21:31 |
| crinkle | spredzy: oh the problem is with image show, not list | 21:31 |
| spredzy | ah true ... | 21:31 |
| * spredzy doesn't see clearly anymore | 21:31 | |
| EmilienM | spredzy: manila should be fixed now, no? | 21:32 |
| EmilienM | spredzy: do recheck | 21:32 |
| crinkle | spredzy: look they broke it http://paste.openstack.org/show/423763/ | 21:32 |
| spredzy | crinkle, where are everything :) ? | 21:33 |
| crinkle | spredzy: it's gone!!! | 21:33 |
| spredzy | crinkle, oh boy /o\ | 21:33 |
| spredzy | crinkle, yet the db is correctly filled right ? | 21:33 |
| crinkle | spredzy: yep db looks fine | 21:35 |
| spredzy | crinkle, do you happen to know the URL of their bug tracker ? | 21:36 |
| spredzy | whats the /XX after bugs.launchpad.net | 21:36 |
| crinkle | spredzy: not sure | 21:37 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy MySQL and Keystone (WSGI) https://review.openstack.org/207070 | 21:39 |
| spredzy | crinkle, https://bugs.launchpad.net/python-openstackclient/+bugs | 21:39 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: configure and run tempest https://review.openstack.org/207078 | 21:40 |
| spredzy | crinkle, can I bother you with a last test that is deploy from scratch with v1 on liberty/ubuntu ? | 21:40 |
| crinkle | spredzy: of course | 21:41 |
| crinkle | spredzy: I think it's a problem with glanceclient not openstackclient | 21:41 |
| crinkle | i can show you | 21:41 |
| spredzy | crinkle, sure | 21:42 |
| crinkle | spredzy: http://paste.openstack.org/show/423770/ | 21:42 |
| spredzy | crinkle, oh ok, def not related with openstackclient | 21:43 |
| spredzy | crinkle, apparently no ticket about that one yeet | 21:44 |
| spredzy | crinkle, let me know how the deployment goes | 21:53 |
| spredzy | crinkle, I am about to go afk | 21:53 |
| crinkle | spredzy: okay I'll file a ticket with python-glanceclient | 21:55 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: acceptance/eventlet: make sure apache is stopped https://review.openstack.org/215799 | 21:56 |
| EmilienM | crinkle: re- upgrades issue eventlet/wsgi ^ | 21:56 |
| spredzy | crinkle, ack thanks. | 21:59 |
| *** ducttape_ has quit IRC | 21:59 | |
| *** sergmelikyan has quit IRC | 22:09 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: [WIP] Support for Keystone as Identity Provider https://review.openstack.org/208054 | 22:15 |
| crinkle | spredzy: i don't know why it works on centos... | 22:18 |
| crinkle | even upgrading the glance client to 0.19.0 it works | 22:18 |
| iurygregory | bye people, have a good weekend o/ | 22:18 |
| EmilienM | iurygregory: o/ | 22:19 |
| crinkle | spredzy: so it seems like maybe a packaging issue | 22:19 |
| EmilienM | crinkle: which os? | 22:23 |
| crinkle | EmilienM: ubuntu | 22:23 |
| crinkle | EmilienM: for puppet-glance on liberty | 22:23 |
| EmilienM | damn | 22:23 |
| crinkle | EmilienM: you'll notice most of the glance patches are failing :) | 22:24 |
| EmilienM | crinkle: I haven't followed your discussion - what is broken? | 22:24 |
| EmilienM | AWESOME | 22:24 |
| crinkle | EmilienM: the glance_image provider is broken | 22:24 |
| EmilienM | a friday moreover | 22:24 |
| crinkle | (fwiw it would still be broken if we were using glance client directly instead of openstackclient) | 22:24 |
| EmilienM | crinkle: are you trying to convince yourself ? :-P | 22:25 |
| crinkle | :P | 22:25 |
| *** juzo has joined #puppet-openstack | 22:28 | |
| EmilienM | crinkle: oh by the way I fixed upgrade jobs for keystone: https://jenkins05.openstack.org/job/gate-puppet-keystone-puppet-beaker-rspec-upgrade-dsvm-centos7/71/consoleFull - see https://review.openstack.org/215799 | 22:28 |
| juzo | aks | 22:29 |
| *** juzo is now known as juzo-serio | 22:30 | |
| crinkle | EmilienM: i know, i'm trying to focus on one thing at a time :P | 22:31 |
| juzo-serio | you looser | 22:31 |
| EmilienM | crinkle: sorry, I just wanted to tell you it works as expected | 22:31 |
| juzo-serio | quien habla español? | 22:32 |
| *** juzo-serio is now known as juzo | 22:32 | |
| *** dgurtner has quit IRC | 22:32 | |
| juzo | español??? | 22:33 |
| juzo | quien habla español? | 22:33 |
| juzo | joder | 22:35 |
| openstackgerrit | Merged openstack/puppet-cinder: Reflect provider change in puppet-openstacklib https://review.openstack.org/209869 | 22:35 |
| juzo | death of the angel | 22:36 |
| *** juzo was kicked by crinkle (juzo) | 22:37 | |
| EmilienM | I was about to do it | 22:37 |
| EmilienM | thanks crinkle | 22:37 |
| crinkle | lol | 22:37 |
| EmilienM | lol | 22:37 |
| *** sergmelikyan has joined #puppet-openstack | 22:38 | |
| *** juzo has joined #puppet-openstack | 22:38 | |
| *** sergmelikyan has quit IRC | 22:38 | |
| juzo | life of the angel | 22:39 |
| *** juzo was kicked by crinkle (juzo) | 22:39 | |
| *** crinkle sets mode: +b *!canaima@186.167.242.158 | 22:39 | |
| *** crinkle sets mode: +b *!~canaima@186.167.242.158 | 22:39 | |
| EmilienM | weird people | 22:40 |
| *** sergmelikyan has joined #puppet-openstack | 22:47 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Puppetfile: block inifile to 1.4.1 https://review.openstack.org/215822 | 22:50 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Initial run_tests.sh script https://review.openstack.org/210412 | 22:51 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Create puppet folder layout https://review.openstack.org/201298 | 22:52 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy MySQL and Keystone (WSGI) https://review.openstack.org/207070 | 22:53 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: configure and run tempest https://review.openstack.org/207078 | 22:53 |
| *** sergmelikyan has quit IRC | 22:53 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Puppetfile: block inifile to 1.4.1 https://review.openstack.org/215822 | 22:53 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Initial run_tests.sh script https://review.openstack.org/210412 | 22:54 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Create puppet folder layout https://review.openstack.org/201298 | 22:54 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy MySQL and Keystone (WSGI) https://review.openstack.org/207070 | 22:55 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: configure and run tempest https://review.openstack.org/207078 | 22:55 |
| *** sergmelikyan has joined #puppet-openstack | 22:55 | |
| *** sergmelikyan has quit IRC | 23:00 | |
| openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Manage swift with swift_service defined type https://review.openstack.org/203220 | 23:05 |
| *** dfisher has quit IRC | 23:11 | |
| *** imcsk8 has quit IRC | 23:11 | |
| *** imcsk8 has joined #puppet-openstack | 23:11 | |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Endpoints are APIs version-less https://review.openstack.org/213957 | 23:14 |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_group for Keystone V3 API https://review.openstack.org/202409 | 23:14 |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_trust for Keystone V3 API https://review.openstack.org/200996 | 23:14 |
| *** social has quit IRC | 23:18 | |
| *** fedexo has joined #puppet-openstack | 23:20 | |
| *** aimon has quit IRC | 23:47 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: [WIP] Support for Keystone as Identity Provider https://review.openstack.org/208054 | 23:50 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!