Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-07-27

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:26 openstackgerrit Brent Eagles proposed openstack/puppet-octavia: Adding housekeeping.pp  https://review.openstack.org/347527
00:37 aernhart_ joined #puppet-openstack
00:39 bklei joined #puppet-openstack
00:50 vinsh joined #puppet-openstack
00:56 moravec joined #puppet-openstack
00:56 ducttape_ joined #puppet-openstack
01:20 xarses joined #puppet-openstack
01:34 openstackgerrit joined #puppet-openstack
02:29 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-nova: Fix typo in tests  https://review.openstack.org/347598
02:29 iurygregory_ joined #puppet-openstack
02:41 v1k0d3n joined #puppet-openstack
02:48 ducttape_ joined #puppet-openstack
03:20 johnmilton joined #puppet-openstack
03:31 ducttape_ joined #puppet-openstack
04:32 rwsu joined #puppet-openstack
04:35 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-barbican: WIP - Move barbican to authtoken  https://review.openstack.org/347105
04:41 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Fix typo in resource test  https://review.openstack.org/347619
04:57 ducttape_ joined #puppet-openstack
05:21 maestropandy joined #puppet-openstack
05:47 v1k0d3n joined #puppet-openstack
05:58 ducttape_ joined #puppet-openstack
06:10 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/347635
06:11 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/347636
06:38 hjensas joined #puppet-openstack
06:44 tesseract- joined #puppet-openstack
06:46 rcernin joined #puppet-openstack
06:48 maestropandy joined #puppet-openstack
06:50 rhefner joined #puppet-openstack
06:51 igorbelikov joined #puppet-openstack
06:52 davidmichaelkarr joined #puppet-openstack
06:59 ducttape_ joined #puppet-openstack
07:15 danpawlik joined #puppet-openstack
07:36 mtreinish joined #puppet-openstack
07:38 shardy joined #puppet-openstack
07:39 basilAB joined #puppet-openstack
07:40 foxhunt_away joined #puppet-openstack
07:49 atom_ joined #puppet-openstack
07:50 jtomasek joined #puppet-openstack
08:00 ducttape_ joined #puppet-openstack
08:03 derekh joined #puppet-openstack
08:05 openstackgerrit Saravanan KR proposed openstack/puppet-vswitch: Add DPDK driver modprobe for loading modules on reboot  https://review.openstack.org/347464
08:06 dgurtner joined #puppet-openstack
08:06 dgurtner joined #puppet-openstack
08:10 jmiu_ joined #puppet-openstack
08:11 snarwade joined #puppet-openstack
08:20 hjensas joined #puppet-openstack
08:25 alee_dinner joined #puppet-openstack
08:31 fxpester just deployed mitaka on 14.04, trying to understand default network setup of AIO
08:32 fxpester found this - http://paste.openstack.org/show/542517/ , is it ok loop1 device - no such device ?
08:34 paramite joined #puppet-openstack
08:34 fxpester oh and I was starting investigating cause cloud-init don`t work, but neutron-metadata don`t show any problems
08:43 priteau joined #puppet-openstack
09:00 moravec joined #puppet-openstack
09:01 ducttape_ joined #puppet-openstack
09:19 maestropandy joined #puppet-openstack
09:31 gfidente joined #puppet-openstack
09:40 openstackgerrit Ade Lee proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/339806
09:57 LarsErikP joined #puppet-openstack
09:59 alee_dinner joined #puppet-openstack
10:02 openstackgerrit Daniel Pawlik proposed openstack/puppet-watcher: Added planer sections  https://review.openstack.org/341788
10:03 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: added contrail analytics puppet module for tripleO  https://review.openstack.org/347739
10:16 furlongm joined #puppet-openstack
10:21 openstackgerrit Daniel Pawlik proposed openstack/puppet-nova: Add MultiStrOpt for scheduler_available_filters  https://review.openstack.org/329991
10:26 openstackgerrit Merged openstack/puppet-tempest: Ensure tempest_roles is an array  https://review.openstack.org/347541
10:27 maestropandy joined #puppet-openstack
10:42 aernhart_ joined #puppet-openstack
10:54 openstackgerrit Merged openstack/puppet-neutron: Deprecate advertise_mtu option  https://review.openstack.org/340631
11:12 rmart04 joined #puppet-openstack
11:24 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: added contrail analytics puppet module for tripleO  https://review.openstack.org/347739
11:34 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: added contrail control puppet module for tripleO  https://review.openstack.org/347781
11:46 bogdando joined #puppet-openstack
11:47 cdelatte joined #puppet-openstack
12:03 ducttape_ joined #puppet-openstack
12:03 morazi joined #puppet-openstack
12:06 ducttape_ joined #puppet-openstack
12:10 openstackgerrit Lukas Bezdicka proposed openstack/puppet-ironic: Create ::ironic::inspector::ipxe to fully deploy iPXE boot  https://review.openstack.org/334881
12:15 maestropandy joined #puppet-openstack
12:16 moravec joined #puppet-openstack
12:22 openstackgerrit Brent Eagles proposed openstack/puppet-octavia: Adding housekeeping.pp  https://review.openstack.org/347527
12:26 openstackgerrit Simon Murray proposed openstack/puppet-horizon: Allow Optional Password Auto-Completion  https://review.openstack.org/347803
12:26 openstackgerrit Merged openstack/puppet-mistral: Prepare 8.2.0 (mitaka)  https://review.openstack.org/346195
12:30 openstackgerrit Simon Murray proposed openstack/puppet-horizon: Allow Optional Password Auto-Completion  https://review.openstack.org/347803
12:31 openstackgerrit Simon Murray proposed openstack/puppet-horizon: Allow Optional Password Auto-Completion  https://review.openstack.org/347803
12:35 beagles joined #puppet-openstack
12:46 severion joined #puppet-openstack
13:00 dfisher joined #puppet-openstack
13:06 severion joined #puppet-openstack
13:14 bklei joined #puppet-openstack
13:17 EmilienM alee: http://logs.openstack.org/06/339806/40/c​heck/gate-puppet-openstack-integration-3​-scenario002-tempest-centos-7/4dfba12/co​nsole.html#_2016-07-27_10_18_13_218781
13:17 EmilienM congrats
13:17 EmilienM about https://review.openstack.org/#/c/339806/39..40/manifests/keystone.pp - should not be created by tempest?
13:18 alee EmilienM, yup
13:18 openstackgerrit Michael  Henkel proposed openstack/puppet-tripleo: added contrail webui puppet module for tripleO  https://review.openstack.org/347824
13:18 alee EmilienM, so yeah - I got it working, but we should talk a bit to see the right way to do it.
13:19 alee EmilienM, give a couple of minutes to get some links
13:19 EmilienM take time
13:22 dgurtner joined #puppet-openstack
13:28 moravec joined #puppet-openstack
13:31 alee EmilienM, my connection is painfully slow today for some reason -- so this may take a little time.
13:32 alee EmilienM, what we needed was to add the creator role to the tempest users
13:32 alee and to do that we used tempest_roles
13:32 alee what we found when we did that though was that for some reason, the Member role disappeared
13:33 alee and the reason for that is this ..
13:33 alee https://github.com/openstack/tempest/blob/mas​ter/tempest/common/dynamic_creds.py#L161-L169
13:33 alee EmilienM, this is code that is in tempest that creates the Member role as well as assigning the user to that role
13:34 alee and notice that it only executes if role_assigned is not true
13:34 EmilienM ok
13:35 alee which it is when you use tempest_roles ..
13:35 EmilienM ok, so I guess we need to manage this role manually?
13:35 johnmilton joined #puppet-openstack
13:35 moravec joined #puppet-openstack
13:35 alee see lines 150-152 in the same file
13:36 ducttape_ joined #puppet-openstack
13:36 EmilienM ok I see
13:36 EmilienM so yeah, we need to create Member manually :)
13:36 EmilienM alee: could you explain it ^ in the puppet code, where we create the role?
13:36 EmilienM it's always good to have documentation why we're doing that
13:37 alee yup  -- either that or we added similar code for creator to tempest
13:37 alee ie similar as for member and then we would not need tempest_roles
13:37 EmilienM alee: second options looks cool
13:37 alee but I tend to believe the whole Member code here is a hack ..
13:37 richm joined #puppet-openstack
13:38 alee subject to random breakage as we have here
13:38 EmilienM alee: let's do 1) now
13:38 EmilienM just add some doc now and we're good I guess
13:38 EmilienM I'll make a last review on the code afterward
13:38 alee ok - will do
13:39 EmilienM thanks!
13:39 EmilienM alee: we made it ! \o/
13:39 alee \o/
13:39 alee triple-O next ..
13:40 EmilienM alee: yeah, it will be much easier since we already fixed all our blockers
13:41 ayoung joined #puppet-openstack
13:41 openstackgerrit craig delatte proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/344464
13:42 maestropandy left #puppet-openstack
13:49 openstackgerrit Saravanan KR proposed openstack/puppet-vswitch: Add DPDK driver modprobe for loading modules on reboot  https://review.openstack.org/347464
13:53 openstackstatus NOTICE: Gerrit is being restarted now to relieve performance degradation
13:54 iberezovskiy hurray
13:55 EmilienM new mitaka release ongoing https://review.openstack.org/347852
13:55 alee EmilienM, https://paste.fedoraproject.org/396375/14696277/  --> this clear enough?
13:56 iberezovskiy cool
13:57 EmilienM alee: that's excellent
13:57 alee EmilienM, cool - will submit now (if I can)
13:59 skramaja joined #puppet-openstack
14:00 openstackgerrit Ade Lee proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/339806
14:00 johnmilton joined #puppet-openstack
14:02 moravec1 joined #puppet-openstack
14:05 openstackgerrit Merged openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/347635
14:06 openstackgerrit Merged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/347636
14:06 openstackgerrit Merged openstack/puppet-watcher: Added planer sections  https://review.openstack.org/341788
14:09 chem|off joined #puppet-openstack
14:11 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/347860
14:13 EmilienM gfidente: excellent candidate to backport it to stable/hammer FYI ^
14:14 EmilienM gfidente: it would have been great to have a commit message
14:20 gfidente EmilienM ack still working on the change
14:20 gfidente will be more verbose in the update
14:30 openstackgerrit Giulio Fidente proposed openstack/puppet-ceph: Refresh keys if caps are changed  https://review.openstack.org/347860
14:33 beagles EmilienM (or anyone) quick question about a line in puppet-nova https://github.com/openstack/puppet-nov​a/blob/master/manifests/compute.pp#L228
14:33 beagles EmilienM, oh wait.. scrap that
14:34 beagles EmilienM, well actually no.. it seems kind of weird
14:34 beagles EmilienM, what is that line trying to accomplish ... why not just use the vcpu_pin_set parameter
14:35 * beagles is asking for educational purposes more than anything
14:36 EmilienM $vcpu_pin_set_real = pick(join(any2array($vcpu_pin_set), ','), $::os_service_default)
14:36 alee joined #puppet-openstack
14:37 EmilienM so if $vcpu_pin_set is set in Hiera or when declaring the class, we make sure the value is an array
14:37 EmilienM if not set, we make sure the value is not set in the config file
14:37 EmilienM so we depend on default in Nova
14:37 EmilienM beagles: http://docs.openstack.org/developer/pu​ppet-openstack-guide/coding-style.html​#config-file-defaults-and-parameters
14:37 colonwq EmilienM, My RGW/Keystone V3 configuration is being reviewed for correctness.
14:38 EmilienM colonwq: cool
14:38 alee joined #puppet-openstack
14:38 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Allow to configure [ssh] section in ironic config  https://review.openstack.org/347449
14:40 beagles aaah, EmilienM, so if the config value in hiera is bogus we just continue with the defaults?
14:41 beagles so it is more of a "hardening" thing?
14:41 beagles (the pick with the os_service_default thing threw me)
14:42 EmilienM beagles: pick is just taking the value on the left if possible otherwise on the righ
14:42 EmilienM https://github.com/puppetlabs/puppetlabs-stdlib/​blob/master/lib/puppet/parser/functions/pick.rb
14:42 beagles EmilienM, sure...
14:42 iurygregory morning
14:43 iberezovskiy hey
14:43 EmilienM beagles: is it clear?
14:43 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-ironic: Configure keystone authtoken options  https://review.openstack.org/346028
14:44 iurygregory denis is not around?
14:44 iberezovskiy he took day off
14:44 EmilienM lucky guy
14:44 moravec joined #puppet-openstack
14:45 iurygregory haha
14:45 beagles EmilienM, 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:46 iurygregory EmilienM, mwhahaha, iberezovskiy  fyi https://review.openstack.org/#/c/347105/ ;) authtoken class
14:46 beagles (defaults = treat as unset)
14:47 EmilienM beagles: what is invalid?
14:47 EmilienM iurygregory: yeah I saw it
14:48 EmilienM but didn't review it, saw WIP
14:48 EmilienM beagles: if a string or array is passed, it's good value
14:48 iurygregory EmilienM, lol sorry i just let the old commit message =(
14:48 iberezovskiy iurygregory, hm. we should do the same for all modules, shoudn't we?
14:48 EmilienM iberezovskiy: wait
14:49 EmilienM we wait for one module to be merge
14:49 iurygregory yeah
14:49 EmilienM so we are consistent everywhere
14:49 iberezovskiy sure, that's why I'm asking :)
14:49 dfisher chem|off: ping
14:49 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-barbican: Move barbican to authtoken  https://review.openstack.org/347105
14:50 chem dfisher: hi, I've seen your comment, why not use the same code as in the function with Puppet::Util:Exec.execute  ?
14:50 iurygregory EmilienM, not WIP anymore hahah
14:50 beagles EmilienM, 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 pi
14:50 beagles ck thought was undefined
14:51 dfisher chem: a straight execute will need the logic to look in the user's env, $HOME/openrc, or a supplied env hash … right?
14:51 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-ironic: Configure keystone authtoken options  https://review.openstack.org/346028
14:51 beagles EmilienM, if no value was supplied it would effectively be no-op
14:52 EmilienM beagles: exactly
14:52 EmilienM beagles: the value is not set in nova.conf
14:52 EmilienM and we rely on the default in Nova
14:53 beagles EmilienM, 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 effect
14:53 EmilienM beagles: right
14:54 beagles EmilienM, okay cool, thanks for the clarification :)
14:56 openstackgerrit Emilien Macchi proposed openstack/puppet-modulesync-configs: WIP - Add CI scripts  https://review.openstack.org/344916
14:56 EmilienM beagles: my pleasure
15:01 alee EmilienM, hmm something strange happened in last run ..
15:02 alee EmilienM, looks like tempest_roles was not set correctly ..
15:03 iurygregory dmsimard, the mascot should represent the component and the team =)
15:04 dmsimard iurygregory: so #puppet-openstack howls all the time or something ? I don't know
15:05 iurygregory dmsimard, not in the context lol like we work hard and i put wolf =P
15:05 dmsimard SO IT'S YOU
15:05 iurygregory yeah
15:05 iurygregory :p
15:06 dmsimard sorry, I'm an honest guy and I don't see the relationship between wolf and puppet ;(
15:06 EmilienM me neither :)
15:06 iurygregory dmsimard, no problem lol
15:06 dmsimard openstack-infra had some interesting choices
15:06 dmsimard they put up their vote recently
15:07 alee EmilienM, any idea what might have gone on here ? http://logs.openstack.org/06/339806/41/check/gat​e-puppet-openstack-integration-3-scenario002-tem​pest-centos-7/c921e54/logs/tempest.conf.txt.gz
15:07 alee EmilienM, to tempest_roles ..
15:10 alee EmilienM, maybe some change to puppet-tempest merged that broke paring of lists?
15:10 alee parsing ..
15:13 EmilienM alee_afk: weird
15:13 chem dfisher: sorry was in a meeting
15:13 dfisher chem: no worries - take your time
15:14 chem dfisher: hum ... how did you do it in the function call ?
15:14 dfisher https://review.openstack.org/#/c/330623/7/lib/pup​pet/parser/functions/openstack_resource_id_getter.rb
15:14 dfisher exactly that.
15:15 dfisher if argv.size == 1:  look at user's env or $HOME/openrc
15:15 dfisher if argv.size == 2:  use the supplied env hash
15:17 iurygregory it's not my fault but other people put +1 in wolf lol
15:21 chem dfisher: 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 dfisher heh.  ok :)
15:21 dfisher shell out to erlang!  go!
15:22 openstackgerrit Simon Murray proposed openstack/puppet-horizon: Allow Optional Password Auto-Completion  https://review.openstack.org/347803
15:29 openstackgerrit Merged openstack/puppet-octavia: Adding housekeeping.pp  https://review.openstack.org/347527
15:30 bklei_ joined #puppet-openstack
15:35 ntpttr__ joined #puppet-openstack
15:36 alee_afk EmilienM, yeah
15:38 alee_afk EmilienM, I'll take out the depends-on - because that already merged - and retest?
15:38 EmilienM alee_afk: ok
15:41 openstackgerrit Ade Lee proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/339806
15:42 openstackgerrit Merged openstack/puppet-cinder: SolidFire backend deployment should use $::os_service_default  https://review.openstack.org/347066
15:44 guimaluf joined #puppet-openstack
15:45 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-neutron: WIP: transform name to uuid.  https://review.openstack.org/347468
15:45 chem dfisher: 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 dfisher ok
15:46 openstackgerrit Merged openstack/puppet-neutron: Remove router_id option  https://review.openstack.org/340639
15:51 dfisher ^ remove router_id.  lol.
15:51 dfisher all the work i'm doing is to set that damn variable.
15:51 openstackgerrit Merged openstack/puppet-neutron: Deprecate min_l3_agents_per_router option  https://review.openstack.org/340637
15:51 openstackgerrit Merged openstack/puppet-nova: Add MultiStrOpt for scheduler_available_filters  https://review.openstack.org/329991
15:52 openstackgerrit Merged openstack/puppet-nova: Fix typo in tests  https://review.openstack.org/347598
15:53 dfisher chem: looks straight-forward enough.  gimme a few to test it
15:55 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class and configure cache correctly.  https://review.openstack.org/328347
15:57 openstackgerrit Merged openstack/puppet-neutron: Changing ovn plugin to ml2 mechanism driver  https://review.openstack.org/340343
16:11 dfisher chem:  it's very close.  it's currently dying because it can't find the neutron method:  https://github.com/openstack/puppet-neutron/b​lob/master/lib/puppet/provider/neutron.rb#L96
16:12 ducttape_ joined #puppet-openstack
16:12 dfisher i don't even know where that's set ...
16:13 dfisher ah, i see.  it's set in the providers that inherit from this provider
16:19 dfisher so, 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:23 chem dfisher: 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 dfisher ok.  poking ...
16:24 chem dfisher: so at least we won't have to redo the auth ourself
16:24 dfisher my ruby class internals knowledge is slightly above absolute zero
16:26 chem dfisher: 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:28 dfisher still getting
16:28 dfisher Error: /Stage[main]/Main/Neutron_l3_ag​ent_config[DEFAULT/router_id]: Could not evaluate: undefined method `neutron' for Puppet::Provider::Neutron:Class
16:29 dfisher http://paste.openstack.org/show/542624/
16:29 dfisher not sure if I declared that correctly
16:30 openstackgerrit Emilien Macchi proposed openstack/puppet-modulesync-configs: WIP - Add CI scripts  https://review.openstack.org/344916
16:30 rwsu joined #puppet-openstack
16:31 xarses joined #puppet-openstack
16:38 chem dfisher: oups, sorry, you have to define the function in the right class... -> http://paste.openstack.org/show/542627/
16:39 dfisher is that in neutron_l3_agent_config/ini_settings.rb ?
16:39 chem dfisher: if that doesn't work, I will take some time to explore why we don't have access to the neutron command
16:41 chem dfisher: 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 minute
16:41 dfisher ok
16:42 guimaluf hey 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:43 alee_afk EmilienM, same behavior -- something changed :/
16:44 alee_afk EmilienM, going to dinner -- will check again later -- but for some reason the tempest_roles are not being set correctly
16:46 EmilienM alee_afk: ok
16:55 bklei joined #puppet-openstack
17:00 chem dfisher: all in all it's absurd that 'commands :neutron => "neutron"' is not in neutron.rb
17:01 dfisher i tried adding it there too and that didn't work either :(
17:01 chem dfisher: try adding 'initvars' just before it in neutron.rb
17:02 chem dfisher: that's what is done in openstacklib/**/openstack.rb
17:03 dfisher chem: http://paste.openstack.org/show/542666/ ?
17:03 chem dfisher: yep
17:03 dfisher ok, that didn't error … didn't work but I'll track it down further...
17:05 chem dfisher: do you see the commands triggered ?
17:05 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/347956
17:06 dfisher yes.  the value passed to 'to_uuid' isn't showing up in the command though.
17:06 dfisher trying to figure that out ...
17:07 chem dfisher: can you try to hardcode it to check that the stuff really works, with authent and all ?
17:07 dfisher sure, one sec.
17:08 dfisher it does end with:
17:08 dfisher Error: /Stage[main]/Main/Neutron_l3_ag​ent_config[DEFAULT/router_id]: Could not evaluate: undefined method `cleanup_csv_with_id' for #<Puppet::Type::Neutron_l3_agent_config​::ProviderIni_setting:0x00000002cf8c10>
17:09 chem hum remove the method for now, it's not that important, we'll get back to this later
17:10 dfisher i will.  still trying to get the hardcoded stuff to work.  hold on a tick...
17:20 dfisher chem: http://paste.openstack.org/show/542671/
17:20 dfisher i don't understand what it's doing…
17:20 dfisher i dumped out the authenv before running neutron(…) and it's right
17:21 dfisher i need to run off to lunch however.  will be back in about an hour or so.
17:29 chem dfisher: 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 dfisher no, it's neutron router-show <routername> —format —column
17:29 dfisher or whatever
17:29 dfisher but yeah.  it's something like that.
17:29 dfisher just need to figure it out.
17:30 dfisher will poke when I get nback.
17:31 guimaluf joined #puppet-openstack
17:34 openstackgerrit Merged openstack/puppet-tripleo: Unit tests for HAproxy dual stack  https://review.openstack.org/346958
17:51 openstackgerrit Emilien Macchi proposed openstack/puppet-modulesync-configs: WIP - Add CI scripts  https://review.openstack.org/344916
17:51 chem dfisher: what I meant is that I think you passed a String instead of an Array
17:51 chem dfisher: not sure I was clear :)
18:12 EmilienM I never saw a duplicated resource when using ensure_resource
18:12 EmilienM http://logs.openstack.org/47/328347/20/check/ga​te-tripleo-ci-centos-7-nonha-multinode-nv/bbfec​4b/logs/postci.txt.gz#_2016-07-27_16_49_42_000
18:12 EmilienM if your resource don't have the same tag when declared multiple times, is it a big deal?
18:14 openstackgerrit Emilien Macchi proposed openstack/puppet-oslo: cache: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/347980
18:15 openstackgerrit Emilien Macchi proposed openstack/puppet-horizon: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/347981
18:16 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/347982
18:17 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class and configure cache correctly.  https://review.openstack.org/328347
18:22 iurygregory EmilienM, a while a go we have this problem i think see https://review.openstack.org/#/c/327185/
18:23 EmilienM cool
18:23 EmilienM iurygregory: nice catch
18:24 iurygregory EmilienM,  the guy maybe have proposed the same 3 patches lol
18:24 iurygregory maybe we should add abandon since is not updated =)
18:25 iurygregory EmilienM, we should block this patch I think since auth_version is not deprecated https://review.openstack.org/#/c/336328/
18:25 EmilienM I need to eat
18:25 iurygregory go o/
18:52 ntpttr__ joined #puppet-openstack
19:01 dfisher joined #puppet-openstack
19:03 dfisher1 joined #puppet-openstack
19:03 dfisher1 chem: ah, ok.  back from lunch and working on this some more
19:06 chem dfisher1: thanks a lot, I really don't have the time to help more ...
19:06 dfisher1 it's cool.  i'll go from here.
19:06 dfisher1 thanks for the help so far.
19:06 dfisher1 will update gerrit when I get somewhere
19:09 haplo37__ joined #puppet-openstack
19:11 dfisher1 chem: Warning: Neutron_l3_agent_config[DEFAULT/r​outer_id](provider=ini_setting): DATA is 0c1fc0ac-b82d-412d-9772-ecc4a108a923
19:11 dfisher1 got it.  ;)
19:11 dfisher1 now to clean up everything.
19:11 chem dfisher1: \o/ !!
19:11 chem dfisher1: this is supper good news.
19:12 dfisher1 thanks to you!
19:20 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-neutron: Put neutron provider commands in neutron.rb  https://review.openstack.org/348007
19:21 chem dfisher1: 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:25 dfisher1 i will absolutely do all of that.
19:26 chem dfisher1: ack, thanks
19:28 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: don't hardcode keystone-package tag to memcached package  https://review.openstack.org/347982
19:33 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class and configure cache correctly.  https://review.openstack.org/328347
19:35 dfisher1 chem: really really dumb question for you
19:35 chem dfisher1: no such thing as dumb question :)
19:36 dfisher1 chem: https://review.openstack.org/#/c/347463/2/lib​/puppet/provider/openstack_config/ini_setting.rb
19:36 chem dfisher1: (well ... most of the time..)
19:36 dfisher1 we're tweaking setting
19:36 dfisher1 shouldn't we be tweaking value?
19:36 alee_afk joined #puppet-openstack
19:36 dfisher1 or value= ?
19:37 dfisher1 or should I update @resource[:value] to what to_uuid() returns?
19:38 chem dfisher1: humm ... let me have a look this was a bit to fast I think.
19:38 * dfisher1 nods
19:40 chem dfisher1: oki, this is wrong, let me do another "template" for ya
19:41 dfisher1 ok
19:41 dfisher1 you can just paste it.   i'll take it from there.
19:50 ntpttr- joined #puppet-openstack
19:53 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-openstacklib: WIP: add a way for provider to transform value in config.  https://review.openstack.org/347463
19:53 chem dfisher1: ^ this is the proper version ...
19:53 * dfisher1 nods
19:54 chem dfisher1: so you have to implement the to_uuid function as well so we've got idempotency
19:54 dfisher1 ok.
19:54 dfisher1 will do
19:54 dfisher1 chem: thanks again!  (i'm sure I'll be back before too long with some other hairball :) )
19:56 chem dfisher1: 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 dfisher1 amen.
19:56 dfisher1 pints in Barca ;)
19:56 dfisher1 or, sangria
19:57 chem dfisher1: won't be there unfortunatly ...
19:57 dfisher1 doh!
19:58 iurygregory try the TSP to go =)
20:09 openstackgerrit Guilherme Balzana proposed openstack/puppet-monasca: [WIP] Provides an example on how to use this module  https://review.openstack.org/346218
20:16 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-neutron: Put neutron provider commands in neutron.rb  https://review.openstack.org/348007
20:20 ntpttr__ joined #puppet-openstack
20:20 ericksonsantos joined #puppet-openstack
20:23 guimaluf joined #puppet-openstack
20:25 openstackgerrit Merged openstack/puppet-ironic: Allow to configure [ssh] section in ironic config  https://review.openstack.org/347449
20:30 dfisher1 chem: it's *so* close.
20:31 chem dfisher1: what's wrong ?
20:31 dfisher1 the first run sets router_id=<router_name>
20:31 dfisher1 if I run it again with no changes it sets it to router_id=<router_uuid>
20:31 dfisher1 i can't figure out
20:31 dfisher1 why
20:32 dfisher1 what kind of logs can I get you?
20:34 dfisher1 chem: http://paste.openstack.org/show/542690/
20:35 chem dfisher1: hum ... you should commit all your modifications to gerrit as they are so that there is no confusion about the current state of the code
20:35 dfisher1 will do.
20:35 dfisher1 give me a moment.
20:36 ntpttr__ joined #puppet-openstack
20:37 chem dfisher1: you should use "puppet apply -d test.pp" so that we see more information
20:37 dfisher1 i can do that too.
20:38 chem dfisher1: and we should rename transform_to to transformation ... it reads better.
20:40 openstackgerrit Drew Fisher proposed openstack/puppet-openstacklib: WIP: add a way for provider to transform value in config.  https://review.openstack.org/347463
20:41 dfisher1 doing neutron one now
20:44 openstackgerrit Drew Fisher proposed openstack/puppet-neutron: WIP: transform name to uuid.  https://review.openstack.org/347468
20:45 dfisher1 chem: both uploaded
20:45 chem dfisher1: ack
20:51 cdelatte joined #puppet-openstack
20:52 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Update Jewel Ceph RGW to support Keystone V3  https://review.openstack.org/347956
20:55 openstackgerrit Michele Baldessari proposed openstack/puppet-tripleo: Create role for the fake openstack-core resource  https://review.openstack.org/347315
20:55 openstackgerrit Michele Baldessari proposed openstack/puppet-tripleo: Remove global openstack-core resource  https://review.openstack.org/347472
20:55 openstackgerrit Michele Baldessari proposed openstack/puppet-tripleo: Next generation HA architecture work  https://review.openstack.org/342650
20:55 openstackgerrit Michele Baldessari proposed openstack/puppet-tripleo: Move nova constraints from controller manifest to each service  https://review.openstack.org/347309
20:55 ayoung joined #puppet-openstack
20:57 chem dfisher1: 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 dfisher1 chem: sounds good.  have a good evening!
20:57 ducttape_ joined #puppet-openstack
20:58 chem dfisher1: well ... it's more like good night :)
20:58 dfisher1 or that!
21:01 openstackgerrit Drew Fisher proposed openstack/puppet-neutron: WIP: transform name to uuid.  https://review.openstack.org/347468
21:11 ntpttr__ joined #puppet-openstack
21:13 moravec joined #puppet-openstack
21:28 dgurtner joined #puppet-openstack
21:28 openstackgerrit Merged openstack/puppet-keystone: Fix typo in resource test  https://review.openstack.org/347619
21:52 openstackgerrit Merged openstack/puppet-cinder: Deprecate drivers in Cinder.conf DEFAULT section.  https://review.openstack.org/346531
22:08 bklei joined #puppet-openstack
22:24 moravec joined #puppet-openstack
23:37 ducttape_ joined #puppet-openstack
23:46 bklei joined #puppet-openstack

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary