Wednesday, 2016-07-27

« Tuesday, 2016-07-26 Index Thursday, 2016-07-28 »
openstackgerritBrent Eagles proposed openstack/puppet-octavia: Adding housekeeping.pp  https://review.openstack.org/34752700:26
*** aernhart_ has joined #puppet-openstack00:37
*** bklei has quit IRC00:39
*** aernhart has quit IRC00:39
*** bklei has joined #puppet-openstack00:39
*** bklei has quit IRC00:44
*** vinsh has joined #puppet-openstack00:50
*** ducttape_ has quit IRC00:54
*** aernhart_ has quit IRC00:55
*** moravec has joined #puppet-openstack00:56
*** ducttape_ has joined #puppet-openstack00:56
*** vinsh has quit IRC01:02
*** ChanServ sets mode: -oo iberezovskiy|off _degorenko|afk01:12
*** ducttape_ has quit IRC01:17
*** xarses has joined #puppet-openstack01:20
*** openstackgerrit has quit IRC01:33
*** openstackgerrit has joined #puppet-openstack01:34
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-nova: Fix typo in tests  https://review.openstack.org/34759802:29
*** iurygregory_ has joined #puppet-openstack02:29
*** severion has quit IRC02:32
*** v1k0d3n has joined #puppet-openstack02:41
*** ducttape_ has joined #puppet-openstack02:48
*** ducttape_ has quit IRC02:53
*** johnmilton has quit IRC03:04
*** johnmilton has joined #puppet-openstack03:20
*** ducttape_ has joined #puppet-openstack03:31
*** ducttape_ has quit IRC03:36
*** v1k0d3n has quit IRC03:57
*** rwsu has quit IRC04:18
*** richm has quit IRC04:27
*** gfidente|afk has quit IRC04:31
*** rwsu has joined #puppet-openstack04:32
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-barbican: WIP - Move barbican to authtoken  https://review.openstack.org/34710504:35
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Fix typo in resource test  https://review.openstack.org/34761904:41
*** jtomasek has quit IRC04:41
*** iurygregory_ has quit IRC04:43
*** ducttape_ has joined #puppet-openstack04:57
*** ducttape_ has quit IRC05:02
*** maestropandy has joined #puppet-openstack05:21
*** beagles has quit IRC05:26
*** v1k0d3n has joined #puppet-openstack05:47
*** saneax_AFK is now known as saneax05:49
*** maestropandy has quit IRC05:54
*** v1k0d3n has quit IRC05:58
*** ducttape_ has joined #puppet-openstack05:58
*** ducttape_ has quit IRC06:02
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/34763506:11
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/34763606:11
*** Guest90133 is now known as leanderthal06:30
*** hjensas has joined #puppet-openstack06:38
*** tesseract- has joined #puppet-openstack06:44
*** rcernin has joined #puppet-openstack06:47
*** maestropandy has joined #puppet-openstack06:48
*** rhefner has quit IRC06:49
*** igorbelikov has quit IRC06:49
*** rhefner has joined #puppet-openstack06:50
*** davidmichaelkarr has quit IRC06:50
*** igorbelikov has joined #puppet-openstack06:51
*** davidmichaelkarr has joined #puppet-openstack06:52
*** ducttape_ has joined #puppet-openstack06:59
*** ducttape_ has quit IRC07:03
*** danpawlik has joined #puppet-openstack07:15
*** alee_dinner has quit IRC07:26
*** hjensas has quit IRC07:29
*** mtreinish has quit IRC07:36
*** mtreinish has joined #puppet-openstack07:36
*** foxhunt_away has quit IRC07:38
*** shardy has joined #puppet-openstack07:38
*** basilAB has quit IRC07:38
*** maestropandy has quit IRC07:38
*** basilAB has joined #puppet-openstack07:39
*** foxhunt_away has joined #puppet-openstack07:40
*** ifarkas_afk is now known as ifarkas07:41
*** atom_ has joined #puppet-openstack07:49
*** jtomasek has joined #puppet-openstack07:50
*** johnmilton has quit IRC07:52
*** ducttape_ has joined #puppet-openstack08:00
*** derekh has joined #puppet-openstack08:03
*** ducttape_ has quit IRC08:04
openstackgerritSaravanan KR proposed openstack/puppet-vswitch: Add DPDK driver modprobe for loading modules on reboot  https://review.openstack.org/34746408:05
*** dgurtner has joined #puppet-openstack08:06
*** dgurtner has quit IRC08:06
*** dgurtner has joined #puppet-openstack08:06
*** snarwade has quit IRC08:07
*** jmiu has quit IRC08:08
*** jmiu_ is now known as jmiu08:08
*** jmiu_ has joined #puppet-openstack08:10
*** snarwade has joined #puppet-openstack08:11
*** hjensas has joined #puppet-openstack08:20
*** alee_dinner has joined #puppet-openstack08:25
fxpesterjust deployed mitaka on 14.04, trying to understand default network setup of AIO08:31
fxpesterfound this - http://paste.openstack.org/show/542517/ , is it ok loop1 device - no such device ?08:32
*** paramite has joined #puppet-openstack08:34
fxpesteroh and I was starting investigating cause cloud-init don`t work, but neutron-metadata don`t show any problems08:34
*** priteau has joined #puppet-openstack08:43
*** priteau has quit IRC08:57
*** moravec has quit IRC09:00
*** moravec has joined #puppet-openstack09:00
*** imcsk8 is now known as imcsk8|zZz09:00
*** ducttape_ has joined #puppet-openstack09:01
*** ducttape_ has quit IRC09:05
*** maestropandy has joined #puppet-openstack09:19
*** gfidente has joined #puppet-openstack09:32
openstackgerritAde Lee proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980609:40
*** alee_dinner has quit IRC09:54
*** iberezovskiy|off is now known as iberezovskiy09:55
*** maestropandy has quit IRC09:57
*** LarsErikP has joined #puppet-openstack09:57
*** alee_dinner has joined #puppet-openstack09:59
openstackgerritDaniel Pawlik proposed openstack/puppet-watcher: Added planer sections  https://review.openstack.org/34178810:02
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: added contrail analytics puppet module for tripleO  https://review.openstack.org/34773910:03
*** furlongm has quit IRC10:15
*** furlongm has joined #puppet-openstack10:16
openstackgerritDaniel Pawlik proposed openstack/puppet-nova: Add MultiStrOpt for scheduler_available_filters  https://review.openstack.org/32999110:21
openstackgerritMerged openstack/puppet-tempest: Ensure tempest_roles is an array  https://review.openstack.org/34754110:26
*** maestropandy has joined #puppet-openstack10:27
*** aernhart_ has joined #puppet-openstack10:42
*** aernhart_ has quit IRC10:43
*** maestropandy has quit IRC10:51
*** hjensas has quit IRC10:53
openstackgerritMerged openstack/puppet-neutron: Deprecate advertise_mtu option  https://review.openstack.org/34063110:54
*** rmart04 has joined #puppet-openstack11:12
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: added contrail analytics puppet module for tripleO  https://review.openstack.org/34773911:24
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: added contrail control puppet module for tripleO  https://review.openstack.org/34778111:34
*** shardy is now known as shardy_lunch11:43
*** bogdando has joined #puppet-openstack11:46
*** cdelatte has joined #puppet-openstack11:48
*** saneax is now known as saneax_AFK11:55
*** moravec has quit IRC12:01
*** ducttape_ has joined #puppet-openstack12:03
*** morazi has joined #puppet-openstack12:04
*** ducttape_ has quit IRC12:05
*** ducttape_ has joined #puppet-openstack12:06
openstackgerritLukas Bezdicka proposed openstack/puppet-ironic: Create ::ironic::inspector::ipxe to fully deploy iPXE boot  https://review.openstack.org/33488112:10
*** maestropandy has joined #puppet-openstack12:16
*** moravec has joined #puppet-openstack12:16
*** ducttape_ has quit IRC12:22
openstackgerritBrent Eagles proposed openstack/puppet-octavia: Adding housekeeping.pp  https://review.openstack.org/34752712:22
*** trown|outtypewww is now known as trown12:23
*** alee_dinner is now known as alee12:24
openstackgerritSimon Murray proposed openstack/puppet-horizon: Allow Optional Password Auto-Completion  https://review.openstack.org/34780312:26
openstackgerritMerged openstack/puppet-mistral: Prepare 8.2.0 (mitaka)  https://review.openstack.org/34619512:26
openstackgerritSimon Murray proposed openstack/puppet-horizon: Allow Optional Password Auto-Completion  https://review.openstack.org/34780312:30
*** moravec has quit IRC12:31
openstackgerritSimon Murray proposed openstack/puppet-horizon: Allow Optional Password Auto-Completion  https://review.openstack.org/34780312:31
*** beagles has joined #puppet-openstack12:35
*** severion has joined #puppet-openstack12:46
*** severion has quit IRC12:47
*** dfisher has joined #puppet-openstack13:00
*** shardy_lunch is now known as shardy13:01
*** severion has joined #puppet-openstack13:06
*** dgurtner has quit IRC13:10
*** bklei has joined #puppet-openstack13:14
EmilienMalee: http://logs.openstack.org/06/339806/40/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/4dfba12/console.html#_2016-07-27_10_18_13_21878113:17
EmilienMcongrats13:17
EmilienMabout https://review.openstack.org/#/c/339806/39..40/manifests/keystone.pp - should not be created by tempest?13:18
aleeEmilienM, yup13:18
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: added contrail webui puppet module for tripleO  https://review.openstack.org/34782413:18
aleeEmilienM, so yeah - I got it working, but we should talk a bit to see the right way to do it.13:18
aleeEmilienM, give a couple of minutes to get some links13:19
EmilienMtake time13:19
*** dgurtner has joined #puppet-openstack13:23
*** moravec has joined #puppet-openstack13:29
aleeEmilienM, my connection is painfully slow today for some reason -- so this may take a little time.13:31
aleeEmilienM, what we needed was to add the creator role to the tempest users13:32
aleeand to do that we used tempest_roles13:32
*** moravec has quit IRC13:32
aleewhat we found when we did that though was that for some reason, the Member role disappeared13:32
aleeand the reason for that is this ..13:33
aleehttps://github.com/openstack/tempest/blob/master/tempest/common/dynamic_creds.py#L161-L16913:33
aleeEmilienM, this is code that is in tempest that creates the Member role as well as assigning the user to that role13:33
aleeand notice that it only executes if role_assigned is not true13:34
EmilienMok13:34
aleewhich it is when you use tempest_roles ..13:35
EmilienMok, so I guess we need to manage this role manually?13:35
*** johnmilton has joined #puppet-openstack13:35
*** moravec has joined #puppet-openstack13:35
aleesee lines 150-152 in the same file13:35
*** ducttape_ has joined #puppet-openstack13:36
EmilienMok I see13:36
EmilienMso yeah, we need to create Member manually :)13:36
EmilienMalee: could you explain it ^ in the puppet code, where we create the role?13:36
EmilienMit's always good to have documentation why we're doing that13:36
aleeyup  -- either that or we added similar code for creator to tempest13:37
aleeie similar as for member and then we would not need tempest_roles13:37
EmilienMalee: second options looks cool13:37
aleebut I tend to believe the whole Member code here is a hack ..13:37
*** richm has joined #puppet-openstack13:37
aleesubject to random breakage as we have here13:38
EmilienMalee: let's do 1) now13:38
EmilienMjust add some doc now and we're good I guess13:38
EmilienMI'll make a last review on the code afterward13:38
aleeok - will do13:39
EmilienMthanks!13:39
EmilienMalee: we made it ! \o/13:39
alee\o/13:39
aleetriple-O next ..13:39
EmilienMalee: yeah, it will be much easier since we already fixed all our blockers13:41
*** ayoung has joined #puppet-openstack13:41
openstackgerritcraig delatte proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/34446413:42
*** maestropandy has left #puppet-openstack13:42
*** johnmilton has quit IRC13:47
openstackgerritSaravanan KR proposed openstack/puppet-vswitch: Add DPDK driver modprobe for loading modules on reboot  https://review.openstack.org/34746413:50
-openstackstatus- NOTICE: Gerrit is being restarted now to relieve performance degradation13:53
iberezovskiyhurray13:54
EmilienMnew mitaka release ongoing https://review.openstack.org/34785213:55
aleeEmilienM, https://paste.fedoraproject.org/396375/14696277/  --> this clear enough?13:56
iberezovskiycool13:56
EmilienMalee: that's excellent13:57
aleeEmilienM, cool - will submit now (if I can)13:57
*** rmart04 has quit IRC13:59
*** skramaja has joined #puppet-openstack13:59
openstackgerritAde Lee proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980614:00
*** johnmilton has joined #puppet-openstack14:00
*** trown is now known as trown|brb14:02
*** moravec1 has joined #puppet-openstack14:02
*** moravec has quit IRC14:04
*** trown|brb is now known as trown14:05
openstackgerritMerged openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/34763514:06
openstackgerritMerged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/34763614:06
openstackgerritMerged openstack/puppet-watcher: Added planer sections  https://review.openstack.org/34178814:06
*** chem|off has quit IRC14:09
*** chem|off has joined #puppet-openstack14:10
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/34786014:11
EmilienMgfidente: excellent candidate to backport it to stable/hammer FYI ^14:13
EmilienMgfidente: it would have been great to have a commit message14:14
gfidenteEmilienM ack still working on the change14:20
gfidentewill be more verbose in the update14:20
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/34786014:30
beaglesEmilienM (or anyone) quick question about a line in puppet-nova https://github.com/openstack/puppet-nova/blob/master/manifests/compute.pp#L22814:33
beaglesEmilienM, oh wait.. scrap that14:33
beaglesEmilienM, well actually no.. it seems kind of weird14:34
*** alee has quit IRC14:34
beaglesEmilienM, what is that line trying to accomplish ... why not just use the vcpu_pin_set parameter14:34
* beagles is asking for educational purposes more than anything14:35
*** imcsk8|zZz is now known as imcsk814:36
EmilienM$vcpu_pin_set_real = pick(join(any2array($vcpu_pin_set), ','), $::os_service_default)14:36
*** alee has joined #puppet-openstack14:36
EmilienMso if $vcpu_pin_set is set in Hiera or when declaring the class, we make sure the value is an array14:37
*** alee has quit IRC14:37
EmilienMif not set, we make sure the value is not set in the config file14:37
EmilienMso we depend on default in Nova14:37
EmilienMbeagles: http://docs.openstack.org/developer/puppet-openstack-guide/coding-style.html#config-file-defaults-and-parameters14:38
colonwqEmilienM, My RGW/Keystone V3 configuration is being reviewed for correctness.14:38
EmilienMcolonwq: cool14:38
*** alee has joined #puppet-openstack14:38
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Allow to configure [ssh] section in ironic config  https://review.openstack.org/34744914:39
beaglesaaah, EmilienM, so if the config value in hiera is bogus we just continue with the defaults?14:40
beaglesso it is more of a "hardening" thing?14:41
beagles(the pick with the os_service_default thing threw me)14:41
*** moravec1 has quit IRC14:42
EmilienMbeagles: pick is just taking the value on the left if possible otherwise on the righ14:42
EmilienMhttps://github.com/puppetlabs/puppetlabs-stdlib/blob/master/lib/puppet/parser/functions/pick.rb14:42
beaglesEmilienM, sure...14:42
iurygregorymorning14:42
iberezovskiyhey14:43
EmilienMbeagles: is it clear?14:43
openstackgerritIvan Berezovskiy proposed openstack/puppet-ironic: Configure keystone authtoken options  https://review.openstack.org/34602814:43
iurygregorydenis is not around?14:44
iberezovskiyhe took day off14:44
EmilienMlucky guy14:44
*** moravec has joined #puppet-openstack14:45
iurygregoryhaha14:45
beaglesEmilienM, it is the first time I noticed a line with pick with $::os_service_default on the right, is all. Is this a common idiom? If an invalid value is passed, we ignore it and go to the defaults?14:45
iurygregoryEmilienM, mwhahaha, iberezovskiy  fyi https://review.openstack.org/#/c/347105/ ;) authtoken class14:46
beagles(defaults = treat as unset)14:46
EmilienMbeagles: what is invalid?14:47
EmilienMiurygregory: yeah I saw it14:48
EmilienMbut didn't review it, saw WIP14:48
EmilienMbeagles: if a string or array is passed, it's good value14:48
iurygregoryEmilienM, lol sorry i just let the old commit message =(14:48
iberezovskiyiurygregory, hm. we should do the same for all modules, shoudn't we?14:48
EmilienMiberezovskiy: wait14:49
EmilienMwe wait for one module to be merge14:49
iurygregoryyeah14:49
EmilienMso we are consistent everywhere14:49
iberezovskiysure, that's why I'm asking :)14:49
dfisherchem|off: ping14:49
*** paramite has quit IRC14:49
*** chem|off is now known as chem14:49
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-barbican: Move barbican to authtoken  https://review.openstack.org/34710514:49
chemdfisher: hi, I've seen your comment, why not use the same code as in the function with Puppet::Util:Exec.execute  ?14:50
iurygregoryEmilienM, not WIP anymore hahah14:50
beaglesEmilienM, okay... so IIUC, vcpu_pin_set is $::os_service_default if no value is specified by the code. The aforementioned line attempts to convert the supplied value to an intermediate value, if were to result in no value, it would revert to the $::os_service_default. So it seems like the only reason for the pick would be if something passed a vcpu_pin_set value that resulted in the join(any2array()) bit returning something that pi14:50
beaglesck thought was undefined14:50
dfisherchem: a straight execute will need the logic to look in the user's env, $HOME/openrc, or a supplied env hash … right?14:51
openstackgerritIvan Berezovskiy proposed openstack/puppet-ironic: Configure keystone authtoken options  https://review.openstack.org/34602814:51
beaglesEmilienM, if no value was supplied it would effectively be no-op14:51
EmilienMbeagles: exactly14:52
EmilienMbeagles: the value is not set in nova.conf14:52
EmilienMand we rely on the default in Nova14:52
beaglesEmilienM, if a value were supplied then the join(any2array()) seems like it is a data hardening thing if the value passed into vcpu_pin_set wasn't an array or something to that effect14:53
EmilienMbeagles: right14:53
*** EmilienM sets mode: +o iberezovskiy14:54
*** EmilienM sets mode: +o _degorenko|afk14:54
*** EmilienM sets mode: +o mfisch14:54
beaglesEmilienM, okay cool, thanks for the clarification :)14:55
openstackgerritEmilien Macchi proposed openstack/puppet-modulesync-configs: WIP - Add CI scripts  https://review.openstack.org/34491614:56
EmilienMbeagles: my pleasure14:56
*** jistr is now known as jistr|call15:00
aleeEmilienM, hmm something strange happened in last run ..15:01
aleeEmilienM, looks like tempest_roles was not set correctly ..15:02
iurygregorydmsimard, the mascot should represent the component and the team =)15:03
dmsimardiurygregory: so #puppet-openstack howls all the time or something ? I don't know15:04
iurygregorydmsimard, not in the context lol like we work hard and i put wolf =P15:05
dmsimardSO IT'S YOU15:05
iurygregoryyeah15:06
iurygregory:p15:06
dmsimardsorry, I'm an honest guy and I don't see the relationship between wolf and puppet ;(15:06
EmilienMme neither :)15:06
iurygregorydmsimard, no problem lol15:06
dmsimardopenstack-infra had some interesting choices15:06
dmsimardthey put up their vote recently15:06
aleeEmilienM, any idea what might have gone on here ? http://logs.openstack.org/06/339806/41/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/c921e54/logs/tempest.conf.txt.gz15:07
aleeEmilienM, to tempest_roles ..15:07
aleeEmilienM, maybe some change to puppet-tempest merged that broke paring of lists?15:10
aleeparsing ..15:10
*** alee is now known as alee_afk15:12
EmilienMalee_afk: weird15:13
chemdfisher: sorry was in a meeting15:13
dfisherchem: no worries - take your time15:14
chemdfisher: hum ... how did you do it in the function call ?15:14
dfisherhttps://review.openstack.org/#/c/330623/7/lib/puppet/parser/functions/openstack_resource_id_getter.rb15:14
dfisherexactly that.15:14
dfisherif argv.size == 1:  look at user's env or $HOME/openrc15:15
dfisherif argv.size == 2:  use the supplied env hash15:15
iurygregoryit's not my fault but other people put +1 in wolf lol15:17
*** danpawlik has quit IRC15:18
chemdfisher: I may have an idea, but it's very unusual, so I test it before.  Hold on, I let you know if it goes well, if not then you will have to go with the same code as above.15:21
dfisherheh.  ok :)15:21
dfishershell out to erlang!  go!15:21
openstackgerritSimon Murray proposed openstack/puppet-horizon: Allow Optional Password Auto-Completion  https://review.openstack.org/34780315:23
*** rcernin has quit IRC15:24
*** leanderthal is now known as leanderthal|afk15:24
openstackgerritMerged openstack/puppet-octavia: Adding housekeeping.pp  https://review.openstack.org/34752715:29
*** bklei_ has joined #puppet-openstack15:30
*** bklei has quit IRC15:34
*** jistr|call is now known as jistr15:35
*** ntpttr__ has joined #puppet-openstack15:35
alee_afkEmilienM, yeah15:36
*** ducttape_ has quit IRC15:37
alee_afkEmilienM, I'll take out the depends-on - because that already merged - and retest?15:38
EmilienMalee_afk: ok15:38
*** ifarkas is now known as ifarkas_away15:39
openstackgerritAde Lee proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980615:41
*** xarses has quit IRC15:42
openstackgerritMerged openstack/puppet-cinder: SolidFire backend deployment should use $::os_service_default  https://review.openstack.org/34706615:42
*** guimaluf has joined #puppet-openstack15:45
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-neutron: WIP: transform name to uuid.  https://review.openstack.org/34746815:45
chemdfisher: there may be hope.  I don't have a neutron vm up, so I've just uploaded the code to gerrit if you want to test it.  It gives you the idea.  I'll spawn a vm later on.15:46
dfisherok15:46
openstackgerritMerged openstack/puppet-neutron: Remove router_id option  https://review.openstack.org/34063915:46
dfisher^ remove router_id.  lol.15:51
dfisherall the work i'm doing is to set that damn variable.15:51
openstackgerritMerged openstack/puppet-neutron: Deprecate min_l3_agents_per_router option  https://review.openstack.org/34063715:51
openstackgerritMerged openstack/puppet-nova: Add MultiStrOpt for scheduler_available_filters  https://review.openstack.org/32999115:51
openstackgerritMerged openstack/puppet-nova: Fix typo in tests  https://review.openstack.org/34759815:52
dfisherchem: looks straight-forward enough.  gimme a few to test it15:53
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class and configure cache correctly.  https://review.openstack.org/32834715:55
openstackgerritMerged openstack/puppet-neutron: Changing ovn plugin to ml2 mechanism driver  https://review.openstack.org/34034315:57
*** trown is now known as trown|lunch16:02
dfisherchem:  it's very close.  it's currently dying because it can't find the neutron method:  https://github.com/openstack/puppet-neutron/blob/master/lib/puppet/provider/neutron.rb#L9616:11
*** rwsu has quit IRC16:11
*** ducttape_ has joined #puppet-openstack16:12
dfisheri don't even know where that's set ...16:13
dfisherah, i see.  it's set in the providers that inherit from this provider16:13
dfisherso, setting 'commands :neutron => "neutron"' in the l3_agent provider doesn't seem to work for me.  how do you set this so neutron.rb picks it up?16:19
*** jtomasek has quit IRC16:22
chemdfisher: the way the class is created is *very* unusual, so there may be dragons there.  That's very unfortunate that the commands definition doesn't work.  My next try would be define self.neutron with Puppet::Util::Exec.execute('neutron', args)16:24
dfisherok.  poking ...16:24
chemdfisher: so at least we won't have to redo the auth ourself16:24
dfishermy ruby class internals knowledge is slightly above absolute zero16:24
*** atom_ has quit IRC16:25
chemdfisher: well, here it's the way puppet create stuff.  Usually you creating provider using the Puppet::Type interfacte, not directly.  But, in the end it's a class with code, so it may be working.16:26
dfisherstill getting16:28
dfisherError: /Stage[main]/Main/Neutron_l3_agent_config[DEFAULT/router_id]: Could not evaluate: undefined method `neutron' for Puppet::Provider::Neutron:Class16:28
dfisherhttp://paste.openstack.org/show/542624/16:29
dfishernot sure if I declared that correctly16:29
openstackgerritEmilien Macchi proposed openstack/puppet-modulesync-configs: WIP - Add CI scripts  https://review.openstack.org/34491616:30
*** rwsu has joined #puppet-openstack16:30
*** xarses has joined #puppet-openstack16:31
chemdfisher: oups, sorry, you have to define the function in the right class... -> http://paste.openstack.org/show/542627/16:38
dfisheris that in neutron_l3_agent_config/ini_settings.rb ?16:39
chemdfisher: if that doesn't work, I will take some time to explore why we don't have access to the neutron command16:39
chemdfisher: hold on, this is not the right way.  I'm redefining it globaly while I want only our instance of the puppet::provider::neutron class to have it ... wait a minute16:41
dfisherok16:41
guimalufhey guys, I'm running a simple acceptance/beaker test with docker, but even using `docker_preserve_image: true` it's taking 7 minutes to run just a notify {"working":}, how can I improve test speed?16:42
*** derekh has quit IRC16:42
alee_afkEmilienM, same behavior -- something changed :/16:43
alee_afkEmilienM, going to dinner -- will check again later -- but for some reason the tempest_roles are not being set correctly16:44
EmilienMalee_afk: ok16:46
*** alee_afk has quit IRC16:50
*** bklei_ has quit IRC16:53
*** bklei has joined #puppet-openstack16:55
*** trown|lunch is now known as trown16:55
chemdfisher: all in all it's absurd that 'commands :neutron => "neutron"' is not in neutron.rb17:00
dfisheri tried adding it there too and that didn't work either :(17:01
chemdfisher: try adding 'initvars' just before it in neutron.rb17:01
chemdfisher: that's what is done in openstacklib/**/openstack.rb17:02
dfisherchem: http://paste.openstack.org/show/542666/ ?17:03
chemdfisher: yep17:03
dfisherok, that didn't error … didn't work but I'll track it down further...17:04
chemdfisher: do you see the commands triggered ?17:05
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/34795617:05
*** colonwq is now known as colonwq_lunch17:05
dfisheryes.  the value passed to 'to_uuid' isn't showing up in the command though.17:06
dfishertrying to figure that out ...17:06
chemdfisher: can you try to hardcode it to check that the stuff really works, with authent and all ?17:07
dfishersure, one sec.17:08
dfisherit does end with:17:08
dfisherError: /Stage[main]/Main/Neutron_l3_agent_config[DEFAULT/router_id]: Could not evaluate: undefined method `cleanup_csv_with_id' for #<Puppet::Type::Neutron_l3_agent_config::ProviderIni_setting:0x00000002cf8c10>17:08
chemhum remove the method for now, it's not that important, we'll get back to this later17:09
dfisheri will.  still trying to get the hardcoded stuff to work.  hold on a tick...17:10
*** tesseract- has quit IRC17:13
dfisherchem: http://paste.openstack.org/show/542671/17:20
dfisheri don't understand what it's doing…17:20
dfisheri dumped out the authenv before running neutron(…) and it's right17:20
dfisheri need to run off to lunch however.  will be back in about an hour or so.17:21
*** iberezovskiy is now known as iberezovskiy|off17:26
*** guimaluf has quit IRC17:28
chemdfisher: well if auth is right, we are nearly done with the heavy stuff.  That should mean that this solution is doable.  The error you've got look like you passed 'router-show provider --format=value --column=id' to neutron_auth instead of 'neutron-show', 'provider', '--format=value' ...17:29
dfisherno, it's neutron router-show <routername> —format —column17:29
dfisheror whatever17:29
dfisherbut yeah.  it's something like that.17:30
dfisherjust need to figure it out.17:30
dfisherwill poke when I get nback.17:30
*** guimaluf has joined #puppet-openstack17:31
*** shardy has quit IRC17:32
openstackgerritMerged openstack/puppet-tripleo: Unit tests for HAproxy dual stack  https://review.openstack.org/34695817:34
*** gfidente has quit IRC17:43
openstackgerritEmilien Macchi proposed openstack/puppet-modulesync-configs: WIP - Add CI scripts  https://review.openstack.org/34491617:51
*** trown is now known as trown|brb17:51
chemdfisher: what I meant is that I think you passed a String instead of an Array17:51
chemdfisher: not sure I was clear :)17:51
*** ntpttr__ has quit IRC17:59
*** colonwq_lunch is now known as colonwq18:11
EmilienMI never saw a duplicated resource when using ensure_resource18:12
EmilienMhttp://logs.openstack.org/47/328347/20/check/gate-tripleo-ci-centos-7-nonha-multinode-nv/bbfec4b/logs/postci.txt.gz#_2016-07-27_16_49_42_00018:12
EmilienMif your resource don't have the same tag when declared multiple times, is it a big deal?18:12
openstackgerritEmilien Macchi proposed openstack/puppet-oslo: cache: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/34798018:14
openstackgerritEmilien Macchi proposed openstack/puppet-horizon: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/34798118:15
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/34798218:16
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class and configure cache correctly.  https://review.openstack.org/32834718:17
iurygregoryEmilienM, a while a go we have this problem i think see https://review.openstack.org/#/c/327185/18:22
EmilienMcool18:23
EmilienMiurygregory: nice catch18:23
iurygregoryEmilienM,  the guy maybe have proposed the same 3 patches lol18:24
iurygregorymaybe we should add abandon since is not updated =)18:24
*** ayoung has quit IRC18:25
iurygregoryEmilienM, we should block this patch I think since auth_version is not deprecated https://review.openstack.org/#/c/336328/18:25
EmilienMI need to eat18:25
iurygregorygo o/18:25
*** trown|brb is now known as trown18:28
*** dfisher has quit IRC18:32
*** ericksonsantos has quit IRC18:44
*** beagles is now known as beagles-brb18:51
*** ntpttr__ has joined #puppet-openstack18:52
*** dfisher has joined #puppet-openstack19:01
*** dfisher1 has joined #puppet-openstack19:03
dfisher1chem: ah, ok.  back from lunch and working on this some more19:03
*** dfisher has quit IRC19:05
chemdfisher1: thanks a lot, I really don't have the time to help more ...19:06
*** beagles-brb is now known as beagles19:06
dfisher1it's cool.  i'll go from here.19:06
dfisher1thanks for the help so far.19:06
dfisher1will update gerrit when I get somewhere19:06
*** haplo37__ has joined #puppet-openstack19:09
dfisher1chem: Warning: Neutron_l3_agent_config[DEFAULT/router_id](provider=ini_setting): DATA is 0c1fc0ac-b82d-412d-9772-ecc4a108a92319:11
dfisher1got it.  ;)19:11
dfisher1now to clean up everything.19:11
chemdfisher1: \o/ !!19:11
chemdfisher1: this is supper good news.19:11
dfisher1thanks to you!19:12
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-neutron: Put neutron provider commands in neutron.rb  https://review.openstack.org/34800719:20
chemdfisher1: I've made another patch for neutron to get the command in neutron.rb.  So that we can isolate potential problem with this.  You should make the l3 review depends on it.  And don't forget to add the co authored by tag to the reviews we've done together.19:21
*** foxhunt_away is now known as foxhunt7219:23
dfisher1i will absolutely do all of that.19:25
chemdfisher1: ack, thanks19:26
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/34798219:28
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class and configure cache correctly.  https://review.openstack.org/32834719:33
dfisher1chem: really really dumb question for you19:35
chemdfisher1: no such thing as dumb question :)19:35
dfisher1chem: https://review.openstack.org/#/c/347463/2/lib/puppet/provider/openstack_config/ini_setting.rb19:36
chemdfisher1: (well ... most of the time..)19:36
dfisher1we're tweaking setting19:36
dfisher1shouldn't we be tweaking value?19:36
*** alee_afk has joined #puppet-openstack19:36
dfisher1or value= ?19:36
dfisher1or should I update @resource[:value] to what to_uuid() returns?19:37
chemdfisher1: humm ... let me have a look this was a bit to fast I think.19:38
* dfisher1 nods19:38
chemdfisher1: oki, this is wrong, let me do another "template" for ya19:40
dfisher1ok19:41
dfisher1you can just paste it.   i'll take it from there.19:41
*** ntpttr- has quit IRC19:50
*** foxhunt72 is now known as foxhunt_away19:50
*** ntpttr- has joined #puppet-openstack19:51
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-openstacklib: WIP: add a way for provider to transform value in config.  https://review.openstack.org/34746319:53
chemdfisher1: ^ this is the proper version ...19:53
* dfisher1 nods19:54
*** ntpttr__ has quit IRC19:54
chemdfisher1: so you have to implement the to_uuid function as well so we've got idempotency19:54
dfisher1ok.19:54
dfisher1will do19:54
dfisher1chem: thanks again!  (i'm sure I'll be back before too long with some other hairball :) )19:54
chemdfisher1: my plaisure, it's a interseting problem and not so uncommon.  I'm sure there are other places where this could be handy.  Anyway let's see it completly working before drinking pints of beer.19:56
dfisher1amen.19:56
dfisher1pints in Barca ;)19:56
dfisher1or, sangria19:56
chemdfisher1: won't be there unfortunatly ...19:57
dfisher1doh!19:57
iurygregorytry the TSP to go =)19:58
openstackgerritGuilherme Balzana proposed openstack/puppet-monasca: [WIP] Provides an example on how to use this module  https://review.openstack.org/34621820:09
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-neutron: Put neutron provider commands in neutron.rb  https://review.openstack.org/34800720:16
*** guimaluf has quit IRC20:16
*** cdelatte has quit IRC20:17
*** ntpttr__ has joined #puppet-openstack20:20
*** ericksonsantos has joined #puppet-openstack20:20
*** guimaluf has joined #puppet-openstack20:23
openstackgerritMerged openstack/puppet-ironic: Allow to configure [ssh] section in ironic config  https://review.openstack.org/34744920:26
dfisher1chem: it's *so* close.20:31
*** ducttape_ has quit IRC20:31
chemdfisher1: what's wrong ?20:31
dfisher1the first run sets router_id=<router_name>20:31
dfisher1if I run it again with no changes it sets it to router_id=<router_uuid>20:31
dfisher1i can't figure out20:31
dfisher1why20:31
*** dgurtner has quit IRC20:32
dfisher1what kind of logs can I get you?20:32
dfisher1chem: http://paste.openstack.org/show/542690/20:35
chemdfisher1: hum ... you should commit all your modifications to gerrit as they are so that there is no confusion about the current state of the code20:35
dfisher1will do.20:35
dfisher1give me a moment.20:35
*** ntpttr__ has quit IRC20:35
*** ntpttr__ has joined #puppet-openstack20:36
chemdfisher1: you should use "puppet apply -d test.pp" so that we see more information20:37
dfisher1i can do that too.20:37
chemdfisher1: and we should rename transform_to to transformation ... it reads better.20:38
openstackgerritDrew Fisher proposed openstack/puppet-openstacklib: WIP: add a way for provider to transform value in config.  https://review.openstack.org/34746320:40
dfisher1doing neutron one now20:41
openstackgerritDrew Fisher proposed openstack/puppet-neutron: WIP: transform name to uuid.  https://review.openstack.org/34746820:44
dfisher1chem: both uploaded20:45
chemdfisher1: ack20:45
*** cdelatte has joined #puppet-openstack20:51
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/34795620:52
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Create role for the fake openstack-core resource  https://review.openstack.org/34731520:55
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Remove global openstack-core resource  https://review.openstack.org/34747220:55
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Next generation HA architecture work  https://review.openstack.org/34265020:55
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Move nova constraints from controller manifest to each service  https://review.openstack.org/34730920:55
*** ayoung has joined #puppet-openstack20:55
*** moravec has quit IRC20:56
chemdfisher1: oki, for the moment I cannot why it fails in such way.  maybe with the debug output it will be clearer.  Anyway, I'm off now.  I let a comment https://review.openstack.org/#/c/347468/4 .  See you tomorrow for the ultimate iteration :)  I will fire up in neutron instance, I promise :)20:57
dfisher1chem: sounds good.  have a good evening!20:57
*** ducttape_ has joined #puppet-openstack20:57
chemdfisher1: well ... it's more like good night :)20:58
dfisher1or that!20:58
openstackgerritDrew Fisher proposed openstack/puppet-neutron: WIP: transform name to uuid.  https://review.openstack.org/34746821:01
*** ntpttr__ has quit IRC21:05
*** ntpttr__ has joined #puppet-openstack21:11
*** moravec has joined #puppet-openstack21:13
*** ntpttr__ has quit IRC21:14
*** moravec has quit IRC21:17
*** dgurtner has joined #puppet-openstack21:28
openstackgerritMerged openstack/puppet-keystone: Fix typo in resource test  https://review.openstack.org/34761921:28
openstackgerritMerged openstack/puppet-cinder: Deprecate drivers in Cinder.conf DEFAULT section.  https://review.openstack.org/34653121:52
*** haplo37__ has quit IRC21:52
*** ducttape_ has quit IRC22:05
*** bklei has quit IRC22:08
*** bklei has joined #puppet-openstack22:08
*** bklei has quit IRC22:13
*** moravec has joined #puppet-openstack22:24
*** morazi has quit IRC22:25
*** trown is now known as trown|outtypewww22:49
*** dfisher1 has quit IRC23:28
*** saneax_AFK is now known as saneax23:35
*** ducttape_ has joined #puppet-openstack23:37
*** dgurtner has quit IRC23:43
*** bklei has joined #puppet-openstack23:46
*** bklei has quit IRC23:51
« Tuesday, 2016-07-26 Index Thursday, 2016-07-28 »

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!