| *** bklei has joined #puppet-openstack | 00:06 | |
| *** jasondotstar is now known as jasondotstar|awa | 00:28 | |
| *** jasondotstar|awa is now known as jasondotstar|afk | 00:28 | |
| *** saneax has joined #puppet-openstack | 00:31 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: acceptance: bump to Liberty release https://review.openstack.org/209286 | 00:39 |
|---|---|---|
| openstackgerrit | Emilien Macchi proposed openstack/puppet-cinder: acceptance: bump to Liberty release https://review.openstack.org/209287 | 00:39 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-designate: acceptance: bump to Liberty release https://review.openstack.org/209288 | 00:39 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: acceptance: bump to Liberty release https://review.openstack.org/209289 | 00:39 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-heat: acceptance: bump to Liberty release https://review.openstack.org/209290 | 00:39 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-horizon: acceptance: bump to Liberty release https://review.openstack.org/209291 | 00:39 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ironic: acceptance: bump to Liberty release https://review.openstack.org/209292 | 00:40 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-manila: acceptance: bump to Liberty release https://review.openstack.org/209293 | 00:40 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: acceptance: bump to Liberty release https://review.openstack.org/209294 | 00:40 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: acceptance: bump to Liberty release https://review.openstack.org/209295 | 00:40 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-sahara: acceptance: bump to Liberty release https://review.openstack.org/209296 | 00:40 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: acceptance: bump to Liberty release https://review.openstack.org/209297 | 00:40 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-trove: acceptance: bump to Liberty release https://review.openstack.org/209298 | 00:40 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-vswitch: acceptance: bump to Liberty release https://review.openstack.org/209299 | 00:40 |
| *** delattec has quit IRC | 00:42 | |
| *** delattec has joined #puppet-openstack | 00:43 | |
| *** prad has quit IRC | 00:43 | |
| *** tiswanso has joined #puppet-openstack | 00:48 | |
| *** tiswanso has quit IRC | 00:51 | |
| *** tiswanso has joined #puppet-openstack | 00:52 | |
| *** xingchao has joined #puppet-openstack | 00:58 | |
| *** rcallawa_ has quit IRC | 01:04 | |
| *** rcallawa has joined #puppet-openstack | 01:06 | |
| *** rcallawa has quit IRC | 01:06 | |
| *** rcallawa has joined #puppet-openstack | 01:06 | |
| *** guimaluf has quit IRC | 01:13 | |
| *** guimaluf has joined #puppet-openstack | 01:14 | |
| *** tiswanso has quit IRC | 01:19 | |
| *** tiswanso has joined #puppet-openstack | 01:19 | |
| *** xingchao has quit IRC | 01:21 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: acceptance: bump to Liberty release https://review.openstack.org/209286 | 01:22 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-cinder: acceptance: bump to Liberty release https://review.openstack.org/209287 | 01:22 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-designate: acceptance: bump to Liberty release https://review.openstack.org/209288 | 01:22 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-heat: acceptance: bump to Liberty release https://review.openstack.org/209290 | 01:22 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ironic: acceptance: bump to Liberty release https://review.openstack.org/209292 | 01:22 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-manila: acceptance: bump to Liberty release https://review.openstack.org/209293 | 01:22 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: acceptance: bump to Liberty release https://review.openstack.org/209294 | 01:22 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: acceptance: bump to Liberty release https://review.openstack.org/209295 | 01:22 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-sahara: acceptance: bump to Liberty release https://review.openstack.org/209296 | 01:22 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-trove: acceptance: bump to Liberty release https://review.openstack.org/209298 | 01:22 |
| *** gildub has quit IRC | 01:32 | |
| openstackgerrit | Dan Prince proposed openstack/puppet-tripleo: Add pre_create_files resource and helper class https://review.openstack.org/208774 | 01:33 |
| openstackgerrit | Dan Prince proposed openstack/puppet-tripleo: tripleo::noop: add support for more resources https://review.openstack.org/209030 | 01:33 |
| *** tiswanso has quit IRC | 01:34 | |
| *** tiswanso has joined #puppet-openstack | 01:34 | |
| *** imcsk8 has quit IRC | 01:44 | |
| *** imcsk8 has joined #puppet-openstack | 01:44 | |
| *** ilbot3 has quit IRC | 01:45 | |
| *** sergmelikyan has joined #puppet-openstack | 01:45 | |
| *** ilbot3 has joined #puppet-openstack | 01:47 | |
| *** gildub has joined #puppet-openstack | 01:48 | |
| *** saneax has quit IRC | 02:11 | |
| openstackgerrit | Xiaohua Yuan proposed openstack/puppet-designate: Add notification related parameters to designate https://review.openstack.org/209318 | 02:14 |
| *** xarses has quit IRC | 02:27 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: WSGI: make it work, and test it with acceptance https://review.openstack.org/209320 | 02:37 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: acceptance: clean an old FIXME in manifest https://review.openstack.org/209321 | 02:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-vswitch: acceptance: bump to Liberty release https://review.openstack.org/209299 | 02:57 |
| openstackgerrit | Clayton O'Neill proposed openstack/puppet-nova: Move os_region_name config option https://review.openstack.org/209324 | 02:57 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: WSGI: make it work, and test it with acceptance https://review.openstack.org/209320 | 03:02 |
| *** xarses has joined #puppet-openstack | 03:16 | |
| *** sergmelikyan has quit IRC | 03:21 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: WSGI: make it work, and test it with acceptance https://review.openstack.org/209320 | 03:32 |
| *** tiswanso has quit IRC | 03:37 | |
| *** xingchao has joined #puppet-openstack | 03:37 | |
| *** benh57 has quit IRC | 03:38 | |
| *** saneax has joined #puppet-openstack | 03:56 | |
| *** xingchao has quit IRC | 04:00 | |
| *** jasondotstar|afk has quit IRC | 04:09 | |
| openstackgerrit | Limor Stotland proposed openstack/puppet-mistral: puppet-mistral: initial commit https://review.openstack.org/208457 | 04:14 |
| *** fedexo has joined #puppet-openstack | 04:40 | |
| *** michchap has joined #puppet-openstack | 04:47 | |
| *** pcaruana has quit IRC | 04:59 | |
| *** saneax has quit IRC | 05:38 | |
| *** fedexo has quit IRC | 05:39 | |
| *** saneax has joined #puppet-openstack | 05:40 | |
| *** saneax has quit IRC | 05:54 | |
| *** saneax has joined #puppet-openstack | 05:58 | |
| *** jasondotstar|afk has joined #puppet-openstack | 06:10 | |
| *** mdorman has quit IRC | 06:11 | |
| *** jasondotstar|afk has quit IRC | 06:15 | |
| *** pcaruana has joined #puppet-openstack | 06:36 | |
| *** darkxploit has quit IRC | 06:40 | |
| *** rcarrillocruz has quit IRC | 06:42 | |
| *** darkxploit has joined #puppet-openstack | 06:52 | |
| *** darkxploit has quit IRC | 06:53 | |
| *** darkxploit has joined #puppet-openstack | 06:54 | |
| *** darkxploit has quit IRC | 06:55 | |
| *** darkxploit has joined #puppet-openstack | 06:56 | |
| *** ricky1 has joined #puppet-openstack | 06:56 | |
| *** darkxploit has quit IRC | 06:58 | |
| *** darkxploit has joined #puppet-openstack | 06:58 | |
| *** darkxploit has quit IRC | 07:00 | |
| *** darkxploit has joined #puppet-openstack | 07:00 | |
| *** xingchao has joined #puppet-openstack | 07:05 | |
| *** dgurtner has joined #puppet-openstack | 07:15 | |
| *** ricky1 has quit IRC | 07:19 | |
| *** ddmitriev1 has joined #puppet-openstack | 07:25 | |
| *** ricky1 has joined #puppet-openstack | 07:25 | |
| openstackgerrit | Limor Stotland proposed openstack/puppet-mistral: puppet-mistral: initial commit https://review.openstack.org/209382 | 07:37 |
| openstackgerrit | Limor Stotland proposed openstack/puppet-mistral: puppet-mistral: initial commit https://review.openstack.org/209382 | 07:38 |
| *** arnaud_orange has joined #puppet-openstack | 07:38 | |
| *** arnaud_orange has left #puppet-openstack | 07:39 | |
| *** xingchao has quit IRC | 07:45 | |
| openstackgerrit | Limor Stotland proposed openstack/puppet-mistral: puppet-mistral: initial commit https://review.openstack.org/208457 | 07:45 |
| *** trash has quit IRC | 07:49 | |
| *** dachary1 has joined #puppet-openstack | 07:59 | |
| *** paramite has joined #puppet-openstack | 08:06 | |
| *** gildub has quit IRC | 08:08 | |
| *** trash has joined #puppet-openstack | 08:10 | |
| *** jistr has joined #puppet-openstack | 08:14 | |
| *** xingchao has joined #puppet-openstack | 08:20 | |
| *** darkxploit has quit IRC | 08:20 | |
| *** darkxploit has joined #puppet-openstack | 08:32 | |
| openstackgerrit | Merged openstack/puppet-neutron: acceptance: test OVS bridge listing https://review.openstack.org/192176 | 08:45 |
| *** xingchao has quit IRC | 08:45 | |
| *** ricky1 is now known as rcarrillocruz | 08:48 | |
| *** jasondotstar|afk has joined #puppet-openstack | 08:48 | |
| *** xingchao has joined #puppet-openstack | 08:49 | |
| *** jasondotstar|afk is now known as jasondotstar | 08:54 | |
| openstackgerrit | Martin Mágr proposed openstack/puppet-cinder: DB purge cron job https://review.openstack.org/208939 | 08:58 |
| *** darkxploit has quit IRC | 09:11 | |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-cinder: Add 'host' parameter to cinder.conf https://review.openstack.org/209412 | 09:20 |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-cinder: Add 'host' parameter to cinder.conf https://review.openstack.org/209412 | 09:21 |
| openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Fix rgw keystone https://review.openstack.org/153783 | 09:23 |
| *** darkxploit has joined #puppet-openstack | 09:23 | |
| *** gfidente has joined #puppet-openstack | 09:33 | |
| *** gfidente has quit IRC | 09:33 | |
| *** gfidente has joined #puppet-openstack | 09:33 | |
| *** xingchao has quit IRC | 09:33 | |
| *** chem has joined #puppet-openstack | 09:35 | |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-manila: Introduce manila::db class https://review.openstack.org/208402 | 10:02 |
| *** jasondotstar has quit IRC | 10:02 | |
| *** jasondotstar has joined #puppet-openstack | 10:03 | |
| *** chem has quit IRC | 10:07 | |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-cinder: Add 'host' parameter to cinder.conf https://review.openstack.org/209412 | 10:12 |
| *** xingchao has joined #puppet-openstack | 10:13 | |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-cinder: Add 'host' parameter to cinder.conf https://review.openstack.org/209412 | 10:17 |
| *** saneax has quit IRC | 10:32 | |
| *** gildub has joined #puppet-openstack | 10:38 | |
| openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Clean the documentation for rspec-beaker https://review.openstack.org/209449 | 10:48 |
| openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Clean the documentation for rspec-beaker https://review.openstack.org/209449 | 10:53 |
| *** rcallawa has quit IRC | 10:59 | |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-openstacklib: Allow the use of an ensure_absent_val param https://review.openstack.org/202574 | 11:07 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-openstacklib: Allow the use of an ensure_absent_val param https://review.openstack.org/202574 | 11:08 |
| *** saneax has joined #puppet-openstack | 11:09 | |
| *** sergmelikyan has joined #puppet-openstack | 11:11 | |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-keystone: Reflect provider change in openstacklib https://review.openstack.org/202513 | 11:19 |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-cinder: Test commit https://review.openstack.org/209468 | 11:22 |
| *** gildub has quit IRC | 11:38 | |
| *** xingchao has quit IRC | 11:45 | |
| *** gildub has joined #puppet-openstack | 11:50 | |
| *** chem has joined #puppet-openstack | 11:54 | |
| *** ferest has joined #puppet-openstack | 11:56 | |
| *** jistr has quit IRC | 11:56 | |
| *** morazi has joined #puppet-openstack | 11:59 | |
| *** ferest has quit IRC | 12:08 | |
| *** dprince has joined #puppet-openstack | 12:10 | |
| *** jayg|g0n3 is now known as jayg | 12:11 | |
| *** tiswanso has joined #puppet-openstack | 12:17 | |
| *** jistr has joined #puppet-openstack | 12:18 | |
| *** jistr is now known as jistr|biab | 12:19 | |
| *** tiswanso has quit IRC | 12:19 | |
| *** tiswanso has joined #puppet-openstack | 12:20 | |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-cinder: Add 'host' parameter to cinder.conf https://review.openstack.org/209412 | 12:24 |
| *** prad has joined #puppet-openstack | 12:52 | |
| *** tiswanso has quit IRC | 12:56 | |
| *** rcallawa has joined #puppet-openstack | 13:04 | |
| *** jistr|biab is now known as jistr | 13:06 | |
| *** chem has quit IRC | 13:07 | |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-keystone: Reflect provider change in openstacklib https://review.openstack.org/202513 | 13:09 |
| *** xingchao has joined #puppet-openstack | 13:12 | |
| spredzy | clayton, morning. While running the acceptance test for <SERVIC DEFAULT> https://review.openstack.org/#/c/202513/6/spec/acceptance/keystone_config_spec.rb,cm I realize that test work fine on the first round, but on the second round it go and recreates the param (which is latter handled as a remove me from the file). This causes the second run (the one testing for idempotency) to fail as line does not exist in the file but we are asking puppet t | 13:12 |
| spredzy | o create it (trace are here: http://paste.openstack.org/show/408326/) | 13:12 |
| spredzy | Would you have an idea on how to tackle this ? I was thinking about maybe overidding exists? somehow since we have access to the resources[] var but I am unsure if this is the right way to go :/ | 13:13 |
| clayton | looking | 13:13 |
| spredzy | thanks | 13:15 |
| iurygregory | good morning | 13:18 |
| spredzy | o/ | 13:18 |
| clayton | I think it would be cleaner (and would probably avoid some of these issues) if we could figure out how to set the ensure property to absent when the value is set to that, instead of trying to remove it under the hood | 13:18 |
| spredzy | true | 13:19 |
| iurygregory | hey spredzy, about https://review.openstack.org/#/c/207639/ really need update the commit message, right? | 13:20 |
| clayton | spredzy: unfortunately the documentation for this sort of stuff really seems almost non-existent | 13:21 |
| spredzy | iurygregory, is that the correct link ? If so not sure I understand | 13:21 |
| spredzy | clayton, yep I have no idea on how I could do that | 13:22 |
| spredzy | I guess I'll look for it | 13:22 |
| clayton | looking at the inifile type, it's not clear it's even using the default ensurable definition | 13:22 |
| spredzy | I hope it is doable to be able to do that sort of thing. Like if value == X undestand it as ensure: absnt | 13:22 |
| clayton | oh, it is using ensureable, I was wrong | 13:23 |
| iurygregory | yeah spredzy , the commit message says diretory instead of directory XD | 13:23 |
| *** dprince has quit IRC | 13:23 | |
| spredzy | iurygregory, I wouldn't block the commit on this personally | 13:23 |
| clayton | so it occurs to me that we already do something similar in these providers when we set the filename for the inifile | 13:24 |
| iurygregory | thanks spredzy ^^ | 13:24 |
| *** dprince has joined #puppet-openstack | 13:24 | |
| clayton | maybe it's as simple as overriding the ensure method? | 13:24 |
| clayton | spredzy: I think it might be, for exampel, look at how section and setting are defined in the ini child providers | 13:25 |
| *** gildub has quit IRC | 13:25 | |
| spredzy | resource[:name].split | 13:25 |
| clayton | heh, found this thread with Luke explaining to Dan how to implement keystone_config - https://groups.google.com/forum/#!topic/puppet-dev/ZAzxiKJvrRk | 13:25 |
| spredzy | 2012 | 13:27 |
| spredzy | woo | 13:27 |
| spredzy | puppet-keystone has been around for a while | 13:27 |
| *** ferest has joined #puppet-openstack | 13:29 | |
| *** tiswanso has joined #puppet-openstack | 13:30 | |
| *** ferest has quit IRC | 13:32 | |
| *** paramite is now known as paramite|afk | 13:37 | |
| pabelanger | crinkle: if you go back to the initial patchset from the issue, you'll see the Rakefile was how it exists now. EmilienM asked me to change it around patch 3/4 | 13:44 |
| *** tiswanso has quit IRC | 13:48 | |
| spredzy | clayton, \o/. Thanks a lot ! | 13:50 |
| spredzy | clayton, http://pastebin.com/Z1QKAVrD | 13:50 |
| *** paramite|afk is now known as paramite | 13:51 | |
| *** dfisher has joined #puppet-openstack | 13:51 | |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-openstacklib: Allow the use of an ensure_absent_val param https://review.openstack.org/202574 | 13:54 |
| clayton | spredzy: that fixes it then? | 13:55 |
| spredzy | yes, I am updating my review and reruning acceptance test on keystone to confirm | 13:56 |
| *** tiswanso has joined #puppet-openstack | 13:56 | |
| spredzy | but it looks like it | 13:56 |
| clayton | nice :) | 13:56 |
| spredzy | my initial though was way too complicated | 13:56 |
| spredzy | Thanks a lot for leading me to the proper solution | 13:56 |
| spredzy | clayton++ | 13:56 |
| clayton | :) | 13:56 |
| *** zigo has quit IRC | 14:05 | |
| spredzy | clayton, http://pastebin.com/g6zYq9Lc :) | 14:06 |
| *** zigo has joined #puppet-openstack | 14:06 | |
| clayton | nice! | 14:06 |
| clayton | how are you running the acceptance tests locally? I was trying to do that the other day but I only saw the nodepool nodesets | 14:07 |
| openstackgerrit | Ryan Hefner proposed openstack/puppet-tripleo: Enable Manila Service https://review.openstack.org/204249 | 14:08 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-keystone: Reflect provider change in openstacklib https://review.openstack.org/202513 | 14:09 |
| spredzy | clayton, in the spec/acceptance/nodeset/default.yml I comment box_url and modify box to point to a local vagrant box (I am using libvirt as a backend - so the box needs to be compatabile) | 14:11 |
| spredzy | and then run bundle exec rake beaker | 14:12 |
| spredzy | Theire prob might be easier ways, but using libvirt, prevent me from using the raw (ubuntu/centos) files provides in the modules | 14:13 |
| spredzy | as it goes try to download https://vagrantcloud.com/puppetlabs/ubuntu-14.04-64-nocm | 14:13 |
| spredzy | that works only with virtualbox and vmware :( | 14:13 |
| spredzy | pabelanger, quick q. Will the acceptance test in the CI pull the proper dependenies if specied in the commit message ? https://review.openstack.org/#/c/202513/ | 14:15 |
| spredzy | will this acceptance job pull the right openstacklib commit before running tests ? | 14:16 |
| spredzy | clayton, last question on that topic. Since openstack_config is a proxy (ie. we can't call it directly and it is not the intemps - type has no parameters) can it be tested ? Not sure what to test here anyway :/ | 14:17 |
| spredzy | The keystone review is ensuring it is functionnaly working, but I am not sure I can unit test the openstack_config resource | 14:18 |
| pabelanger | spredzy: you should use the Depends-On header in your commit message if your patch depends on another openstack project commit. Zuul will then resolve the cross repo dependency and make sure the patch gets pull into the repo | 14:20 |
| pabelanger | so, looks like you did it right | 14:20 |
| spredzy | pabelanger, ok cool. I was wondering for our acceptance test. Looks like it does. Thanks ! | 14:21 |
| pabelanger | spredzy: Ya, I think crinkle added the properly zuul-cloner functionality into it | 14:21 |
| *** delattec has quit IRC | 14:22 | |
| dmsimard | dgurtner: reviewed rgw | 14:25 |
| *** saneax has quit IRC | 14:25 | |
| *** chem has joined #puppet-openstack | 14:36 | |
| clayton | spredzy: I think it'd be reasonable to include a test provider in openstacklib, preferably under spec if we can get the include path to allow that | 14:40 |
| spredzy | clayton, but openstack_config can't be called as it has no parameters https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/type/openstack_config.rb | 14:42 |
| spredzy | so except Puppet::Type.new(:openstack_config) there isn't much I can test the provider. | 14:42 |
| spredzy | Am I missing something | 14:42 |
| spredzy | ? | 14:42 |
| *** jistr has quit IRC | 14:44 | |
| openstackgerrit | Merged stackforge/puppet-ceph: Fix a race condition in mon creation https://review.openstack.org/209250 | 14:44 |
| clayton | why not just add parameters to it so we can test it then? | 14:44 |
| *** jistr has joined #puppet-openstack | 14:44 | |
| clayton | why do we need that stub at all? | 14:44 |
| *** delattec has joined #puppet-openstack | 14:45 | |
| spredzy | The idea was to have it in a single place, so things like section / setting / exists? etc... can be changed with a single commit | 14:45 |
| spredzy | and not 1 commit / module | 14:45 |
| spredzy | but it is not intended to be call directly | 14:46 |
| clayton | well, I get why we need the provider in openstacklib, I'm not sure what purpose the type serves | 14:46 |
| spredzy | Oh that might just be a mis-conception from my part, I thought we needed to have a type that comes along a provider | 14:47 |
| clayton | I don't think so, but I'm not sure | 14:48 |
| degorenko | hi :) let's merge this: https://review.openstack.org/#/c/206981/ | 14:48 |
| chem | I think something is missing in the openstack keystone package, where I should discuss or bugzilla it ? | 14:49 |
| spredzy | clayton, yes we do else it fails | 14:49 |
| spredzy | just tried it | 14:49 |
| spredzy | chem, if it is on CentOS 7 I guess #rdo | 14:50 |
| chem | spredzy: oki I'll go there, thanks | 14:50 |
| openstackgerrit | Merged stackforge/puppet-ceph: Clean the documentation for rspec-beaker https://review.openstack.org/209449 | 14:50 |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-cinder: Test commit https://review.openstack.org/209468 | 14:50 |
| *** darkxploit has quit IRC | 14:57 | |
| degorenko | folks, please https://review.openstack.org/#/c/205987/ :) | 14:58 |
| dgurtner | dmsimard: thanks you :) | 15:04 |
| dgurtner | dmsimard: seems today is a good day for reviews | 15:04 |
| dgurtner | dachary1: xarses thank you for your reviews as well! | 15:04 |
| *** beddari has quit IRC | 15:04 | |
| *** rlaurika has quit IRC | 15:04 | |
| *** rlaurika has joined #puppet-openstack | 15:05 | |
| *** beddari has joined #puppet-openstack | 15:06 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Enable Manila Service https://review.openstack.org/204249 | 15:07 |
| openstackgerrit | Merged stackforge/puppet-ceph: osd: fix pattern when matching ${data} https://review.openstack.org/201730 | 15:10 |
| *** dprince has quit IRC | 15:11 | |
| openstackgerrit | craig delatte proposed openstack/puppet-cinder: module to set quotas per volume type https://review.openstack.org/197572 | 15:12 |
| dgurtner | dmsimard: lol... switching to kilo has some silly difficulties involved | 15:15 |
| dgurtner | dmsimard: there's no kilo branch on the rdo repositories to grab the kilo GPG key | 15:16 |
| dgurtner | dmsimard: so It's going to change and thus brake over time :( | 15:16 |
| *** sergmelikyan has quit IRC | 15:24 | |
| *** dachary1 has quit IRC | 15:26 | |
| openstackgerrit | Merged openstack/puppet-murano: Update Murano README https://review.openstack.org/207639 | 15:27 |
| openstackgerrit | Merged openstack/puppet-cinder: set parameter host in vmdk.pp https://review.openstack.org/157004 | 15:28 |
| openstackgerrit | Merged openstack/puppet-designate: Add notification related parameters to designate https://review.openstack.org/209318 | 15:36 |
| *** dachary1 has joined #puppet-openstack | 15:42 | |
| *** dachary1 has quit IRC | 15:43 | |
| openstackgerrit | Arnoud de Jonge proposed openstack/puppet-designate: Added pool manager. https://review.openstack.org/209569 | 15:45 |
| openstackgerrit | Arnoud de Jonge proposed openstack/puppet-designate: SSL support for RabbitMQ https://review.openstack.org/209570 | 15:45 |
| *** paramite has quit IRC | 15:50 | |
| dmsimard | dgurtner: can you add a comment prior to that apt/yum repo block with a TODO or something ? | 15:57 |
| dgurtner | dmsimard: sure | 15:58 |
| openstackgerrit | Cody Herriges proposed openstack/puppet-modulesync-configs: Add helper for concat verification https://review.openstack.org/209588 | 16:14 |
| *** jistr has quit IRC | 16:23 | |
| *** jistr has joined #puppet-openstack | 16:23 | |
| *** jistr has quit IRC | 16:24 | |
| *** pcaruana has quit IRC | 16:26 | |
| *** openstackgerrit_ has joined #puppet-openstack | 16:30 | |
| *** dprince has joined #puppet-openstack | 16:38 | |
| *** xarses has quit IRC | 16:40 | |
| dgurtner | dmsimard: hmm, there seems to be something broken with keystone kilo on trusty :( | 16:47 |
| dmsimard | oh ? | 16:47 |
| dgurtner | dmsimard: Error: Execution of '/usr/bin/openstack role create --format shell admin' returned 1: ERROR: openstackclient.shell Exception raised: (pbr 0.7.0 (/usr/lib/python2.7/dist-packages), Requirement.parse('pbr>=0.6,!=0.7,<1.0')) | 16:48 |
| dmsimard | perhaps check what they're trying to implement in the beaker tests, ex: https://review.openstack.org/#/c/207070/6/modules/openstack_integration/manifests/testing/scenario001.pp | 16:50 |
| dgurtner | dmsimard: that's actually failing trusty as well (but because of other reasons) | 16:53 |
| dmsimard | dgurtner: Yeah I realized after sending the link :P | 16:54 |
| dmsimard | EmilienM: Have you figured out why trusty is failing in https://review.openstack.org/#/c/207070 ? | 16:54 |
| dmsimard | We're trying to test Ceph RGW with Keystone kilo on trusty | 16:54 |
| *** xarses has joined #puppet-openstack | 16:58 | |
| dmsimard | pabelanger: EmilienM told me to poke you about ^ | 16:58 |
| dgurtner | dmsimard: ok, it's doable, need to install and update a bunch of stuff for it to work... | 16:59 |
| pabelanger | dmsimard: Ya, on my place to look at. Something is funky with the apt repo | 17:03 |
| *** openstackgerrit_ has quit IRC | 17:05 | |
| dmsimard | dgurtner: can do juno and update to kilo some other time, depends how many hacks we need | 17:16 |
| dgurtner | dmsimard: that would make more sense from my point of view | 17:19 |
| dmsimard | dgurtner: I can settle for that | 17:20 |
| dgurtner | dmsimard: to my understanding the current ceph::rgw::keystone code is not working, so I would rather the changes go through with tests based on keystone juno and then upgrade later | 17:20 |
| *** rcallawa has quit IRC | 17:23 | |
| *** openstackgerrit_ has joined #puppet-openstack | 17:24 | |
| *** xingchao has quit IRC | 17:26 | |
| dgurtner | dmsimard: given it one last shot to see if I can get it fixed :) | 17:27 |
| *** tiswanso has quit IRC | 17:28 | |
| *** tiswanso has joined #puppet-openstack | 17:30 | |
| openstackgerrit | Clayton O'Neill proposed openstack/puppet-nova: Move os_region_name config option https://review.openstack.org/209324 | 17:33 |
| *** openstackgerrit_ has quit IRC | 17:34 | |
| openstackgerrit | Clayton O'Neill proposed openstack/puppet-cinder: Add lock_path to cinder config https://review.openstack.org/209216 | 17:36 |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-cinder: Test commit https://review.openstack.org/209468 | 17:37 |
| dgurtner | dmsimard: ok... it's not going to work | 17:38 |
| dgurtner | dmsimard: should I add another commit just fixing the small things (like the www-data vs. apache default) or should we just take the current? | 17:39 |
| dmsimard | Let's merge that one and address the nitpicks (xarses mentioned one too) in another commit | 17:41 |
| dmsimard | I'll also commit something to remove the tests on the versions we don't want to support anymore (dumpling and giant) | 17:42 |
| dgurtner | dmsimard: sounds good | 17:44 |
| dgurtner | dmsimard: we also still need to have a discussion about supporting centos6 and precise again... I think it was inconclusive and xarses wanted to get input from a wider group of people | 17:44 |
| dgurtner | dmsimard: mainly because openstack past icehouse isn't supported anymore on the older distros | 17:48 |
| dgurtner | dmsimard: ok, have to leave... might be back online later today/tonight | 17:49 |
| dmsimard | dgurtner: I'm of the opinion to support only trusty/centos7 so I don't need convincing :) | 17:49 |
| xarses | dmsimard: sounds good to me | 17:51 |
| xarses | if you are ok to merge now we can iterate in a follow up | 17:51 |
| *** dgurtner has quit IRC | 17:53 | |
| *** sergmelikyan has joined #puppet-openstack | 18:20 | |
| *** mdorman has joined #puppet-openstack | 18:22 | |
| *** sergmelikyan has quit IRC | 18:22 | |
| *** xingchao has joined #puppet-openstack | 18:26 | |
| *** xingchao has quit IRC | 18:31 | |
| openstackgerrit | Giulio Fidente proposed stackforge/puppet-ceph: Make args for conf, mons, osds and keys wrappers optional https://review.openstack.org/209625 | 18:41 |
| *** tiswanso has quit IRC | 18:41 | |
| *** arnaud_orange1 has joined #puppet-openstack | 18:43 | |
| *** tiswanso has joined #puppet-openstack | 18:45 | |
| *** openstackgerrit has quit IRC | 18:46 | |
| *** openstackgerrit has joined #puppet-openstack | 18:47 | |
| *** fvollero has quit IRC | 19:06 | |
| *** fvollero has joined #puppet-openstack | 19:10 | |
| *** angdraug has joined #puppet-openstack | 19:13 | |
| *** barthalion has joined #puppet-openstack | 19:23 | |
| *** sergmelikyan has joined #puppet-openstack | 19:58 | |
| *** jasondot_ has joined #puppet-openstack | 20:07 | |
| *** jasondotstar has quit IRC | 20:08 | |
| *** dgurtner has joined #puppet-openstack | 20:10 | |
| *** sergmelikyan has quit IRC | 20:26 | |
| *** sergmelikyan has joined #puppet-openstack | 20:46 | |
| *** tiswanso has quit IRC | 21:09 | |
| *** dgurtner has quit IRC | 21:22 | |
| *** dgurtner has joined #puppet-openstack | 21:36 | |
| *** beddari_ has joined #puppet-openstack | 21:38 | |
| *** jayg is now known as jayg|g0n3 | 21:38 | |
| *** beddari has quit IRC | 21:39 | |
| *** beddari_ is now known as beddari | 21:39 | |
| *** guimaluf has quit IRC | 21:40 | |
| *** tiswanso has joined #puppet-openstack | 21:47 | |
| *** prad has quit IRC | 21:48 | |
| *** tiswanso has quit IRC | 21:51 | |
| *** gfidente has quit IRC | 21:58 | |
| openstackgerrit | Vasyl Saienko proposed openstack/puppet-horizon: add CUSTOM_THEME_PATH variable https://review.openstack.org/209685 | 22:12 |
| *** angdraug has quit IRC | 22:14 | |
| *** dgurtner has quit IRC | 22:19 | |
| *** delattec has quit IRC | 22:21 | |
| *** dprince has quit IRC | 22:32 | |
| *** angdraug has joined #puppet-openstack | 22:33 | |
| *** prad has joined #puppet-openstack | 22:38 | |
| *** saneax has joined #puppet-openstack | 23:05 | |
| *** dfisher has quit IRC | 23:08 | |
| *** sergmelikyan has quit IRC | 23:46 | |
| *** tiswanso has joined #puppet-openstack | 23:48 | |
| *** tiswanso has quit IRC | 23:52 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!