Thursday, 2016-04-07

« Wednesday, 2016-04-06 Index Friday, 2016-04-08 »
*** xarses_ has quit IRC00:02
*** baker has quit IRC00:04
*** baker has joined #puppet-openstack00:05
*** johnmilton has joined #puppet-openstack00:25
*** dfisher has joined #puppet-openstack00:27
dfisher_ody: ping again?00:28
openstackgerritMark Kirkwood proposed openstack/puppet-swift: Allow a replication network to be specified Closes-Bug: #1491660 Change-Id: I1d1d327b7d32f37550edcc036f5ab2bbb1b37ba1  https://review.openstack.org/30248300:42
openstackbug 1491660 in puppet-swift "Cannot specify replicaton network in swift-object-device" [Undecided,In progress] https://launchpad.net/bugs/1491660 - Assigned to Adam Vinsh (adam-vinsh)00:42
dmsimardCame across a new issue in my Mitaka testing, filed a bug: https://bugs.launchpad.net/puppet-nova/+bug/156715700:47
openstackLaunchpad bug 1567157 in puppet-nova "nova::network::neutron should set use_neutron=true" [Undecided,New]00:47
*** saneax is now known as saneax_AFK00:50
*** dfisher has quit IRC00:58
*** dprince has joined #puppet-openstack01:24
*** baker has quit IRC01:28
*** baker has joined #puppet-openstack01:28
*** ducttape_ has joined #puppet-openstack01:34
*** baker has quit IRC01:39
*** vinsh has quit IRC01:40
*** vinsh_ has joined #puppet-openstack01:40
*** baker has joined #puppet-openstack01:44
*** baker has quit IRC01:47
*** ducttape_ has quit IRC01:48
*** baker has joined #puppet-openstack02:11
*** xingchao has joined #puppet-openstack02:16
*** dprince has quit IRC02:19
*** ducttape_ has joined #puppet-openstack02:20
*** richm has quit IRC02:33
*** baker_ has joined #puppet-openstack02:39
*** baker has quit IRC02:42
*** social has quit IRC02:49
*** social has joined #puppet-openstack02:51
*** openstackstatus has quit IRC03:01
*** michchap has quit IRC03:02
*** xarses has joined #puppet-openstack03:02
*** michchap has joined #puppet-openstack03:06
*** chandankumar has joined #puppet-openstack03:18
*** ducttape_ has quit IRC03:19
*** v1k0d3n has joined #puppet-openstack03:19
*** ducttape_ has joined #puppet-openstack03:20
*** baker_ has quit IRC03:27
*** chandankumar has quit IRC03:38
*** v1k0d3n has quit IRC03:51
*** johnmilton has quit IRC03:54
*** ducttape_ has quit IRC04:00
*** fedexo has joined #puppet-openstack04:02
*** chandankumar has joined #puppet-openstack04:20
*** ntaylor1781 has quit IRC04:28
*** xingchao has quit IRC05:03
*** jaosorior has joined #puppet-openstack05:13
*** michchap has quit IRC05:22
*** michchap has joined #puppet-openstack05:24
*** fedexo has quit IRC05:39
*** fedexo has joined #puppet-openstack05:42
*** rcernin has joined #puppet-openstack05:45
*** xingchao has joined #puppet-openstack05:46
*** xingchao has quit IRC05:53
*** saneax_AFK is now known as saneax05:57
*** jmikolajczak has joined #puppet-openstack05:58
*** gfidente has joined #puppet-openstack06:03
*** fedexo has quit IRC06:27
*** leanderthal|afk is now known as leanderthal06:37
*** tesseract has joined #puppet-openstack06:45
*** tesseract is now known as Guest1287606:45
openstackgerritMichael Chapman proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/29343606:46
*** dgurtner has joined #puppet-openstack07:02
openstackgerritBabu Shanmugam proposed openstack/puppet-neutron: Add support for OVN plugin  https://review.openstack.org/30161307:17
*** foxhunt_away is now known as foxhunt7207:32
*** dtantsur|afk is now known as dtantsur07:36
*** amoralej has joined #puppet-openstack07:41
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Remove individual service certificates  https://review.openstack.org/30149307:44
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add missing services ports to service_ports map  https://review.openstack.org/30257307:44
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer listeners  https://review.openstack.org/30257407:44
*** jpena|off is now known as jpean07:44
*** jpean is now known as jpena07:44
*** jaosorior has quit IRC07:47
*** jaosorior has joined #puppet-openstack07:47
*** guessi has joined #puppet-openstack07:49
*** yolanda has quit IRC08:03
*** paramite has joined #puppet-openstack08:07
openstackgerritMerged openstack/puppet-barbican: Modify ensure_package parameter in dogtag plugin  https://review.openstack.org/30186908:23
*** skolekonov has joined #puppet-openstack08:28
*** jistr has joined #puppet-openstack08:30
*** openstackstatus has joined #puppet-openstack08:30
*** ChanServ sets mode: +v openstackstatus08:30
*** hjensas has quit IRC08:33
-openstackstatus- NOTICE: jobs depending on npm are now working again08:35
*** dtantsur is now known as dtantsur|brb08:38
*** yolanda has joined #puppet-openstack08:43
*** rmart04 has joined #puppet-openstack08:45
*** yolanda has quit IRC08:49
*** yolanda has joined #puppet-openstack08:50
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer listeners  https://review.openstack.org/30257408:50
*** hjensas has joined #puppet-openstack08:51
*** hjensas has quit IRC08:51
*** hjensas has joined #puppet-openstack08:51
openstackgerritMark Kirkwood proposed openstack/puppet-swift: Allow a replication network to be specified Closes-Bug: #1491660 Change-Id: I1d1d327b7d32f37550edcc036f5ab2bbb1b37ba1  https://review.openstack.org/30248308:52
openstackbug 1491660 in puppet-swift "Cannot specify replicaton network in swift-object-device" [Undecided,In progress] https://launchpad.net/bugs/1491660 - Assigned to Mark Kirkwood (mark-kirkwood)08:52
*** mgould has joined #puppet-openstack08:53
*** ToMiles has joined #puppet-openstack08:58
*** jaosorior has quit IRC08:59
*** jaosorior has joined #puppet-openstack09:00
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer listeners  https://review.openstack.org/30257409:01
openstackgerritBabu Shanmugam proposed openstack/puppet-neutron: Add support for OVN plugin  https://review.openstack.org/30161309:06
*** derekh has joined #puppet-openstack09:09
openstackgerritSergey Kolekonov proposed openstack/puppet-neutron: Pass firewall_driver to ML2 plugin conf file  https://review.openstack.org/30044209:22
*** dtantsur|brb is now known as dtantsur09:55
*** karimb has joined #puppet-openstack10:13
*** karimb has quit IRC10:26
*** skolekonov has quit IRC10:29
*** _degorenko|afk is now known as degorenko10:32
*** Matum has quit IRC10:34
*** karimb has joined #puppet-openstack10:38
*** Guest12876 is now known as tesseract10:47
*** tesseract is now known as Guest7403610:48
openstackgerritDmitry Burmistrov proposed openstack/puppet-oslo: Add oslo::messaging::default define  https://review.openstack.org/30266410:52
*** karimb has quit IRC10:53
*** karimb has joined #puppet-openstack10:54
*** iberezovskiy_ is now known as iberezovskiy10:57
*** karimb has quit IRC11:01
*** johnmilton has joined #puppet-openstack11:08
openstackgerritDmitry Burmistrov proposed openstack/puppet-oslo: Add oslo::messaging::default define  https://review.openstack.org/30266411:10
openstackgerritBabu Shanmugam proposed openstack/puppet-ovn: Added .gitreview for the puppet-ovn project  https://review.openstack.org/30204411:16
openstackgerritMiles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/30239011:17
openstackgerritMiles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/30237611:17
EmilienMgood morning11:35
*** cgfbee has quit IRC11:43
dmburmistrovhi!11:44
dmburmistrovEmilienM, can you look at https://review.openstack.org/#/c/301079/ ?11:44
*** cgfbee has joined #puppet-openstack11:44
openstackgerritMiles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/30239011:46
openstackgerritMiles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/30237611:46
mgouldmorning EmilienM11:48
*** foxhunt72 is now known as foxhunt_away11:50
openstackgerritMerged openstack/puppet-neutron: Pass firewall_driver to ML2 plugin conf file  https://review.openstack.org/30044211:51
EmilienMdmburmistrov: sure, I'll look today11:52
*** moravec has quit IRC11:53
*** xingchao has joined #puppet-openstack11:56
*** jhesketh has joined #puppet-openstack12:00
*** ChanServ changes topic to "Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet"12:00
*** xingchao has quit IRC12:01
openstackgerritMerged openstack/puppet-ironic: Revert "Allow chainloading of Inspector ramdisk over UEFI"  https://review.openstack.org/30234812:03
*** trown|outtypewww is now known as trown12:04
*** jhesketh has quit IRC12:07
*** jhesketh has joined #puppet-openstack12:07
openstackgerritSergey Kolekonov proposed openstack/puppet-neutron: Pass firewall_driver to ML2 plugin conf file  https://review.openstack.org/30270312:07
*** moravec has joined #puppet-openstack12:13
*** moravec has quit IRC12:16
*** ducttape_ has joined #puppet-openstack12:18
*** dprince has joined #puppet-openstack12:19
dmburmistrovEmilienM, ok, thank you!12:20
openstackgerritMerged openstack/puppet-neutron: drop useless releasenote  https://review.openstack.org/30140712:23
*** skolekonov has joined #puppet-openstack12:25
*** alee has quit IRC12:26
degorenkoEmilienM, hey :) welcome back!12:26
EmilienMdegorenko: thanks!12:26
*** cdelatte has quit IRC12:30
*** ducttape_ has quit IRC12:32
*** chandankumar has quit IRC12:32
*** morazi has joined #puppet-openstack12:32
*** jpena is now known as jpena|lunch12:35
*** amoralej is now known as amoralej|lunch12:35
*** foxhunt_away is now known as foxhunt7212:37
*** jayg|g0n3 is now known as jayg12:41
*** social_ has joined #puppet-openstack12:44
*** social_ has quit IRC12:45
*** Matum has joined #puppet-openstack12:46
*** cdelatte has joined #puppet-openstack12:47
mjblackEmilienM: I might have a solution to the cinder wsgi problem12:49
EmilienMmjblack: which one?12:52
mjblackthe one where its missing the application entry point12:52
EmilienMmjblack: feel free to use my patch and push over12:53
openstackgerritSergey Kolekonov proposed openstack/puppet-openstack-integration: Set correct firewall_driver for Neutron ML2  https://review.openstack.org/30053812:53
mjblackyeah I'm about to test it out and see if it will work12:53
*** v1k0d3n has joined #puppet-openstack12:53
*** delattec has joined #puppet-openstack12:53
mjblackits a shame that the change that is causing the problem wasnt put into the mitaka release notes12:54
openstackgerritDmitry Burmistrov proposed openstack/puppet-keystone: Set oslo options in keystone module through puppet-oslo  https://review.openstack.org/30273812:55
*** cdelatte has quit IRC12:56
*** richm has joined #puppet-openstack12:56
openstackgerritDmitry Burmistrov proposed openstack/puppet-keystone: Set oslo options in keystone module through puppet-oslo  https://review.openstack.org/30273813:01
*** xarses has quit IRC13:10
*** amoralej|lunch is now known as amoralej13:12
*** xingchao has joined #puppet-openstack13:12
*** xarses has joined #puppet-openstack13:15
*** xarses has quit IRC13:15
*** xarses has joined #puppet-openstack13:15
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: Expose an option to enable/disable cleaning  https://review.openstack.org/30274713:16
*** ducttape_ has joined #puppet-openstack13:18
*** ducttape_ has quit IRC13:18
*** ducttape_ has joined #puppet-openstack13:18
*** moravec has joined #puppet-openstack13:20
*** ducttape_ has quit IRC13:31
openstackgerritMerged openstack/puppet-magnum: Update typos  https://review.openstack.org/26318113:34
*** moravec1 has joined #puppet-openstack13:39
*** xingchao has quit IRC13:39
*** xingchao has joined #puppet-openstack13:39
*** ducttape_ has joined #puppet-openstack13:40
*** chandankumar has joined #puppet-openstack13:41
*** moravec has quit IRC13:41
*** jpena|lunch is now known as jpena13:41
mgouldhi everyone13:41
mgouldI'm trying to run the unit tests for puppet-ironic13:41
mgouldis https://wiki.openstack.org/wiki/Puppet/Unit_testing the right instructions?13:42
EmilienMdegorenko: sounds like something is broken in Murano, when trying to run tempest: http://logs.openstack.org/36/250436/30/check/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-trusty/cbde3db/console.html#_2016-04-05_14_32_42_79913:43
EmilienMmgould: yes, you're on the right page13:43
*** jmikolajczak has quit IRC13:43
mgouldOK, thanks13:43
mjblackEmilienM: you got a mitaka on centos 7 handy?13:43
mgouldI get the error "rake is not part of the bundle. Add it to Gemfile."13:44
EmilienMmjblack: sudo yum -y install ruby-devel rubygems13:44
mjblackEmilienM: I think I found the wsgi script, I just wanted to verify on centos its in the correct spot13:45
openstackgerritMatthew J Black proposed openstack/puppet-cinder: Implement Cinder API WSGI support  https://review.openstack.org/28796413:46
degorenkoEmilienM, nothing changed from last week, i already mentioned, that murano has missed package. It was fixed in: https://review.openstack.org/#/c/297647/113:46
EmilienMdegorenko: cool!13:46
degorenkojust waiting for new package13:46
EmilienMdegorenko: indeed13:47
openstackgerritSergey Kolekonov proposed openstack/puppet-nova: Do not use network_api_class option  https://review.openstack.org/29301013:52
mgouldOK, tests running: I had to delete Gemfile.lock and run `bundle install` again13:55
mwhahahaI see the polar bears were unsuccessful in consuming EmilienM13:58
dmsimardlol, polar bears are much, much more up north :p13:59
mwhahaha:D14:00
EmilienMnorth coast is so beautiful14:00
EmilienMI wanted to stay there14:00
EmilienMbut there is no internet :-P14:01
mgouldEmilienM, where were you?14:01
EmilienMmgould: somewhere around https://goo.gl/maps/U1nspNdM43q14:03
mgouldawesome!14:04
mgouldlooks beautiful14:05
openstackgerritMiles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/30239014:05
openstackgerritMiles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/30237614:05
EmilienMmgould: that is the place, if you want to observe whales14:05
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Implement a script to generate Puppetfile  https://review.openstack.org/30069614:06
EmilienMomg _ody got puppet4 jobs working14:08
openstackgerritMerged openstack/puppet-cinder: Fix typo in README  https://review.openstack.org/30227714:08
openstackgerritMerged openstack/puppet-glance: Release notes for glance_image updates  https://review.openstack.org/30239114:08
openstackgerritMerged openstack/puppet-glance: Add ability to set properties with glance_image  https://review.openstack.org/30170614:08
degorenkomwhahaha, can you +A this one? https://review.openstack.org/30219714:08
*** xingchao_ has joined #puppet-openstack14:10
mwhahahayup14:10
*** xingchao has quit IRC14:11
EmilienMnow, we have Puppet OpenStack modules working on Puppet4, I'm wondering if we should switch puppet-openstack-integration to gate on puppet414:11
EmilienManother solution is to keep puppet3 by default, and have periodic jobs for puppet414:11
openstackgerritMerged openstack/puppet-glance: Switch glance to os_service_default facts  https://review.openstack.org/29982214:11
EmilienMboth together might be too much for openstack infra14:11
openstackgerritMerged openstack/puppet-glance: Implement ability to pass CA bundle certificate for vCenter server  https://review.openstack.org/30246114:11
EmilienMpabelanger: what do you think?^14:12
openstackgerritMerged openstack/puppet-gnocchi: Switch gnocchi to rspec-puppet-facts  https://review.openstack.org/29852814:12
degorenkoEmilienM, as i know we have a few modules, that we have problem with Puppet4, don't we?14:12
mwhahahathought swift was not puppet4 compatible14:12
EmilienMdegorenko: it seems like puppet4 jobs are green14:12
degorenkoyeah, at least swift14:12
degorenkoEmilienM, we can check this only running test jobs for all modules14:13
EmilienMmwhahaha, degorenko: see https://review.openstack.org/#/c/296557/14:13
pabelangerEmilienM: should be fine. Just means longer wait time for your patches to land14:13
EmilienMpabelanger, mwhahaha: all went fine on the JJB migration?14:13
pabelangerit went14:13
EmilienMpabelanger: I was afk 2 days14:13
EmilienMexcellent, thanks14:13
mwhahahafine being relative ;)14:13
pabelangerwe lost access to nodepool.o.o yesterday, which blocked puppet runs for about 3.5 hours14:14
pabelangeronce puppet started running again, puppet-jobs eventually started working again14:14
pabelanger2 issues found14:14
EmilienMpabelanger: do we have more bits to migrate?14:15
pabelangerpython-requests is installed by default now (from pip) in centos-7, so we had to remove that. And bindep needed to be run first before calling puppet-prepare-node, because you remove RDO repos from the slave14:15
pabelangerand bindep depends on RDO for centos-714:15
pabelangerEmilienM: no, puppet jobs are on ubuntu-trusty and centos-7 dibs now14:16
*** twm2016 has joined #puppet-openstack14:16
pabelangergood until we turn up ubuntu-xenial imagtes14:16
pabelangerimages*14:16
pabelangerwell, they do work, but we're asking projects to only use them in experimental for now14:16
EmilienMpabelanger, mwhahaha: we might want to start testing our modules on next LTS to be ready14:18
EmilienMwe can work on it during newton14:18
*** ToMiles has quit IRC14:20
*** baker has joined #puppet-openstack14:32
*** psanchez has quit IRC14:33
*** baker has quit IRC14:36
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: WIP: multi-domain testing with ldap backend.  https://review.openstack.org/29637014:36
*** baker has joined #puppet-openstack14:40
*** mdorman has joined #puppet-openstack14:43
openstackgerritMerged openstack/puppet-keystone: Make the SSL apache module happens at correct time  https://review.openstack.org/30192414:45
openstackgerritMerged openstack/puppet-swift: Keystone endpoints should go before the service  https://review.openstack.org/30197114:46
*** moravec1 has quit IRC14:46
*** saneax is now known as saneax_AFK14:50
*** dhollinger has joined #puppet-openstack14:51
mgoulddegorenko, mwhahaha: do I need the unless clauses in https://review.openstack.org/#/c/302376/ or not?14:53
mwhahahayou do14:53
degorenkoyou not14:53
mgouldhahaha14:53
degorenko:D14:53
* mgould doesn't understand the idempotence argument14:53
mwhahahawithout it the exec runs every time the manifest is processed14:53
mgould"run this command, unless you run this command successfully"14:53
mwhahahawhich makes it non-idempotent14:53
mgouldyeah, but to test if you need to run it you run it anyway...14:54
mwhahahaso by adding the unless, it skips the check if the file exists14:54
mwhahahawe only want those execs as fail tests14:54
mwhahahaother than that we don't want them to run14:54
mwhahahatechincally you could just switch command to exit 1 and just test with the unless14:54
mwhahahabecause the condition will trigger then14:55
degorenkooh, yes, right, i forgot about idempotency14:55
* mwhahaha slaps degorenko14:55
degorenkomgould, yes, you need :)14:55
mwhahaha:D14:55
degorenkoD:14:55
twm2016@mwhahaha: I almost suggested that14:55
mgouldOK, sounds like we should make it "fail unless"14:55
mgouldwhich I think makes the intention clearer14:55
mwhahahayea we're using an exec to throw a fatal only if a file doesn't exist14:56
degorenkowe should run exec unless :)14:56
twm2016yeah that makes it clear14:56
mgouldwhat happens if we don't have the unless?14:56
mwhahahait'll run all every run14:56
mgouldfirst run: test -f $file14:56
degorenkoit will not pass idempotency test14:56
mwhahahayou're getting into the weirdness of puppet14:56
mgouldOK, please help me understand14:57
mgouldwith the unless clause, on the second run14:57
mwhahahaexec { 'file missing': path=>"file", command => "exit 1", unless => "test -f file" }14:57
mwhahahabasically the unless prevents the thing from running14:58
degorenkoexec will not be run when file exists at all14:58
mwhahahaso puppet just skips it14:58
mgouldOK14:58
mgouldbut if the body of the exec is "test -f" and there's no unless clause, it runs test -f every time, and fails if the file isn't present14:58
mwhahahaif you just have exec { 'file test': path => "file", command => "test -f file" }, it'll run every time the manifest is processed14:58
mwhahahawhich in puppet terms is non-idempotent14:58
mwhahahayou should only want to have something run one time14:59
mgouldnb the path argument is actually the executable PATH14:59
mgouldaaaaah14:59
*** ducttape_ has quit IRC14:59
mwhahahayea my brain isn't working enough yet, but i'm just throwing out some basic examples14:59
mgouldso Puppet doesn't care that the exec's body is idempotent, it only cares that you ran it twice?14:59
mwhahahayea15:00
mgouldgotcha15:00
mwhahahasince the exec gets run twice it considers it as non-idempotent and shows up as a change every time15:00
mgouldcool15:00
mwhahahaeven tho it's a noop15:00
mgouldI suppose checking the body of the exec for idempotence would be Hard15:00
mgoulduncomputable? probably15:00
mwhahahayea15:00
mwhahahathats where the unless comes in15:00
mwhahahait's you telling puppet when the change has been done so don't do the exec again15:01
mgouldright, OK15:01
mgouldI'll do fail { unless }, I think, for clarity15:01
*** xingchao_ has quit IRC15:02
*** dgurtner has quit IRC15:02
*** dgurtner has joined #puppet-openstack15:04
openstackgerritMerged openstack/puppet-tempest: Update spec tests for Tempest  https://review.openstack.org/30219715:05
*** 7JTAALB9C has joined #puppet-openstack15:06
*** baker has quit IRC15:07
*** guimaluf has joined #puppet-openstack15:09
mgouldnope, fail is a function15:09
mwhahahayea you can't use fail15:10
mwhahahayou still have to use exec15:10
mwhahaha:D15:10
*** baker has joined #puppet-openstack15:10
*** Guest74036 has quit IRC15:10
mwhahahanotice is the only one that you could do that with15:10
mwhahahabut no one reads those15:10
* mgould nods15:11
openstackgerritMiles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/30239015:11
openstackgerritMiles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/30237615:11
mgouldyes, gerrit, I do want to upload multiple commits15:11
*** yottatsa has joined #puppet-openstack15:11
mgouldGit is really really good at sequences of commits, stop trying to discourage me from using one of its most powerful features15:11
mwhahahait's because they are dependencies so it has to update the 2nd one since you're working on the first one15:12
*** moravec has joined #puppet-openstack15:12
EmilienMdmburmistrov, degorenko: hey about https://review.openstack.org/#/c/301079/4/manifests/policy.pp15:13
EmilienMI don't have strong opinion15:13
EmilienMit just adds conditionnals in the code15:13
dmburmistrovEmilienM, <mwhahaha> suggested to be permissive to not brake other modules (and other existing code)15:15
EmilienMok15:15
EmilienMapproved15:15
mwhahahawe generally have been allowing the string values so adding the array restrictions into oslo may break other things15:16
mwhahahaif we want to switch to an array, i'd recommend a deprecation-like messaging in the modules that switch to oslo15:16
mwhahahathen we can switch to arrays in a later release15:16
dmburmistrovsounds reasonable - which  way should I rework my patches today?15:17
mwhahahaany2array in oslo15:17
dmburmistrovgood, ok15:17
mwhahahapull the logic out of the modules that switch to oslo and just pass the values15:17
mwhahahait'll make life easier all around :)15:18
EmilienMdmburmistrov: should I drop +A ?15:18
mwhahahaonce they all get switched we could just add the warning logic into oslo if we want to add those restrictions (but i doubt it)15:18
openstackgerritIvan Berezovskiy proposed openstack/puppet-nova: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/29230915:19
openstackgerritIvan Berezovskiy proposed openstack/puppet-nova: Configure oslo related sections using puppet-oslo module  https://review.openstack.org/30283915:19
*** xingchao has joined #puppet-openstack15:20
EmilienMdmburmistrov: I -1 it to avoid merge15:20
dmburmistrovEmilienM, yeah, thanks!15:20
*** leanderthal is now known as leanderthal|afk15:23
openstackgerritIvan Berezovskiy proposed openstack/puppet-nova: Configure oslo related sections using puppet-oslo module  https://review.openstack.org/30283915:23
mgouldmwhahaha, yeah, I know why it's doing it15:23
mgouldI just think it's stupid :-)15:23
mwhahahai find it more annoying when it's a chain of like 6 of them :D15:24
mgouldmwhahaha, degorenko: remind me, what was wrong with ensure { file, noop }?15:25
dtantsura couple of times it actually saved me from submitting a couple of wrong commits15:25
dtantsurso it's not so stupid (or maybe I am stupid? :D)15:26
mwhahahait won't fail15:26
degorenko^ yes15:26
mwhahahafile would try and create it15:26
mwhahahaour goal is for failure :D15:26
mgouldOK, cool15:26
EmilienMcrinkle: have you seen https://bugs.launchpad.net/puppet-keystone/+bug/1566190 ?15:26
openstackLaunchpad bug 1566190 in puppet-keystone "'disable_admin_token_auth' should also handle admin_token in the config" [Undecided,New]15:26
mgouldOK, I think I've addressed all your comments; I'm gonna disappear for a bit to do some DHCP testing15:27
*** paramite has quit IRC15:27
*** mgould has quit IRC15:27
mwhahahaninja vanish15:28
EmilienMskolekonov, mjblack: for https://review.openstack.org/#/c/301797/ - we need to engage efforts on puppet-octavia. I initiated the module, we have a basic structure ready. Please let me know if you're interested to continue the work a bit15:32
mjblackEmilienM: that is the plugin for octavia15:34
mjblackoctavia is its own self contained service15:34
EmilienMmjblack: ok so we need to configure plugin in puppet-neutron and service in puppet-octavia?15:34
mjblackyup15:35
EmilienMkk15:35
EmilienMmjblack: still add tests :P15:35
mjblackI'm working on it :D15:35
mjblackI just wanted to make sure the layout was ok15:35
EmilienMit looks good.15:35
mjblackEmilienM: one thing to keep in mind with lbaas, the neutron::agents::lbaas is *only* for haproxy15:35
openstackgerritPradeep Kilambi proposed openstack/puppet-gnocchi: Add sync_db param to gnocchi api class  https://review.openstack.org/30060415:36
mjblackor rather15:36
mjblackthe only plugin that work with the lbaas agent that I know of is haproxy15:37
skolekonovmjblack, is it possible to fully configure lbaasv2? AFAIK it requires a specific image to work15:37
*** ntaylor1781 has joined #puppet-openstack15:38
mjblackskolekonov: right now, the changes for lbaasv2 service in neutron is still in progress15:38
mjblackthe lbaas agent that puppet-neutron handles is only v1 but it can be updated to do v2 as well15:39
crinkleEmilienM: i did but forgot to respond, i don't believe it's a bug15:40
EmilienMcrinkle: yeah, I commented15:41
EmilienMdoes it make sense?15:41
crinkleEmilienM: my understanding of auth_token is disabling it in the pipeline will render having it set in keystone.conf useless so it doesn't need to be removed15:41
EmilienMcrinkle: exactly15:42
crinkleEmilienM: and puppet can't really handle setting it, using it, then unsetting it in one puppet run, so doing it like this is easiest15:42
EmilienMexactly15:43
openstackgerritDmitry Burmistrov proposed openstack/puppet-oslo: Update oslo::policy define  https://review.openstack.org/30107915:43
EmilienMcrinkle: invalid bug.15:43
*** aimon has joined #puppet-openstack15:44
*** dfisher has joined #puppet-openstack15:44
*** afranc has joined #puppet-openstack15:44
*** foxhunt72 is now known as foxhunt_away15:44
mjblackEmilienM: looks like scenarios are failing because of keystone15:45
EmilienMdegorenko: do you have news on murano packaging? all deps are solved?15:45
mjblackseeing "Error: Could not prefetch cinder_type provider 'openstack': Could not authenticate"15:46
degorenkoEmilienM, no, i don't. For tempest we just waiting for rc315:46
EmilienMdegorenko: for RDO I mean15:46
openstackgerritMerged openstack/puppet-neutron: Set ini/openstack libs to pass file_path properly  https://review.openstack.org/30240815:46
degorenkoEmilienM, ah, also don't know. Going to ask15:46
*** xingchao has quit IRC15:47
*** jaosorior has quit IRC15:47
openstackgerritMatthew J Black proposed openstack/puppet-cinder: Implement Cinder API WSGI support  https://review.openstack.org/28796415:47
*** jaosorior has joined #puppet-openstack15:47
*** xingchao has joined #puppet-openstack15:52
*** derekh has quit IRC15:53
*** 7JTAALB9C has quit IRC15:54
*** moravec has quit IRC15:55
*** ducttape_ has joined #puppet-openstack15:56
*** moravec1 has joined #puppet-openstack16:00
*** xarses has quit IRC16:01
*** xarses has joined #puppet-openstack16:02
openstackgerritMerged openstack/puppet-oslo: Add oslo::messaging::default define  https://review.openstack.org/30266416:03
openstackgerritMerged openstack/puppet-nova: Autorequire nova security group  https://review.openstack.org/30153416:03
openstackgerritMerged openstack/puppet-vswitch: add missing dev keyword to ip link command  https://review.openstack.org/30117616:03
*** vinsh_ is now known as Vinsh16:04
*** yottatsa has quit IRC16:04
*** angdraug has joined #puppet-openstack16:04
openstackgerritPradeep Kilambi proposed openstack/puppet-aodh: Add sync_db param to aodh api class  https://review.openstack.org/30292116:06
openstackgerritDenis Egorenko proposed openstack/puppet-tempest: Correct comments for config class  https://review.openstack.org/30182616:10
*** dtantsur is now known as dtantsur|afk16:14
*** baker has quit IRC16:15
*** baker has joined #puppet-openstack16:17
openstackgerritDmitry Burmistrov proposed openstack/puppet-oslo: Process list values inside oslo module  https://review.openstack.org/30227116:17
twm2016I know this review has gotten a lot of attention lately https://review.openstack.org/#/c/302376/ but I think instead of unless, we should use creates.16:23
twm2016Because of this page i found https://review.openstack.org/#/c/302376/16:23
twm2016sorry this one: http://www.puppetcookbook.com/posts/run-exec-if-file-absent.html16:23
openstackgerritDmitry Burmistrov proposed openstack/puppet-oslo: Process list values inside oslo module  https://review.openstack.org/30227116:23
openstackgerritDmitry Burmistrov proposed openstack/puppet-oslo: Update oslo::policy define  https://review.openstack.org/30107916:23
*** trown is now known as trown|lunch16:24
*** guimaluf has quit IRC16:28
dmburmistrovEmilienM, mwhahaha - jfyi: I reworked my patches (local test succeeded), waiting for zuul; urls: https://review.openstack.org/#/c/301079/ & https://review.openstack.org/#/c/302271/16:30
EmilienMdmburmistrov: excellent16:30
*** xingchao has quit IRC16:34
*** moravec has joined #puppet-openstack16:34
*** moravec2 has joined #puppet-openstack16:36
*** haplo37 has joined #puppet-openstack16:37
*** dgurtner has quit IRC16:37
*** moravec1 has quit IRC16:37
*** moravec has quit IRC16:38
mjblackEmilienM,mfisch,degorenko: I'm seeing the integration tests failing and it looks like keystone16:43
EmilienMmjblack: no it's httpd failing to start16:43
mjblackwell close enough :D16:43
EmilienMI haven't read logs but I'm sure that's it :)16:43
mjblackthat might be the case though16:43
mjblackwhere can I get the logs?16:44
EmilienMmjblack: look httpd logs when you have it, I'm sure that's it16:44
EmilienMwell, zuul has not finished to execute all jobs on https://review.openstack.org/#/c/287964/16:44
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-openstacklib: Add the possibility to execute without retry.  https://review.openstack.org/30244216:44
EmilienMmjblack: but otherwise they'll be on http://logs.openstack.org/64/287964/11/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/46fb35d/logs/apache/16:44
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: WIP: Remove user/role prefetch to support multi-domain.  https://review.openstack.org/29930116:47
openstackgerritMerged openstack/puppet-ironic: Expose an option to enable/disable cleaning  https://review.openstack.org/30274716:48
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: WIP: multi-domain testing with ldap backend.  https://review.openstack.org/29637016:52
*** rcernin has quit IRC16:52
*** moravec2 has quit IRC16:53
*** moravec has joined #puppet-openstack16:53
openstackgerritDan Prince proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/29343616:56
*** angdraug has quit IRC16:58
*** dprince has quit IRC16:58
*** dprince has joined #puppet-openstack16:59
*** jistr has quit IRC17:01
*** amoralej has quit IRC17:03
*** jpena is now known as jpena|off17:03
*** rmart04 has quit IRC17:07
mwhahahai think the centos 7 beaker test is broken for barbican17:09
EmilienMmwhahaha: how that?17:11
mwhahahacause it keeps failing17:11
mwhahahai need to look into it further but i don't think i've seen it pass in the last 2 days17:11
dfisherhas anybody seen _ody?  I've been unable to maintain a constant IRC connection due to being at a conference17:12
_odyo.O17:18
*** mgould has joined #puppet-openstack17:19
*** moravec has quit IRC17:19
*** xarses has quit IRC17:20
dfisherwhy hello there :)17:20
dfisher_ody: are there plans to cut the 8.0.0 tarballs on the forge?17:20
_odydfisher: Yes.  I am holding off for packages to be released.17:21
dfisherweren't they released just today?17:21
_odyI hadn't heard but if yes then yes I'll start cutting releases.17:22
_odyWell looks like it has according to openstack.org.17:22
dfisherdhellman sent out the email officially announcing Mitaka17:22
*** jmikolajczak has joined #puppet-openstack17:22
dfishermany thanks for cutting the 8.0.0 releases (preemptive but I'll probably vanish from IRC when this conference session ends)17:23
*** gchamoul has quit IRC17:24
_odyok.  Yeah.  I'll get it down by EOW.17:24
dfisher\o/17:24
*** trown|lunch is now known as trown17:24
*** moravec has joined #puppet-openstack17:25
*** iberezovskiy is now known as iberezovskiy_17:26
*** rcernin has joined #puppet-openstack17:33
*** jmikolajczak has quit IRC17:33
*** moravec has quit IRC17:35
*** dfisher has quit IRC17:39
*** degorenko is now known as _degorenko|afk17:40
*** richm has quit IRC17:49
*** mgould has quit IRC17:50
*** ducttape_ has quit IRC17:51
*** rmart04 has joined #puppet-openstack17:52
*** dfisher has joined #puppet-openstack17:53
*** ducttape_ has joined #puppet-openstack17:53
*** xarses has joined #puppet-openstack17:56
*** ducttape_ has quit IRC17:57
*** skolekonov has quit IRC17:59
*** gchamoul has joined #puppet-openstack17:59
*** mgould has joined #puppet-openstack18:00
*** dprince has quit IRC18:00
*** dprince has joined #puppet-openstack18:00
*** richm has joined #puppet-openstack18:05
*** twm2016_ has joined #puppet-openstack18:06
*** twm2016 has quit IRC18:07
*** haplo37 has quit IRC18:08
mjblackEmilienM,mfisch: whats your opinion of having lbaas v1 and v2 agents installed on the same host? This is really only affects haproxy, no other load balancer that I can see18:08
*** moravec has joined #puppet-openstack18:08
mjblackI want to say that it should be one or the other18:09
*** chandankumar has quit IRC18:09
*** richm has quit IRC18:10
*** foxhunt_away is now known as foxhunt7218:11
*** dprince has quit IRC18:13
*** dprince has joined #puppet-openstack18:15
*** v1k0d3n has quit IRC18:19
*** richm has joined #puppet-openstack18:24
*** mgould has quit IRC18:32
*** twm2016_ has quit IRC18:33
*** dprince has quit IRC18:42
*** ducttape_ has joined #puppet-openstack18:43
*** dprince has joined #puppet-openstack18:43
*** angdraug has joined #puppet-openstack18:44
*** delattec has quit IRC18:49
*** dfisher has quit IRC18:50
*** rstarmer has joined #puppet-openstack18:53
*** chem has quit IRC18:56
*** jmikolajczak has joined #puppet-openstack18:57
*** cdelatte has joined #puppet-openstack19:03
*** jmikolajczak has quit IRC19:04
mwhahahamjblack: i thought there was a mailing list post about that you can't have both19:09
mwhahahaor was that for the api and not the agents19:09
mjblackmwhahaha: sure but that doesnt stop someone from installing both ;)19:12
mwhahahayup19:12
mjblackjust wondering if I need to add in something to block it...19:13
mwhahahayou could probably do a defined check to throw a warning, but we shouldn't block it19:13
EmilienMit sounds complicated19:14
mjblackmwhahaha: I was going to reuse the lbaas agent and just have a flag for v219:15
mjblacksince they both use the same config file19:15
mwhahahahmm then maybe we should throw something to preven it19:16
mwhahahaprevent rather19:16
mjblackI was going to have a $enable_lbaasv219:16
mjblackwhich would turn off v1 service and turn on v2 service19:16
mjblackredhat the v1 and v2 agent is bundled together but on ubuntu it is two packages19:16
openstackgerritMerged openstack/puppet-tempest: Correct comments for config class  https://review.openstack.org/30182619:17
mwhahahaso just reusing the lbaas class and adding a v2 flag?19:17
mjblackyeah19:18
mwhahahaperhaps add both19:18
mjblackit would but only one service would be enabled19:18
mwhahahai'd add an enable_v1 and enable_v2 flag and perhaps throw a warning if both are enabled19:18
mwhahahawith v1 currently enabled by default19:19
mjblackthat might be better but I think instead of a warning a fail would be better19:19
mjblackonce the agent stands up, it'll register with neutron so if they dont want v1, for example, and later turn it off, it'll show up as a dead agent19:20
mwhahahasure fail would be ok as well19:20
mwhahahathe point being to add it with those flags and add some kind of logic around it19:20
EmilienMI'm working on puppet-octavia right no19:20
EmilienMnow*19:20
mjblackEmilienM: does it create the init/upstart/systemd scripts? :D19:21
EmilienMmjblack: no but packaging should19:21
EmilienMmjblack: why would it be doing that?19:21
mjblackcause octavia so far has no packages19:22
mjblackwell...no packages for ubuntu, the devs say to install it with pip19:22
EmilienMare you sure?19:22
mjblackyeah19:22
mjblackI saw rpm packages for it though19:22
*** dgautam has joined #puppet-openstack19:22
EmilienMah :)19:22
EmilienMRDO provides packaging19:22
mjblackhttps://github.com/openstack-packages/octavia19:22
EmilienMyeah, I confirm we have packaging already19:23
EmilienMthat's why we can start deploying it19:23
dgautamI am facing puppet/type/keystone_tenant.rb error. Any poiters to resolve this ?19:23
dgautamApr  7 12:21:48 cmbu-cl62 puppet-agent[4567]: Could not autoload puppet/type/keystone_tenant: interning empty string Apr  7 12:21:48 cmbu-cl62 puppet-agent[4567]: Could not retrieve catalog from remote server: Could not intern from text/pson: Could not autoload puppet/type/keystone_tenant: interning empty string19:23
*** dgurtner has joined #puppet-openstack19:23
*** dgurtner has quit IRC19:23
*** dgurtner has joined #puppet-openstack19:23
mjblackdgautam: what version of puppet? and what release of puppet-keystone?19:24
dgautampuppet-keystone is 6.1.0 (kilo)19:24
dgautammaster is 3.7.319:25
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer listeners  https://review.openstack.org/30257419:26
*** jaosorior has quit IRC19:27
*** jmikolajczak has joined #puppet-openstack19:29
dgautam#puppet told me, there was some known bug in puppet/type/keystone_tenant.rb.19:30
mjblackare you able to create a paste with your manifest?19:31
EmilienMmjblack: did you figure why https://review.openstack.org/#/c/287964/ fails?19:31
*** cdelatte has quit IRC19:31
mjblackEmilienM: nothing that I could figure out but it is with keystone19:32
mjblackit's before the cinder wsgi gets installed19:32
mjblackonly thing I see that probably is the culprit for it failing is this...19:33
mjblack2016-04-07 14:09:05.644 11656 WARNING keystone.common.wsgi [req-b30dec48-081d-4bf2-ad8c-f6947afb2d9a - - - - -] Authorization failed. The request you have made requires authentication. from ::119:33
EmilienMhttp://logs.openstack.org/64/287964/12/check/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/af61737/console.html#_2016-04-07_17_23_25_39419:33
openstackgerritSergey Kolekonov proposed openstack/puppet-nova: Do not use network_api_class option  https://review.openstack.org/29301019:33
EmilienMmjblack: https://review.openstack.org/#/c/287964/12/manifests/keystone/auth.pp19:33
EmilienMthat's because of that19:34
mjblackI stand corrected...I'm seeing this19:34
EmilienMI'm working on it.19:34
mjblackkeystone.token.controllers [req-b30dec48-081d-4bf2-ad8c-f6947afb2d9a - - - - -] User d65e96f34f1b4b90a99ffd614b86abe9 is unauthorized for tenant 7a782b913a644e8594ea76e9da8be79e19:34
mjblackyou want me to just quickly update it?19:34
EmilienMI'm on it19:34
mjblackok19:34
dgautamshould I paste full manifest file ?19:36
mjblackdgautam: sure19:37
dgautamhttp://paste.openstack.org/show/sYzy0PnraASjKuGQdcSG/19:38
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Implement Cinder API WSGI support  https://review.openstack.org/28796419:40
EmilienMmjblack: https://review.openstack.org/#/c/287964/12..13/manifests/keystone/auth.pp19:40
EmilienMmjblack: and https://review.openstack.org/#/c/287964/12..13/manifests/api.pp19:41
EmilienMmwhahaha: you ok with https://review.openstack.org/#/c/300696/ now?19:41
EmilienMI addressed your review19:41
mwhahahayea i think so i'll take a look in a bit19:42
*** dprince has quit IRC19:42
*** karimb has joined #puppet-openstack19:42
mjblackEmilienM: looks good to me, lets see if it passes the tempest scenario tests now19:43
mjblackdgautam: have you verified the tenants hiera data?19:44
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: deploy cinder api in WSGI with Apache  https://review.openstack.org/28809119:44
EmilienMmjblack: let's see if when enabled, it pass CI ^19:45
EmilienMoops I failed in rebase19:45
mjblackdgautam: also, I dont know about 3.7 puppet but 3.8 puppet uses hiera 1.319:46
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: deploy cinder api in WSGI with Apache  https://review.openstack.org/28809119:46
mjblackyou can reduce a lot of your manifests with hiera 1.319:46
*** karimb has quit IRC19:47
dgautammkblack: yes, i checked hieradata. http://paste.openstack.org/show/493410/19:47
dgautammjblack: surely I'll look for hiera 1.3 . do you have any reference/example about reducing manifest with hiera 1.3 ?19:52
mjblackyeah, you can do hiera data binding19:53
mjblackin the hiera you do this19:53
mjblackcinder::keystone::auth::password: "mypassword"19:53
mjblackand then you dont need to call the class with that19:54
mjblackso like in the first paste, lines 32-70, you probably can remove completely19:54
mjblackdo hiera data bindings and then do hiera_include on an array that contains the classes19:55
EmilienM_ody: weird, it's failing now on https://review.openstack.org/#/c/296557/ - it was working before19:56
mjblackdgautam: and then you can do a few lookups to create a string19:56
*** dfisher has joined #puppet-openstack19:58
dgautamgot it. surely work on that. I guess https://docs.puppet.com/hiera/3.1/complete_example.html#writing-the-data-sources should be good example.19:58
EmilienM_ody: sounds like a packaging issue: http://logs.openstack.org/57/296557/32/experimental/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/672e134/console.html#_2016-04-07_17_13_34_91119:59
EmilienMwith new centos7 images I guess19:59
EmilienMnodepool images for centos7 have been updated yesterday or so, I presume we need to add a package19:59
EmilienMpabelanger: ^20:00
dgautammjblack: any pointer for keystone_tenant.rb ?20:01
pabelangerEmilienM: have a few ways for handing it, either we update bindep-fallback.txt or start adding other-requirements.txt to projects20:02
dgautamwhat way i should debug it ?20:02
pabelangerEmilienM: but let me check why it is missing20:02
mjblackdgautam: still looking into it20:02
dgautamThanks20:03
*** jtomasek has quit IRC20:03
pabelangerEmilienM: odd, we are not running bindep for that job20:03
pabelangerEmilienM: let me check JJB20:03
EmilienMpabelanger: it's an experimental job20:04
EmilienMpabelanger: gate-puppet-openstack-integration-4-scenario00(1,2,3)-tempest-centos-720:05
*** jtomasek has joined #puppet-openstack20:05
mjblackdgautam: one thing that might be an issue20:05
mjblackscope20:05
mjblackwhat you can try to do is20:06
mjblackin the class contrail::profile::openstack::provision, have it accept two parameters20:06
mjblack$users = {} and $tenants = {}20:06
mjblackand then what you would do next is change the hiera key from openstack::keystone::tenants to contrail::profile::openstack::provision::tenants20:07
mjblackthe same would need to be done for users20:07
*** jpena|off is now known as jpena20:07
*** jpena is now known as jpena|off20:08
dgautamlet me try20:09
*** jpena|off is now known as jpena20:09
dgautammjblack: nope, same error.20:12
dgautam 40     class { '::keystone::roles::admin':20:12
dgautaminternally calls keystone_tenant ()20:12
dgautameven if I remove create_resouces(), same error comes.20:13
EmilienMrdo trunk is down now20:14
*** EmilienM changes topic to "CI status: orange (RDO trunk repos are down) - Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet"20:14
bkero:(20:15
bkeroI always wonder why it goes down and who to talk to about it.20:15
_odyEmilienM: I figured such a think might happen with all the churn on images lately and the other nodepool issues about.  Why I ran the job one more time before it got merged.20:15
EmilienM_ody: yeah, pabelanger is on it20:16
EmilienMbkero: #rdo20:16
bkeroah20:17
*** ayoung has quit IRC20:18
*** jayg is now known as jayg|g0n320:24
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Add Glance profiles  https://review.openstack.org/29607620:26
ntaylor1781So I was hoping for some insight. I have been looking at adding a purge_config option for the various modules. Glance had it in the liberty release, but it is no longer fucntional in master. It looks like this is the commit that removed it: https://review.openstack.org/#/c/211593/, specifically patch_set 9. It removed the resources decleration that made the purge_config work. That being said the option is still in the ap20:27
ntaylor1781I'm curious if this was on purpose, or if it got missed.20:27
mwhahahaprobably accidental20:29
ntaylor1781I can get in a review to add it again, or at least get it looked at. Looking at the bug report and the commit, it doesn't look like that is the intention.20:30
*** moravec has quit IRC20:30
EmilienMwhat is the use case of purging config?20:30
mwhahahanot having extra stuff20:30
EmilienMoki20:30
EmilienMworks for me20:31
mwhahahaso only have what you declare20:31
*** johnmilton has quit IRC20:32
ntaylor1781Honestly that is a big one for me, with the system I manage.20:32
openstackgerritJerzy Mikolajczak proposed openstack/puppet-neutron: Remove deprecated options from agents  https://review.openstack.org/30110620:33
ntaylor1781So should I make a bug report for this one, or should I just do the review.20:33
EmilienMntaylor1781: send the code20:33
*** jpena is now known as jpena|off20:33
*** gfidente has quit IRC20:35
*** ducttape_ has quit IRC20:37
*** ericksonsantos has joined #puppet-openstack20:41
*** cdelatte has joined #puppet-openstack20:41
*** ducttape_ has joined #puppet-openstack20:42
openstackgerritNathan Taylor proposed openstack/puppet-glance: Fix the purge_config option for api and registry  https://review.openstack.org/30306420:43
*** aimon has quit IRC20:43
*** dgurtner has quit IRC20:46
*** mdorman has quit IRC20:47
*** dgurtner has joined #puppet-openstack20:48
*** dgurtner has quit IRC20:48
*** dgurtner has joined #puppet-openstack20:48
ntaylor1781EmilienM here is the review https://review.openstack.org/#/c/303064/, The options were still in there and the specs still had settings for the options, so the only thing I needed to add was the resources decleration back.20:51
EmilienMntaylor1781: excellent. Will review it once CI is green20:51
ntaylor1781Thanks!20:52
EmilienMmwhahaha: I found something very weird20:53
EmilienMhttp://logs.openstack.org/26/301826/2/gate/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/3b88dcc/console.html#_2016-04-07_18_48_46_64320:53
EmilienMit is non-fatal, job is passing20:53
* mwhahaha patiently waits for chrome to get on with it20:54
EmilienMactually, image create works fine20:54
EmilienMonly list fails20:54
mwhahahaso i think i noticed something similar when i was reworking the provider20:55
mwhahaha(still waiting for the page)20:55
*** dfisher has quit IRC20:55
mwhahaha2016-04-07 18:48:46.644 | openstack image list: error: unrecognized arguments:'. Retrying for 58 more seconds20:55
EmilienMyes20:55
mwhahahaso something is bad in the cmd line20:55
mwhahahait's that trailing space20:56
mwhahaha2016-04-07 18:49:13.251 | Debug: Executing '/usr/bin/openstack image list --quiet --format csv '20:56
mwhahahaopenstack client hates it20:56
mwhahahadon't ask me why20:56
mwhahahaor it's a null or something20:57
mwhahahabut i saw the same thing when i was messing with20:58
mwhahahawith it20:58
*** dgautam has quit IRC20:58
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: deploy & test neutron lbaasv1 service  https://review.openstack.org/27229720:59
EmilienMmjblack: ^20:59
EmilienMmjblack: just to test what is working now20:59
*** dhollinger has quit IRC21:00
mjblackEmilienM: yeah I'll need to update that too later to add in the v2 tests21:01
EmilienMmjblack: if you got v2 working, you can take over my patch21:02
mjblackEmilienM: I think your v1 should be its own patch since we should be testing v1 to begin with21:02
EmilienMcool²21:02
openstackgerritMerged openstack/puppet-gnocchi: Add sync_db param to gnocchi api class  https://review.openstack.org/30060421:03
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Implement Cinder API WSGI support  https://review.openstack.org/28796421:04
*** trown is now known as trown|outtypewww21:06
*** jmikolajczak has quit IRC21:08
*** dfisher has joined #puppet-openstack21:11
*** rstarmer has quit IRC21:13
*** dhollinger has joined #puppet-openstack21:14
*** foxhunt72 is now known as foxhunt_away21:15
openstackgerritMerged openstack/puppet-neutron: Require the neutron package be installed first  https://review.openstack.org/30184521:15
*** dfisher has quit IRC21:21
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: deploy cinder api in WSGI with Apache  https://review.openstack.org/28809121:22
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001/centos7: deploy Ceph on IPv6  https://review.openstack.org/30308621:28
*** dgurtner has quit IRC21:30
*** dgurtner has joined #puppet-openstack21:31
*** dgurtner has quit IRC21:31
*** dgurtner has joined #puppet-openstack21:31
*** ayoung has joined #puppet-openstack21:32
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001/centos7: deploy Ceph on IPv6  https://review.openstack.org/30308621:32
*** ducttape_ has quit IRC21:33
*** rstarmer has joined #puppet-openstack21:38
*** dhollinger has quit IRC21:55
*** ducttape_ has joined #puppet-openstack22:00
*** ayoung has quit IRC22:01
*** baker has quit IRC22:06
*** baker has joined #puppet-openstack22:06
*** baker has quit IRC22:06
*** ducttape_ has quit IRC22:11
*** ntpttr_ has joined #puppet-openstack22:14
*** ducttape_ has joined #puppet-openstack22:16
*** dgurtner has quit IRC22:17
openstackgerritMark Kirkwood proposed openstack/puppet-swift: Allow a replication network to be specified  https://review.openstack.org/30248322:23
*** ayoung has joined #puppet-openstack22:26
*** rcernin has quit IRC22:29
*** ducttape_ has quit IRC22:31
*** angdraug has quit IRC22:32
*** rmart04 has quit IRC22:32
*** ntpttr_ has quit IRC22:35
*** ntpttr_ has joined #puppet-openstack22:40
*** moravec has joined #puppet-openstack23:00
*** rstarmer has quit IRC23:06
openstackgerritMerged openstack/puppet-oslo: Update oslo::policy define  https://review.openstack.org/30107923:09
*** moravec has quit IRC23:12
openstackgerritMerged openstack/puppet-aodh: Add sync_db param to aodh api class  https://review.openstack.org/30292123:14
*** ntpttr_ has quit IRC23:22
*** ayoung has quit IRC23:26
*** markvoelker has quit IRC23:29
*** ducttape_ has joined #puppet-openstack23:32
*** ducttape_ has quit IRC23:37
*** xarses has quit IRC23:45
*** moravec has joined #puppet-openstack23:51
*** markvoelker has joined #puppet-openstack23:59
« Wednesday, 2016-04-06 Index Friday, 2016-04-08 »

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!