Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-03-23

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

All times shown according to UTC.

Time Nick Message
00:08 ansmith joined #puppet-openstack
00:34 openstackgerrit Merged openstack/puppet-openstack-integration: Round 1 of Puppet 4 support  https://review.openstack.org/294838
00:34 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/296102
00:45 brandon joined #puppet-openstack
00:59 EmilienM dmsimard: it's also failing on ubuntu.
01:02 EmilienM dmsimard: I'm working on the fix right now
01:05 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/296133
01:05 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/296133
01:05 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/296102
01:10 ansmith joined #puppet-openstack
01:20 openstackgerrit Cody Herriges proposed openstack/puppet-gnocchi: Fix typo in database classes.  https://review.openstack.org/296135
01:22 openstackgerrit Cody Herriges proposed openstack/puppet-gnocchi: Fix typo in database classes.  https://review.openstack.org/296135
01:23 EmilienM _ody: nice catch
01:28 EmilienM _ody: I think we broke all beaker jobs
01:28 EmilienM yeah we did, I'm sending a fix
01:30 _ody EmilienM: Those need to be locked to Puppet 3 until after we finish getting p-o-i working properly.
01:30 _ody The work is similar though.
01:31 johnmilton joined #puppet-openstack
01:32 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: install_modules: export PUPPET_BASE_PATH  https://review.openstack.org/296139
01:32 EmilienM _ody: ^
01:32 ayoung joined #puppet-openstack
01:33 ducttape_ joined #puppet-openstack
01:34 EmilienM _ody: I'll quickly merge this one, is it fine?
01:40 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/296133
01:43 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/296102
01:43 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/296133
02:03 xarses joined #puppet-openstack
02:03 haplo37 joined #puppet-openstack
02:11 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/296133
02:12 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: do-not-merge - puppet4 tests  https://review.openstack.org/296015
02:14 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/296102
02:39 haplo37 joined #puppet-openstack
02:43 openstackgerrit Merged openstack/puppet-openstack-integration: install_modules: export PUPPET_BASE_PATH  https://review.openstack.org/296139
02:47 ilbot3 joined #puppet-openstack
02:47 Topic for #puppet-openstack is now Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
02:48 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/296133
02:59 baker joined #puppet-openstack
03:01 baker_ joined #puppet-openstack
03:08 ducttape_ joined #puppet-openstack
03:33 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added keystone config to barbican api manifest  https://review.openstack.org/263461
03:33 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module  https://review.openstack.org/263462
03:33 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added Dogtag settings for barbican-api  https://review.openstack.org/258851
03:33 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Add barbican client class and params  https://review.openstack.org/296156
03:33 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Moved logging to barbican-api module  https://review.openstack.org/296157
03:33 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Add helper files to configure paste file and barbican.conf  https://review.openstack.org/296158
03:33 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Add barbican-api manifest  https://review.openstack.org/296159
03:33 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Add quota and secret limit parameters to barbican-api manifest  https://review.openstack.org/296160
03:33 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Add keystone notification options to barbican-api manifest  https://review.openstack.org/296161
03:33 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added settings for cert and secret store plugins  https://review.openstack.org/296162
03:33 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added KMIP plugin settings to barbican-api manifest  https://review.openstack.org/296163
03:45 ducttape_ joined #puppet-openstack
03:59 markvoelker joined #puppet-openstack
04:03 baker joined #puppet-openstack
04:38 mjblack joined #puppet-openstack
04:51 fedexo joined #puppet-openstack
05:06 openstackgerrit Marcus Furlong proposed openstack/puppet-barbican: minor doc fixes  https://review.openstack.org/296179
05:28 zerda joined #puppet-openstack
05:43 aimon joined #puppet-openstack
06:00 markvoelker joined #puppet-openstack
07:01 markvoelker joined #puppet-openstack
07:10 gfidente joined #puppet-openstack
07:32 aimon joined #puppet-openstack
07:41 tesseract joined #puppet-openstack
08:00 cwolferh joined #puppet-openstack
08:06 jaosorior joined #puppet-openstack
08:24 hjensas joined #puppet-openstack
08:26 chem joined #puppet-openstack
08:37 paramite joined #puppet-openstack
08:40 jmikolajczak joined #puppet-openstack
08:43 permalac joined #puppet-openstack
09:03 markvoelker joined #puppet-openstack
09:04 openstackgerrit joined #puppet-openstack
09:09 karimb joined #puppet-openstack
09:19 derekh joined #puppet-openstack
09:20 aimon joined #puppet-openstack
09:20 foxhunt_away joined #puppet-openstack
09:34 yolanda joined #puppet-openstack
10:14 aimon joined #puppet-openstack
10:19 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-oslo: Improve checks for rabbit host, port and hosts parameters  https://review.openstack.org/293964
10:24 bookwar joined #puppet-openstack
10:25 tore_ joined #puppet-openstack
10:29 tore_ it doesn't seem to be possible to set dnsmasq_dns_servers via neutron::agents::dhcp - is this intentional? any other way to ensure there are dns service provided by default to tenant subnets using puppet modules only?
10:35 jaosorior joined #puppet-openstack
10:38 openstackgerrit Giulio Fidente proposed openstack/puppet-tripleo: Allow the Redis specific monitor to use authentication  https://review.openstack.org/295937
10:44 furlongm_ joined #puppet-openstack
10:55 furlongm joined #puppet-openstack
11:01 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: WIP: multi-domain testing with ldap backend.  https://review.openstack.org/296370
11:04 markvoelker joined #puppet-openstack
11:17 karimb joined #puppet-openstack
11:27 johnmilton joined #puppet-openstack
11:38 EmilienM hello
11:44 degorenko hey  :)
11:44 skolekonov EmilienM, hi! Got a bug in puppet-neutron https://bugs.launchpad.net/puppet-neutron/+bug/1560850 . Can we fix it for Mitaka, what do you think? It looks like an important one
11:44 openstack Launchpad bug 1560850 in puppet-neutron "project_name should be set to tenant_name if the latter is used" [Undecided,Confirmed] - Assigned to Jerzy Mikolajczak (jmikolajczak-b)
11:48 dgurtner joined #puppet-openstack
11:48 dgurtner joined #puppet-openstack
11:50 haplo37 joined #puppet-openstack
11:53 openstackgerrit Denis Egorenko proposed openstack/puppet-glance: Add missed glare_config options for glance::config class  https://review.openstack.org/296390
11:59 EmilienM skolekonov: I set High to the bug, so yes
11:59 EmilienM skolekonov: and even if it lands after the release we would have backported it.
12:00 EmilienM I started this: https://wiki.openstack.org/wiki/Puppet/Backport_policy
12:01 skolekonov ok, thanks
12:03 openstackgerrit Miles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/276187
12:07 dprince joined #puppet-openstack
12:20 markvoelker joined #puppet-openstack
12:23 EmilienM RDO servers had some outage recently, do recheck if you see repos issue in logs
12:40 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-oslo: Improve checks for rabbit host, port and hosts parameters  https://review.openstack.org/293964
12:49 morazi joined #puppet-openstack
12:49 openstackgerrit Denis Egorenko proposed openstack/puppet-glance: Add missed glare_config options for glance::config class  https://review.openstack.org/296390
12:50 degorenko EmilienM, ^^ not sure, about reno notes, do we need that for such patch?
12:50 EmilienM degorenko: no need!
12:50 degorenko ok
12:50 degorenko one typo in patch
12:50 openstackgerrit Denis Egorenko proposed openstack/puppet-glance: Add missed glare_config options for glance::config class  https://review.openstack.org/296390
12:51 degorenko and so many red spaces....
12:51 degorenko :)
12:52 EmilienM degorenko: did you see my review?
12:52 degorenko EmilienM, which one? I'm just back from lunch
12:53 EmilienM degorenko: https://review.openstack.org/#/c/296390/
12:53 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-heat: Use identity_uri and auth_uri by default  https://review.openstack.org/261326
12:53 degorenko EmilienM, i've added tests
12:53 EmilienM I don't see it o_O
12:53 degorenko old patch number in link?
12:54 degorenko it's already 3 patch sets here
12:54 degorenko and will be 4 :D
12:54 EmilienM oh you're about to push it, ok
12:54 degorenko EmilienM, tests already pushed
12:55 ducttape_ joined #puppet-openstack
12:55 dgurtner joined #puppet-openstack
12:55 dgurtner joined #puppet-openstack
12:55 EmilienM degorenko: maybe I have an issue but I don't see it in Gerrit
12:55 openstackgerrit Merged openstack/puppet-gnocchi: Fix typo in database classes.  https://review.openstack.org/296135
12:55 degorenko lol
12:55 degorenko EmilienM, https://review.openstack.org/#/c/296390/3/
12:55 degorenko try that
12:55 EmilienM using another browser, I can't see it neither
12:56 degorenko magic
12:56 EmilienM degorenko: still not
12:56 EmilienM degorenko: I meant acceptance tests btw
12:56 EmilienM you added rspec :-P
12:56 aimon joined #puppet-openstack
12:56 EmilienM which is super cool :-P
12:56 degorenko lol
12:56 degorenko :D
12:56 EmilienM spec/acceptance/glance_config_spec.rb
12:56 degorenko oh yeah
12:57 degorenko f.....
12:57 degorenko sorry :)
12:57 EmilienM :)
12:57 degorenko didn't get it correctly
12:57 EmilienM no problem!
12:59 openstackgerrit Merged openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/296133
13:01 EmilienM rdo servers are down again...
13:02 degorenko :(
13:02 openstackgerrit Denis Egorenko proposed openstack/puppet-glance: Add missed glare_config options for glance::config class  https://review.openstack.org/296390
13:03 degorenko EmilienM, should be better now :) ^
13:04 EmilienM degorenko: thx
13:08 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-heat: Use identity_uri and auth_uri by default  https://review.openstack.org/261326
13:14 openstackgerrit Merged openstack/puppet-nova: Handle config_drive_format option  https://review.openstack.org/295234
13:18 richm joined #puppet-openstack
13:27 moravec joined #puppet-openstack
13:33 ducttape_ joined #puppet-openstack
13:39 openstackgerrit Merged openstack/puppet-magnum: Add acceptance test  https://review.openstack.org/294200
13:55 degorenko EmilienM, can you please take a look on this one: https://review.openstack.org/#/c/261326/ :)
13:55 EmilienM #ŧopic trunk.rdoproject.org is DOWN ci jobs failing - Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
13:55 EmilienM fail
13:56 aimon joined #puppet-openstack
13:56 EmilienM degorenko: yes
13:56 degorenko thanks :)
13:57 haplo37 joined #puppet-openstack
13:57 EmilienM degorenko: I'll look later
13:59 degorenko sure
14:00 dhollinger joined #puppet-openstack
14:01 dmsimard EmilienM: is it down again ? looks up to me
14:01 EmilienM dmsimard: ok cool
14:02 spredzy joined #puppet-openstack
14:07 dmsimard EmilienM: puppet-neutron has no stable/mitaka branch
14:07 dmsimard I wanted to backport it
14:07 dmsimard I guess it hasn't been cut yet ?
14:07 dmsimard also, KB was saying he was hitting a similar issue on liberty
14:07 dmsimard I'm testing right now
14:09 baker joined #puppet-openstack
14:09 EmilienM dmsimard: not yet
14:09 EmilienM dmsimard: go ahead yeah
14:09 EmilienM err
14:10 EmilienM in liberty?
14:10 permalac joined #puppet-openstack
14:10 dmsimard EmilienM: yeah I'm going to try and reproduce
14:10 EmilienM we don't have stable/mitaka right now
14:10 EmilienM I'm working on it this week
14:10 EmilienM only aodh is cut
14:10 EmilienM because I wanted to try the release workflow
14:10 vinsh joined #puppet-openstack
14:16 xarses joined #puppet-openstack
14:17 dmsimard EmilienM: it's running now but I think it's wrong even before trying
14:17 dmsimard EmilienM: https://github.com/openstack-packages/neutron/blob/rpm-liberty/openstack-neutron.spec#L546 vs https://github.com/openstack/puppet-neutron/blob/stable/liberty/lib/puppet/provider/neutron_plugin_linuxbridge/ini_setting.rb
14:18 EmilienM so no need to backport it in liberty iiuc
14:18 dmsimard so no one has used linuxbridge with rdo with puppet-neutron basically
14:18 mtreinish_ joined #puppet-openstack
14:18 dmsimard I don't know if the paths are the same
14:19 dmsimard checking
14:19 alee joined #puppet-openstack
14:20 alee degorenko, so continuing ..
14:20 degorenko yep, what're questions you want ask ? :)
14:21 dmsimard EmilienM: eh I don't know, trying to figure out between the agent and the plugin provider
14:21 aimon joined #puppet-openstack
14:21 alee degorenko, there are a few places where I was asked to use $os_system_default or not to do so ..
14:21 EmilienM dmsimard: plugin is always same file, only agent changes
14:21 alee degorenko, can you tell me when to use it and when not to?
14:22 dmsimard EmilienM: so https://github.com/openstack/puppet-neutron/blob/stable/liberty/lib/puppet/provider/neutron_agent_linuxbridge/ini_setting.rb should be /plugins/ml2/linuxbridge_agent.ini then
14:22 dmsimard EmilienM: which is the same as Mitaka
14:22 degorenko alee, ok, so, for os_service_default is going to be used when your defaults for puppet parameter == defaults from upstream. In this case you can use as puppet default os_service_default
14:22 ansmith joined #puppet-openstack
14:23 dmsimard EmilienM: it's not a clean backport rebase due to that Ubuntu hack but I'll take care of it
14:23 alee degorenko, ok gotcha ..
14:23 dmsimard EmilienM: will we have a linuxbridge test in integration for stable/liberty though ?
14:23 degorenko alee, probably you can use it in most of your patches
14:23 openstack joined #puppet-openstack
14:24 alee degorenko, yup -- ok, in  https://review.openstack.org/#/c/296157/1/manifests/logging.pp
14:24 alee yo mention what needs to be done for deprecation ..
14:24 degorenko yes, this is exactly when we can use it
14:24 degorenko same for db class, if such present
14:24 EmilienM dmsimard: I don't think so
14:25 alee degorenko, does it matter that it is extremely unlikely that anyone was using the class I removed?  Given that the module actually did nothing before my set of patches?
14:26 degorenko alee, do we have branches for barbican?
14:26 degorenko no, we don't
14:26 alee degorenko, prob not yet ..
14:27 degorenko alee, then, since puppet-barbican was not released ever - you can simply remove that, yes, i forgot about that. This module wasn't ever released
14:27 alee degorenko, ok great
14:28 alee degorenko, in https://review.openstack.org/#/c/296157/1/manifests/api/logging.pp , there is a bunch of "pick" code there
14:29 degorenko alee, in this case we don't need that
14:29 alee degorenko, I think what you guys are saying is that I can just eliminate that code ..
14:29 degorenko alee, indeed
14:29 alee degorenko, and then those parameters would not be in the barbican::api manifest at all?
14:29 degorenko yep
14:29 alee cool
14:29 degorenko so, you can add include for this class in api
14:30 alee degorenko, yes - will do
14:30 mfisch iurygregory: we need some unit tests for the ldap_backend class
14:30 degorenko and when you need override such parameters in non-hiera way you can simply declare this class with needed parameters
14:31 degorenko it will not raise duplication error, because include is not actual declaring
14:31 alee degorenko, in https://review.openstack.org/#/c/296158/1/manifests/config.pp, you make comment about removing old config
14:31 iurygregory mfisch ?
14:31 degorenko alee, example for logging and database for a new classes https://review.openstack.org/#/c/293524/
14:32 alee degorenko, thanks - a new example definitely helps :)
14:32 degorenko alee, yes, what's wrong with old barbican_config?
14:32 mfisch iurygregory: didnt you write the ldap_backend class in keystone?
14:32 iurygregory nops =P
14:33 alee degorenko, well - I renamed it to api_config --  the reason is that eventually I will be adding support for barbican -worker processes
14:33 alee which will have their own logging
14:33 iurygregory mfisch, I just said something about ldap at the meeting yesterday
14:33 alee and will be worker_config most likely
14:33 degorenko alee, i don't see patch, where you renamed current provider
14:33 mfisch iurygregory: k ;)
14:34 mfisch ah chem  wrote it
14:34 mfisch sorry iurygregory
14:34 iurygregory maybe i can help with review =)
14:34 iurygregory no problem mfisch =D
14:34 alee degorenko, not sure I understand -- why can I not rename barbican_config to api_config ?
14:34 chem mfisch: what's up ?
14:35 degorenko alee, oh, now i got it
14:35 alee degorenko, ok - so no objection then ?
14:36 degorenko alee, yes, no objection here. Just add new tests for new providers :)
14:36 alee degorenko, will do - thanks :)
14:36 degorenko alee, example is in patch above too :)
14:36 alee degorenko, excellent :)
14:37 degorenko alee, also you can see here: https://review.openstack.org/#/c/296390/ this is rspec tests
14:37 openstackgerrit Matt Fischer proposed openstack/puppet-keystone: Add the ability to control LDAP package mgmt  https://review.openstack.org/296493
14:37 alee degorenko, is there an example of the usage of ensure_resources() too ?
14:37 degorenko alee, hmmm
14:38 alee degorenko, I can play with it if not .. shouldn't be too hard to figure out.
14:38 degorenko alee, we have such examples, let me find them
14:38 degorenko alee, in which patch i dropped this comment?
14:39 alee https://review.openstack.org/#/c/296156/1
14:39 alee for one ..
14:39 degorenko alee, ah, ok, so
14:39 degorenko for example for keystone we have only include for openstackclient
14:40 degorenko not any calling class explicitly
14:40 alee degorenko, ok - I can do that :)
14:40 degorenko alee, ensure function, for example for package exactly: https://github.com/openstack/puppet-openstacklib/blob/master/manifests/openstackclient.pp#L14
14:41 degorenko if you want manage openstackclient - then you need to use such construction ^
14:41 degorenko but i'm not sure why you need that?
14:42 alee degorenko, yeah
14:43 alee degorenko, so maybe remove altogether?
14:43 degorenko alee, calling openstackclient?
14:44 openstackgerrit Dimitri Savineau proposed openstack/puppet-tripleo: Enable HAProxy forwardfor option for Horizon.  https://review.openstack.org/296497
14:44 alee right -- there is barbican stuff in the openstack client -- but I'm not calling it anywhere
14:44 alee alternatively I can do what keystone does and just include it
14:45 degorenko alee, second way is more better
14:45 alee degorenko, I suppose eventually we will be , so I 'll just include it like keystone does.
14:45 degorenko :)
14:47 mdorman joined #puppet-openstack
14:47 alee degorenko, any idea what the test for the client class should look like?
14:47 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-oslo: Fix values for rabbit_hosts  https://review.openstack.org/293964
14:48 degorenko alee, https://github.com/openstack/puppet-keystone/blob/master/spec/classes/keystone_client_spec.rb
14:49 alee degorenko, perfect thanks
14:50 alee degorenko, can you tell me what I need to change for https://review.openstack.org/#/c/296159/1/manifests/api.pp@346  ?
14:52 degorenko alee, it's question probably for other reviewers, but for you: something like https://review.openstack.org/#/c/292309/
14:54 alee degorenko, ok
14:54 mfisch chem: was thinking we should have some unit tests for ldap_backend in keystone
14:54 alee degorenko, https://review.openstack.org/#/c/296162/1/manifests/api.pp , there are some comments there about some code which I added that was commented out
14:55 chem mfisch: yep, I'm working on this, it would be ldap and multi-backend domain
14:55 mfisch k
14:55 alee degorenko, I plan to revisit that code with a follow up patch
14:55 chem mfisch: https://review.openstack.org/#/c/296370/
14:55 alee after this set lands
14:55 mfisch chem: I pushed up a small change for the LDAP code this rmoning
14:55 chem mfisch: yep, I've seen
14:56 degorenko alee, yep, that's because this code will not work for puppet < 4
14:56 chem mfisch: we should remove the duplicate code, now.  At the time it was to avoid unforseen restart of keystone, but I think now that it was not necessary
14:56 chem mfisch: class should only use the define ... but that's another patch :0
14:56 chem :)
14:56 mfisch which define?
14:57 alee degorenko, right -- well at the time, we had no support for multicfg options - which its the original reason I commented it out.
14:58 chem mfisch: in ldap you have a class and a define: ldap is a class and ldap_backend is a define
14:58 alee degorenko, so can I just leave it there, or should I just remove the whole thing?
14:58 mfisch chem: ah
14:58 degorenko alee, then remove this block at all :)
14:58 mfisch chem: I totally missed that ;)
14:58 alee degorenko, ok
14:59 chem mfisch: it's nearly copy/paste code, it was to not have any restart of the keystone server, anyway this should be factor out.
14:59 chem mfisch: for the testing ldap, I hope to have something working before the end of the week.
15:00 alee degorenko, looks like we hit all my questions for now - thanks.  I 'll resubmit shortly.  The only question I have is whether I should use the new rabbit code/module ..
15:00 degorenko take a look here: https://review.openstack.org/#/c/292309/ it is in puppet-oslo module
15:00 degorenko we are planning to switch all modules to a new scheme
15:01 alee degorenko, ok --I'll try out the changes there.
15:06 mfisch chem: restarting keystone to add an LDAP backend, why is that a problme?
15:07 chem mfisch: it was for those who already had the ldap defined using the class.  Using the defined in the class would have triggered an restart of the keystone server even if the user didn't change a thing.
15:09 mfisch is not a huge issue for me
15:10 mfisch chem: im on board with you switching
15:10 mfisch having the code twice is terrible
15:11 chem mfisch: oki, I'll do that on top of your patch
15:13 openstackstatus joined #puppet-openstack
15:22 baker joined #puppet-openstack
15:33 ansmith joined #puppet-openstack
15:39 openstackgerrit Michael Polenchuk proposed openstack/puppet-nova: Autorequire nova security group  https://review.openstack.org/296548
15:44 jaosorior joined #puppet-openstack
15:51 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: (WIP) Explicit containment for more predictability  https://review.openstack.org/296557
16:14 EmilienM looking at https://review.openstack.org/#/c/296557/ - I'm really afraid how puppet4 is not really backward compatible with puppet3
16:14 EmilienM specially that code: https://review.openstack.org/#/c/296557/1/fixtures/scenario-aio.pp
16:14 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-heat: Use identity_uri and auth_uri by default  https://review.openstack.org/261326
16:16 karimb joined #puppet-openstack
16:17 EmilienM _ody: does puppetlabs expect puppet3 users to update their manifests ordering for puppet4 migration?
16:18 openstackgerrit Merged openstack/puppet-vswitch: Add Solaris support  https://review.openstack.org/296010
16:20 ansmith joined #puppet-openstack
16:26 _ody EmilienM: If you always used very explicit relationships, it shouldn't be a a lot of work.  That's not reality though, some relationships are bound to be missed if the cards just happened to fall in the right place.
16:26 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-oslo: Fix values for rabbit_hosts  https://review.openstack.org/293964
16:26 _ody In a lot of ways I think the old real random execution order was a better situation than the psuedo random things we had in Puppet 3, it made it easy to miss relationships.
16:29 _ody EmilienM: I don't know what the "official" word is about if we expect people to update ordering but I am told that the recommendation is to actually move to Puppet 3 + future parser first then migrate to Puppet 4.
16:30 _ody Doing that means you do the work in two stages.  Puppet 3 + future parser means you handle the language changes then when you go to Puppet 4 you handle the ordering changes.
16:31 EmilienM _ody: what do you suggest for Puppet OpenStack ?
16:32 _ody We go directly to Puppet 4.
16:32 EmilienM and what about out users that still have puppet3?
16:32 _ody Less work in my opinion.  Else you just touch all the code first.
16:32 _ody EmilienM: I mean just upgrade the code to be compatible both ways.
16:32 _ody It is sill possible.
16:32 EmilienM _ody: ok
16:33 EmilienM _ody: I think, from what I've seen in your patch, the only changes that are needed now is in our integration testing manifests, right?
16:33 EmilienM if yes, I'm happy with that now
16:33 _ody EmilienM: So far but I am still only in aio.
16:35 EmilienM ok
16:35 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/296102
16:43 openstackgerrit Denis Egorenko proposed openstack/puppet-cinder: Update Cinder BDD backend configuration  https://review.openstack.org/296606
16:47 rahulait joined #puppet-openstack
16:51 ducttape_ joined #puppet-openstack
16:53 cdelatte joined #puppet-openstack
16:56 karimb joined #puppet-openstack
16:56 ducttape_ joined #puppet-openstack
17:24 degorenko EmilienM, i've answered here https://review.openstack.org/296606
17:26 brandon joined #puppet-openstack
17:28 aimon joined #puppet-openstack
17:34 alee degorenko, in logging.pp --> $log_file should be set to '/var/log/barbican/api.log'  or just 'api.log'  as $log_dir is defined ?
17:34 tlbr joined #puppet-openstack
17:34 degorenko alee, first - full path
17:34 openstackgerrit Merged openstack/puppet-neutron: Add ensure_absent_val behavior to Nuage neutron plugin type  https://review.openstack.org/296043
17:36 alee degorenko, thanks
17:36 EmilienM degorenko: will look after lunch :)
17:37 degorenko EmilienM, sure
17:53 EmilienM degorenko: back :)
17:53 EmilienM degorenko: damn, I missed that param :(
17:53 degorenko no problem :)
17:54 degorenko EmilienM, also i will have one more patch for glare - i forgot to add endpoint :D
17:54 angdraug joined #puppet-openstack
17:54 EmilienM degorenko: it can help ;-)
17:54 vinsh joined #puppet-openstack
17:55 degorenko EmilienM, implementation of new service will be fully covered when you're deploying that in same time, fuel doesn't have such resources yet, i'm working on that, so, probably we will have not only one more patch :D
18:02 vinsh joined #puppet-openstack
18:15 openstackgerrit Denis Egorenko proposed openstack/puppet-glance: Configure endpoint for Glance Glare  https://review.openstack.org/296656
18:15 openstackgerrit Merged openstack/puppet-manila: Fix typos  https://review.openstack.org/294375
18:23 degorenko EmilienM, take a look on https://review.openstack.org/296606 it is failed units, but i have one solution and i need your opinion here :)
18:23 EmilienM looking
18:24 degorenko so, the problem is that ensure_ function can't use dependencies
18:24 EmilienM right
18:24 EmilienM you need to use a collector maybe
18:24 EmilienM Package['tgt' -> Service['tgtd']
18:24 EmilienM something like that
18:24 EmilienM maybe a notify?
18:25 degorenko well, that's not a problem with notify
18:25 degorenko we must be sure, that such resources are present
18:25 degorenko but in case of multibackend with iscsi
18:25 degorenko we will get duplication error
18:25 degorenko if they will be ran in one catalog
18:26 ericksonsantos joined #puppet-openstack
18:26 degorenko 1) can we pass require/notify to ensure_ injected to passed params?
18:27 degorenko 2) call iscsi backend from bdd with proper configs - we have almost identical options between them, bdd has a few more
18:27 degorenko mwhahaha, ^
18:27 iurygregory joined #puppet-openstack
18:30 EmilienM degorenko: I don't see why you can't use notify
18:30 EmilienM or is it 1)?
18:31 degorenko we can't use collectors here, because there is no guarantee that we have iscsi also
18:31 degorenko as another option, use !defined(<resource>)
18:31 degorenko in 'if' structure
18:32 EmilienM the collector is secure because catalog won't fail if resource is not here
18:32 EmilienM isn't?
18:32 degorenko but who will installed/run resources?
18:34 EmilienM mhh ok I see
18:34 EmilienM !defined(<resource>) maybe
18:34 openstack EmilienM: Error: "defined(<resource>)" is not a valid command.
18:34 EmilienM openstack: hey bro
18:37 degorenko EmilienM, actually, having hardcoded resources in iscsi is also problem - we can have a few iscsi backends on one node, with different backends. It also will raise duplication error
18:38 degorenko we can have iscsi1 and iscsi2 backends with different volumes_dir or whatever
18:38 degorenko it will have different sections in cinder.conf and error :(
18:39 EmilienM right, it's a bit tricky
18:39 degorenko it's already common situation for me - digging into one bug and discover new one x)
18:41 EmilienM ok I'm going to spam
18:42 degorenko :D branches are coming!
18:43 EmilienM I hope I don't mess up
18:44 EmilienM ok I messed up :P
18:49 angdraug joined #puppet-openstack
18:50 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Release 8.0.0  https://review.openstack.org/296669
18:50 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Release 8.0.0  https://review.openstack.org/296670
18:50 openstackgerrit Emilien Macchi proposed openstack/puppet-designate: Release 8.0.0  https://review.openstack.org/296671
18:50 openstackgerrit Emilien Macchi proposed openstack/puppet-vswitch: Release 4.0.0  https://review.openstack.org/296672
18:51 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: Release 8.0.0  https://review.openstack.org/296673
18:51 openstackgerrit Emilien Macchi proposed openstack/puppet-gnocchi: Release 8.0.0  https://review.openstack.org/296674
18:51 openstackgerrit Emilien Macchi proposed openstack/puppet-heat: Release 8.0.0  https://review.openstack.org/296675
18:51 openstackgerrit Emilien Macchi proposed openstack/puppet-horizon: Release 8.0.0  https://review.openstack.org/296676
18:51 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Release 8.0.0  https://review.openstack.org/296677
18:51 openstackgerrit Emilien Macchi proposed openstack/puppet-manila: Release 8.0.0  https://review.openstack.org/296678
18:51 openstackgerrit Emilien Macchi proposed openstack/puppet-mistral: Release 8.0.0  https://review.openstack.org/296679
18:51 openstackgerrit Emilien Macchi proposed openstack/puppet-murano: Release 8.0.0  https://review.openstack.org/296680
18:51 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Release 8.0.0  https://review.openstack.org/296681
18:52 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Release 8.0.0  https://review.openstack.org/296682
18:52 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: Release 8.0.0  https://review.openstack.org/296683
18:52 openstackgerrit Emilien Macchi proposed openstack/puppet-openstacklib: Release 8.0.0  https://review.openstack.org/296684
18:52 openstackgerrit Emilien Macchi proposed openstack/puppet-sahara: Release 8.0.0  https://review.openstack.org/296685
18:52 openstackgerrit Emilien Macchi proposed openstack/puppet-swift: Release 8.0.0  https://review.openstack.org/296686
18:52 openstackgerrit Emilien Macchi proposed openstack/puppet-tempest: Release 8.0.0  https://review.openstack.org/296687
18:52 openstackgerrit Emilien Macchi proposed openstack/puppet-trove: Release 8.0.0  https://review.openstack.org/296688
18:52 openstackgerrit Emilien Macchi proposed openstack/puppet-zaqar: Release 8.0.0  https://review.openstack.org/296689
18:52 dmsimard them stackalytics stats though
18:53 iurygregory joined #puppet-openstack
18:53 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Release 8.0.0  https://review.openstack.org/296681
18:53 ericksonsantos joined #puppet-openstack
18:53 EmilienM degorenko: ready for review: https://review.openstack.org/#/q/topic:puppet/mitaka+status:open
18:54 EmilienM dmsimard: yeah i'm doing release only for stats
18:54 dmsimard lol
18:55 patrickeast joined #puppet-openstack
18:55 dkaigarodsev joined #puppet-openstack
18:55 degorenko EmilienM, waitning for CI :)
18:55 EmilienM should be fast, only releasenotes & puppetlint will run
18:56 degorenko how is going 'Switch to ubuntu-trusty DIBS' ?
18:56 bapalm joined #puppet-openstack
18:56 EmilienM degorenko: still waiting for pabelanger to do the swift in os infra
18:56 degorenko ok :)
18:58 ericksonsantos joined #puppet-openstack
18:59 petems joined #puppet-openstack
18:59 iurygregory joined #puppet-openstack
19:00 baker joined #puppet-openstack
19:02 aimon joined #puppet-openstack
19:03 openstackgerrit Merged openstack/puppet-keystone: Add the ability to control LDAP package mgmt  https://review.openstack.org/296493
19:03 baker joined #puppet-openstack
19:05 openstackgerrit Denis Egorenko proposed openstack/puppet-cinder: Update Cinder BDD backend configuration  https://review.openstack.org/296606
19:05 degorenko EmilienM, now it will work ^ :-) i'm off for today
19:07 EmilienM _degorenko|afk: thx!
19:08 pma joined #puppet-openstack
19:08 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Prepare stable/mitaka branch  https://review.openstack.org/296702
19:10 dmsimard EmilienM: that review is going to be sitting there a while
19:10 dmsimard EmilienM: RDO GA is upstream + 1 week
19:10 dmsimard don't know about UCA
19:10 EmilienM I know
19:10 EmilienM no worries, I'm just preparing the patches
19:11 EmilienM dmsimard: and even if it takes too long, I can set the repos to current-passed-ci, wdyt?
19:12 dmsimard sure
19:12 kgalanov_ joined #puppet-openstack
19:14 EmilienM coolio
19:15 pleia2_ joined #puppet-openstack
19:16 zeus- joined #puppet-openstack
19:19 mjb joined #puppet-openstack
19:20 fc__ joined #puppet-openstack
19:22 openstack joined #puppet-openstack
19:24 openstackstatus joined #puppet-openstack
19:24 dtantsur joined #puppet-openstack
19:26 sbadia joined #puppet-openstack
19:34 zeus joined #puppet-openstack
19:36 EmilienM mwhahaha: if you have time today, I prepared releases for modules https://review.openstack.org/#/q/topic:puppet/mitaka+status:open
19:36 mwhahaha k
19:37 EmilienM the step after that will be to create branches and we're almost good
19:50 jmikolajczak joined #puppet-openstack
19:54 mwhahaha EmilienM: you've got a trailing space in the 8.0.0 version number in the metadata.json
19:55 EmilienM damn
19:55 EmilienM damn !
19:55 EmilienM everywhere I guess
19:55 mwhahaha so i could review and add to my stackalytics's profile ;) that's a lot of -1s
19:55 EmilienM I used sed
19:55 mwhahaha :D
19:55 EmilienM let me update... /me sigh
20:03 EmilienM ok let's try again
20:03 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Release 8.0.0  https://review.openstack.org/296722
20:03 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Release 8.0.0  https://review.openstack.org/296723
20:03 openstackgerrit Emilien Macchi proposed openstack/puppet-designate: Release 8.0.0  https://review.openstack.org/296724
20:03 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: Release 8.0.0  https://review.openstack.org/296725
20:03 openstackgerrit Emilien Macchi proposed openstack/puppet-gnocchi: Release 8.0.0  https://review.openstack.org/296726
20:03 openstackgerrit Emilien Macchi proposed openstack/puppet-heat: Release 8.0.0  https://review.openstack.org/296727
20:03 openstackgerrit Emilien Macchi proposed openstack/puppet-horizon: Release 8.0.0  https://review.openstack.org/296728
20:03 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Release 8.0.0  https://review.openstack.org/296729
20:03 openstackgerrit Emilien Macchi proposed openstack/puppet-manila: Release 8.0.0  https://review.openstack.org/296730
20:03 openstackgerrit Emilien Macchi proposed openstack/puppet-mistral: Release 8.0.0  https://review.openstack.org/296732
20:03 openstackgerrit Emilien Macchi proposed openstack/puppet-murano: Release 8.0.0  https://review.openstack.org/296733
20:03 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Release 8.0.0  https://review.openstack.org/296734
20:04 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Release 8.0.0  https://review.openstack.org/296735
20:04 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: Release 8.0.0  https://review.openstack.org/296736
20:04 openstackgerrit Emilien Macchi proposed openstack/puppet-openstacklib: Release 8.0.0  https://review.openstack.org/296737
20:04 openstackgerrit Emilien Macchi proposed openstack/puppet-sahara: Release 8.0.0  https://review.openstack.org/296738
20:04 openstackgerrit Emilien Macchi proposed openstack/puppet-swift: Release 8.0.0  https://review.openstack.org/296739
20:04 openstackgerrit Emilien Macchi proposed openstack/puppet-tempest: Release 8.0.0  https://review.openstack.org/296740
20:04 openstackgerrit Emilien Macchi proposed openstack/puppet-trove: Release 8.0.0  https://review.openstack.org/296741
20:04 openstackgerrit Emilien Macchi proposed openstack/puppet-zaqar: Release 8.0.0  https://review.openstack.org/296742
20:05 EmilienM mwhahaha: better? :P
20:05 mwhahaha let me see :D
20:05 EmilienM damn no
20:05 EmilienM no no
20:05 EmilienM I messed up again I should go sleep
20:05 mwhahaha oops
20:06 EmilienM ok this time is good
20:06 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Release 8.0.0  https://review.openstack.org/296744
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Release 8.0.0  https://review.openstack.org/296745
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-designate: Release 8.0.0  https://review.openstack.org/296746
20:07 EmilienM (I hope)
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: Release 8.0.0  https://review.openstack.org/296747
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-gnocchi: Release 8.0.0  https://review.openstack.org/296748
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-heat: Release 8.0.0  https://review.openstack.org/296749
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-horizon: Release 8.0.0  https://review.openstack.org/296750
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Release 8.0.0  https://review.openstack.org/296751
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-manila: Release 8.0.0  https://review.openstack.org/296752
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-mistral: Release 8.0.0  https://review.openstack.org/296753
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-murano: Release 8.0.0  https://review.openstack.org/296754
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Release 8.0.0  https://review.openstack.org/296755
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Release 8.0.0  https://review.openstack.org/296756
20:07 mwhahaha now it's in tripicate
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: Release 8.0.0  https://review.openstack.org/296757
20:07 mwhahaha triplicate
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-openstacklib: Release 8.0.0  https://review.openstack.org/296758
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-sahara: Release 8.0.0  https://review.openstack.org/296759
20:07 mwhahaha since you got new change-ids
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-swift: Release 8.0.0  https://review.openstack.org/296760
20:07 openstackgerrit Emilien Macchi proposed openstack/puppet-tempest: Release 8.0.0  https://review.openstack.org/296761
20:08 mwhahaha abandon time
20:08 openstackgerrit Emilien Macchi proposed openstack/puppet-trove: Release 8.0.0  https://review.openstack.org/296762
20:08 openstackgerrit Emilien Macchi proposed openstack/puppet-zaqar: Release 8.0.0  https://review.openstack.org/296763
20:09 mwhahaha EmilienM: you want to abandon the bad changes?
20:09 EmilienM wait
20:09 EmilienM I used amend
20:09 mwhahaha nope
20:10 EmilienM ok I go kill myself, I come back.
20:10 mwhahaha https://review.openstack.org/296669 https://review.openstack.org/296722 https://review.openstack.org/296744
20:10 mwhahaha :D
20:10 * EmilienM big facepalm
20:10 ansmith joined #puppet-openstack
20:10 mwhahaha it's one of those days
20:11 EmilienM mwhahaha: exactly. Let me abandon them
20:12 EmilienM my script used amend but was overriding commit message.
20:26 EmilienM mwhahaha: https://review.openstack.org/#/q/status:open+branch:master+topic:puppet/mitaka should be OK now.
20:27 mwhahaha ok
20:32 openstackgerrit Gonéri Le Bouder proposed openstack/puppet-ironic: expose the ipxe_enabled parameter  https://review.openstack.org/296774
20:33 openstack joined #puppet-openstack
20:41 rahulait joined #puppet-openstack
20:45 rahulait joined #puppet-openstack
20:50 angdraug joined #puppet-openstack
21:04 mwhahaha EmilienM: puppet-keystone Release 8.0.0?
21:04 EmilienM mwhahaha: looking
21:05 alee joined #puppet-openstack
21:05 EmilienM did I miss it?
21:05 mwhahaha i don't see it
21:05 mwhahaha which is why i asked :D
21:05 EmilienM I missed it
21:06 EmilienM it's 5pm, almost the end of this day.
21:08 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: Release 8.0.0  https://review.openstack.org/296802
21:09 EmilienM mwhahaha: vswitch also
21:09 mwhahaha yea was going to ask about that one
21:10 EmilienM I abandonned it
21:10 EmilienM mwhahaha: https://review.openstack.org/#/c/296672/
21:10 EmilienM it was the only one that is good lol
21:10 mwhahaha :D
21:15 openstackgerrit Merged openstack/puppet-vswitch: Release 4.0.0  https://review.openstack.org/296672
21:19 openstackgerrit Merged openstack/puppet-zaqar: Release 8.0.0  https://review.openstack.org/296763
21:21 openstackgerrit Merged openstack/puppet-trove: Release 8.0.0  https://review.openstack.org/296762
21:21 openstackgerrit Merged openstack/puppet-tempest: Release 8.0.0  https://review.openstack.org/296761
21:22 openstackgerrit Merged openstack/puppet-swift: Release 8.0.0  https://review.openstack.org/296760
21:23 openstackgerrit Merged openstack/puppet-sahara: Release 8.0.0  https://review.openstack.org/296759
21:23 openstackgerrit Merged openstack/puppet-nova: Release 8.0.0  https://review.openstack.org/296756
21:23 openstackgerrit Merged openstack/puppet-openstacklib: Release 8.0.0  https://review.openstack.org/296758
21:23 openstackgerrit Merged openstack/puppet-murano: Release 8.0.0  https://review.openstack.org/296754
21:24 openstackgerrit Merged openstack/puppet-manila: Release 8.0.0  https://review.openstack.org/296752
21:24 rahulait joined #puppet-openstack
21:25 openstackgerrit Merged openstack/puppet-mistral: Release 8.0.0  https://review.openstack.org/296753
21:25 mfisch EmilienM: wrt vswitch
21:25 EmilienM mfisch: why vswitch has some issue in your side but not in our CI?
21:25 mfisch it only errors on the first run
21:25 mfisch and perhaps you
21:25 mfisch its not using the same feature
21:25 mfisch I'm looking
21:26 mfisch yeah I dont see any vs_port references in the CI code
21:26 openstackgerrit Merged openstack/puppet-designate: Release 8.0.0  https://review.openstack.org/296746
21:26 EmilienM yes we have, let me show you
21:27 mfisch so whenever all this code was redone, someone added an abstract function and only a subclass for redhat
21:27 EmilienM mfisch: https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/provision.pp#L53
21:28 mfisch give me a minute
21:28 openstackgerrit Merged openstack/puppet-openstack_extras: Release 8.0.0  https://review.openstack.org/296757
21:28 openstackgerrit Merged openstack/puppet-ironic: Release 8.0.0  https://review.openstack.org/296751
21:28 openstackgerrit Merged openstack/puppet-horizon: Release 8.0.0  https://review.openstack.org/296750
21:28 openstackgerrit Merged openstack/puppet-heat: Release 8.0.0  https://review.openstack.org/296749
21:28 openstackgerrit Merged openstack/puppet-gnocchi: Release 8.0.0  https://review.openstack.org/296748
21:29 openstackgerrit Merged openstack/puppet-neutron: Release 8.0.0  https://review.openstack.org/296755
21:29 openstackgerrit Merged openstack/puppet-glance: Release 8.0.0  https://review.openstack.org/296747
21:29 openstackgerrit Merged openstack/puppet-cinder: Release 8.0.0  https://review.openstack.org/296745
21:29 openstackgerrit Merged openstack/puppet-ceilometer: Release 8.0.0  https://review.openstack.org/296744
21:31 mfisch EmilienM: I will figure out why CI isnt failing
21:32 mfisch if you use this defined type directly it breaks
21:32 mfisch neutron::plugins::ovs::port
21:33 EmilienM this code is highly old
21:33 EmilienM and I'm not sure it works
21:34 mfisch the difference between liberty and mitaka is large
21:34 mfisch oddly though this code is older than liberty
21:34 EmilienM I don't think this code works, we should drop it
21:34 EmilienM why do you need it?
21:35 mfisch will show you
21:35 mfisch ml2-ovs uses it here
21:35 mfisch https://github.com/openstack/puppet-neutron/blob/master/manifests/agents/ml2/ovs.pp#L186-L213
21:35 mfisch I'm really just using that block of ml2 ovs code
21:36 EmilienM ok so you have bridge_mappings
21:36 EmilienM something we don't have in our CI
21:38 mfisch yeah
21:38 mfisch maybe I dont need them
21:38 mfisch we use them at TWC
21:38 mfisch though
21:46 EmilienM mfisch: what is the bug report again in vswitch?
21:52 xarses_ joined #puppet-openstack
22:02 openstackgerrit Merged openstack/puppet-keystone: Release 8.0.0  https://review.openstack.org/296802
22:06 xarses EmilienM: can you review https://review.openstack.org/#/c/296050/ ?
22:07 EmilienM xarses: sure
22:07 EmilienM colonwq: fyi ^
22:07 EmilienM colonwq: it will likely break your patches in tripleo, eventually
22:08 xarses It's passing CI currently, but I want to make we're on board before nuking the options
22:08 xarses if you want deprecation speak up
22:08 EmilienM we don't need it
22:08 EmilienM since no release, your comment is valid
22:09 EmilienM xarses: +1
22:10 xarses thanks
22:12 rahulait joined #puppet-openstack
22:23 emccormick joined #puppet-openstack
22:24 mfisch EmilienM: https://bugs.launchpad.net/puppet-vswitch/+bug/1554096
22:24 openstack Launchpad bug 1554096 in puppet-vswitch "undefined method vlan? for vs_port on Ubuntu" [Undecided,Confirmed]
22:24 EmilienM thanks
22:27 aimon joined #puppet-openstack
22:27 jmikolajczak joined #puppet-openstack
22:41 openstackgerrit Merged openstack/puppet-cinder: Update Cinder BDD backend configuration  https://review.openstack.org/296606
22:44 tristanC joined #puppet-openstack
22:44 EmilienM dmsimard: only one test is failing for linuxbridge :-)
22:44 EmilienM I'm investigating
22:44 EmilienM tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops
22:45 EmilienM the one that test VM access using floating ip
22:45 EmilienM and I still have The external network bridge 'br-ex' does not exist
22:55 imcsk8 joined #puppet-openstack
22:55 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/296102
23:04 aimon_ joined #puppet-openstack
23:04 openstackgerrit Ricardo Rocha proposed openstack/puppet-magnum: Add conductor install and configuration.  https://review.openstack.org/275223
23:05 alee joined #puppet-openstack
23:05 colonwq EmilienM, Thanks for the FYI. I am locally testing how the change will impact next patch.
23:10 ducttape_ joined #puppet-openstack
23:24 openstack joined #puppet-openstack
23:24 openstackstatus joined #puppet-openstack
23:30 ansmith joined #puppet-openstack
23:45 angdraug joined #puppet-openstack
23:45 dmsimard EmilienM: yeah I made a couple comments on that patch
23:46 EmilienM dmsimard: right, I take them in account
23:47 EmilienM the l3 agent logs don't have errors with br-ex anymore
23:48 dmsimard wow, apevec actually found the tuned race in the code
23:48 EmilienM dmsimard: what was it?
23:48 dmsimard it attaches to dbus first (systemctl considers it ready) https://git.fedorahosted.org/cgit/tuned.git/tree/tuned.py#n66 and actually starts it after https://git.fedorahosted.org/cgit/tuned.git/tree/tuned.py#n76
23:49 EmilienM :)
23:57 openstackgerrit Merged openstack/puppet-ceph: Don't clone scenario_node_terminus for acceptance tests anymore  https://review.openstack.org/294167

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