Wednesday, 2016-03-23

« Tuesday, 2016-03-22 Index Thursday, 2016-03-24 »
*** trown|outtypewww is now known as trown00:03
*** ansmith has joined #puppet-openstack00:08
*** johnmilton has quit IRC00:08
*** brandon has quit IRC00:22
openstackgerritMerged openstack/puppet-openstack-integration: Round 1 of Puppet 4 support  https://review.openstack.org/29483800:34
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/29610200:34
*** angdraug has quit IRC00:35
*** saneax is now known as saneax_AFK00:37
*** brandon has joined #puppet-openstack00:45
*** ducttape_ has quit IRC00:58
EmilienMdmsimard: it's also failing on ubuntu.00:59
*** brandon has quit IRC01:01
EmilienMdmsimard: I'm working on the fix right now01:02
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/29613301:05
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/29613301:05
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/29610201:05
*** ansmith has quit IRC01:08
*** ansmith has joined #puppet-openstack01:10
openstackgerritCody Herriges proposed openstack/puppet-gnocchi: Fix typo in database classes.  https://review.openstack.org/29613501:20
openstackgerritCody Herriges proposed openstack/puppet-gnocchi: Fix typo in database classes.  https://review.openstack.org/29613501:22
*** ansmith has quit IRC01:22
EmilienM_ody: nice catch01:23
EmilienM_ody: I think we broke all beaker jobs01:28
EmilienMyeah we did, I'm sending a fix01:28
_odyEmilienM: Those need to be locked to Puppet 3 until after we finish getting p-o-i working properly.01:30
_odyThe work is similar though.01:30
*** johnmilton has joined #puppet-openstack01:31
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: install_modules: export PUPPET_BASE_PATH  https://review.openstack.org/29613901:32
EmilienM_ody: ^01:32
*** ayoung has joined #puppet-openstack01:32
*** ducttape_ has joined #puppet-openstack01:33
EmilienM_ody: I'll quickly merge this one, is it fine?01:34
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/29613301:40
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/29610201:43
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/29613301:43
*** johnmilton has quit IRC01:59
*** xarses has joined #puppet-openstack02:03
*** haplo37 has joined #puppet-openstack02:03
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/29613302:11
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: do-not-merge - puppet4 tests  https://review.openstack.org/29601502:12
*** haplo37 has quit IRC02:13
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/29610202:14
*** ducttape_ has quit IRC02:17
*** haplo37 has joined #puppet-openstack02:39
openstackgerritMerged openstack/puppet-openstack-integration: install_modules: export PUPPET_BASE_PATH  https://review.openstack.org/29613902:43
*** ilbot3 has quit IRC02:45
*** ilbot3 has joined #puppet-openstack02:47
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/29613302:48
*** markvoelker has quit IRC02:59
*** baker has joined #puppet-openstack02:59
*** baker_ has joined #puppet-openstack03:01
*** baker has quit IRC03:04
*** ducttape_ has joined #puppet-openstack03:08
openstackgerritAde Lee proposed openstack/puppet-barbican: Added keystone config to barbican api manifest  https://review.openstack.org/26346103:33
openstackgerritAde Lee proposed openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module  https://review.openstack.org/26346203:33
openstackgerritAde Lee proposed openstack/puppet-barbican: Added Dogtag settings for barbican-api  https://review.openstack.org/25885103:33
openstackgerritAde Lee proposed openstack/puppet-barbican: Add barbican client class and params  https://review.openstack.org/29615603:33
openstackgerritAde Lee proposed openstack/puppet-barbican: Moved logging to barbican-api module  https://review.openstack.org/29615703:33
openstackgerritAde Lee proposed openstack/puppet-barbican: Add helper files to configure paste file and barbican.conf  https://review.openstack.org/29615803:33
openstackgerritAde Lee proposed openstack/puppet-barbican: Add barbican-api manifest  https://review.openstack.org/29615903:33
openstackgerritAde Lee proposed openstack/puppet-barbican: Add quota and secret limit parameters to barbican-api manifest  https://review.openstack.org/29616003:33
openstackgerritAde Lee proposed openstack/puppet-barbican: Add keystone notification options to barbican-api manifest  https://review.openstack.org/29616103:33
openstackgerritAde Lee proposed openstack/puppet-barbican: Added settings for cert and secret store plugins  https://review.openstack.org/29616203:33
openstackgerritAde Lee proposed openstack/puppet-barbican: Added KMIP plugin settings to barbican-api manifest  https://review.openstack.org/29616303:33
*** haplo37 has quit IRC03:39
*** ducttape_ has quit IRC03:42
*** ducttape_ has joined #puppet-openstack03:45
*** richm has quit IRC03:45
*** baker_ has quit IRC03:55
*** markvoelker has joined #puppet-openstack03:59
*** baker has joined #puppet-openstack04:03
*** markvoelker has quit IRC04:04
*** mjblack has quit IRC04:20
*** mjblack has joined #puppet-openstack04:38
*** baker has quit IRC04:44
*** fedexo has joined #puppet-openstack04:51
*** ducttape_ has quit IRC04:53
*** aimon has quit IRC04:55
openstackgerritMarcus Furlong proposed openstack/puppet-barbican: minor doc fixes  https://review.openstack.org/29617905:06
*** zerda has joined #puppet-openstack05:28
*** saneax_AFK is now known as saneax05:38
*** aimon has joined #puppet-openstack05:43
*** aimon has quit IRC05:51
*** markvoelker has joined #puppet-openstack06:00
*** markvoelker has quit IRC06:04
*** fedexo has quit IRC06:35
*** markvoelker has joined #puppet-openstack07:01
*** markvoelker has quit IRC07:07
*** gfidente has joined #puppet-openstack07:10
*** aimon has joined #puppet-openstack07:32
*** furlongm_ is now known as furlongm07:36
*** aimon has quit IRC07:36
*** tesseract has joined #puppet-openstack07:41
*** tesseract is now known as Guest6084207:41
*** cwolferh has quit IRC07:49
*** cwolferh has joined #puppet-openstack08:00
*** jaosorior has joined #puppet-openstack08:06
*** foxhunt_away is now known as foxhunt7208:09
*** _skolekonov is now known as skolekonov08:19
*** chem has quit IRC08:22
*** hjensas has joined #puppet-openstack08:24
*** chem has joined #puppet-openstack08:26
*** paramite has joined #puppet-openstack08:37
*** jmikolajczak has joined #puppet-openstack08:40
*** permalac has joined #puppet-openstack08:43
*** foxhunt72 has quit IRC08:56
*** markvoelker has joined #puppet-openstack09:03
*** openstackgerrit has quit IRC09:03
*** openstackgerrit has joined #puppet-openstack09:04
*** markvoelker has quit IRC09:08
*** karimb has joined #puppet-openstack09:09
*** yolanda has quit IRC09:17
*** derekh has joined #puppet-openstack09:19
*** aimon has joined #puppet-openstack09:20
*** foxhunt_away has joined #puppet-openstack09:20
*** foxhunt_away is now known as foxhunt7209:21
*** aimon has quit IRC09:24
*** yolanda has joined #puppet-openstack09:34
*** paramite is now known as paramite|afk09:41
*** yolanda has quit IRC09:42
*** paramite|afk is now known as paramite10:09
*** aimon has joined #puppet-openstack10:14
*** aimon has quit IRC10:18
openstackgerritIvan Berezovskiy proposed openstack/puppet-oslo: Improve checks for rabbit host, port and hosts parameters  https://review.openstack.org/29396410:20
*** bookwar has quit IRC10:24
*** bookwar has joined #puppet-openstack10:24
*** tore_ has joined #puppet-openstack10:25
*** paramite is now known as paramite|afk10:28
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:29
*** jaosorior has quit IRC10:34
*** jaosorior has joined #puppet-openstack10:36
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Allow the Redis specific monitor to use authentication  https://review.openstack.org/29593710:38
*** furlongm_ has joined #puppet-openstack10:44
*** furlongm has quit IRC10:45
*** _degorenko|afk is now known as degorenko10:51
*** furlongm has joined #puppet-openstack10:55
*** furlongm_ has quit IRC10:55
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: WIP: multi-domain testing with ldap backend.  https://review.openstack.org/29637011:01
*** markvoelker has joined #puppet-openstack11:04
*** markvoelker has quit IRC11:08
*** karimb has quit IRC11:13
*** karimb has joined #puppet-openstack11:17
*** johnmilton has joined #puppet-openstack11:27
*** paramite|afk is now known as paramite11:34
EmilienMhello11:38
degorenkohey  :)11:44
skolekonovEmilienM, 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 one11:44
openstackLaunchpad 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:44
*** dgurtner has joined #puppet-openstack11:48
*** dgurtner has joined #puppet-openstack11:48
*** haplo37 has joined #puppet-openstack11:50
openstackgerritDenis Egorenko proposed openstack/puppet-glance: Add missed glare_config options for glance::config class  https://review.openstack.org/29639011:53
EmilienMskolekonov: I set High to the bug, so yes11:59
EmilienMskolekonov: and even if it lands after the release we would have backported it.11:59
EmilienMI started this: https://wiki.openstack.org/wiki/Puppet/Backport_policy12:00
skolekonovok, thanks12:01
openstackgerritMiles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/27618712:03
*** haplo37 has quit IRC12:04
*** dprince has joined #puppet-openstack12:07
*** jayg|g0n3 is now known as jayg12:11
*** markvoelker has joined #puppet-openstack12:20
*** zerda has quit IRC12:22
EmilienMRDO servers had some outage recently, do recheck if you see repos issue in logs12:23
*** dgurtner has quit IRC12:31
openstackgerritIvan Berezovskiy proposed openstack/puppet-oslo: Improve checks for rabbit host, port and hosts parameters  https://review.openstack.org/29396412:40
*** spredzy has quit IRC12:45
*** morazi has joined #puppet-openstack12:49
openstackgerritDenis Egorenko proposed openstack/puppet-glance: Add missed glare_config options for glance::config class  https://review.openstack.org/29639012:49
degorenkoEmilienM, ^^ not sure, about reno notes, do we need that for such patch?12:50
EmilienMdegorenko: no need!12:50
degorenkook12:50
degorenkoone typo in patch12:50
openstackgerritDenis Egorenko proposed openstack/puppet-glance: Add missed glare_config options for glance::config class  https://review.openstack.org/29639012:50
degorenkoand so many red spaces....12:51
degorenko:)12:51
EmilienMdegorenko: did you see my review?12:52
degorenkoEmilienM, which one? I'm just back from lunch12:52
EmilienMdegorenko: https://review.openstack.org/#/c/296390/12:53
openstackgerritDmitry Burmistrov proposed openstack/puppet-heat: Use identity_uri and auth_uri by default  https://review.openstack.org/26132612:53
degorenkoEmilienM, i've added tests12:53
EmilienMI don't see it o_O12:53
degorenkoold patch number in link?12:53
degorenkoit's already 3 patch sets here12:54
degorenkoand will be 4 :D12:54
EmilienMoh you're about to push it, ok12:54
degorenkoEmilienM, tests already pushed12:54
*** ducttape_ has joined #puppet-openstack12:55
*** dgurtner has joined #puppet-openstack12:55
*** dgurtner has quit IRC12:55
*** dgurtner has joined #puppet-openstack12:55
EmilienMdegorenko: maybe I have an issue but I don't see it in Gerrit12:55
openstackgerritMerged openstack/puppet-gnocchi: Fix typo in database classes.  https://review.openstack.org/29613512:55
degorenkolol12:55
degorenkoEmilienM, https://review.openstack.org/#/c/296390/3/12:55
degorenkotry that12:55
EmilienMusing another browser, I can't see it neither12:55
degorenkomagic12:56
EmilienMdegorenko: still not12:56
EmilienMdegorenko: I meant acceptance tests btw12:56
EmilienMyou added rspec :-P12:56
*** aimon has joined #puppet-openstack12:56
EmilienMwhich is super cool :-P12:56
degorenkolol12:56
degorenko:D12:56
EmilienMspec/acceptance/glance_config_spec.rb12:56
degorenkooh yeah12:57
degorenkof.....12:57
degorenkosorry :)12:57
EmilienM:)12:57
degorenkodidn't get it correctly12:57
EmilienMno problem!12:57
openstackgerritMerged openstack/puppet-neutron: Fix LinuxBridge Agent configuration file name  https://review.openstack.org/29613312:59
EmilienMrdo servers are down again...13:01
*** aimon has quit IRC13:01
degorenko:(13:02
openstackgerritDenis Egorenko proposed openstack/puppet-glance: Add missed glare_config options for glance::config class  https://review.openstack.org/29639013:02
degorenkoEmilienM, should be better now :) ^13:03
EmilienMdegorenko: thx13:04
openstackgerritDmitry Burmistrov proposed openstack/puppet-heat: Use identity_uri and auth_uri by default  https://review.openstack.org/26132613:08
openstackgerritMerged openstack/puppet-nova: Handle config_drive_format option  https://review.openstack.org/29523413:14
*** richm has joined #puppet-openstack13:18
*** ducttape_ has quit IRC13:19
*** moravec has quit IRC13:24
*** moravec has joined #puppet-openstack13:27
*** trown is now known as trown|afk13:28
*** ducttape_ has joined #puppet-openstack13:33
openstackgerritMerged openstack/puppet-magnum: Add acceptance test  https://review.openstack.org/29420013:39
*** trown|afk is now known as trown13:51
degorenkoEmilienM, 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/Puppet13:55
EmilienMfail13:55
*** aimon has joined #puppet-openstack13:56
EmilienMdegorenko: yes13:56
degorenkothanks :)13:56
*** haplo37 has joined #puppet-openstack13:57
EmilienMdegorenko: I'll look later13:57
*** xarses has quit IRC13:58
degorenkosure13:59
*** dhollinger has joined #puppet-openstack14:00
*** aimon has quit IRC14:00
dmsimardEmilienM: is it down again ? looks up to me14:01
EmilienMdmsimard: ok cool14:01
*** spredzy has joined #puppet-openstack14:02
dmsimardEmilienM: puppet-neutron has no stable/mitaka branch14:07
dmsimardI wanted to backport it14:07
dmsimardI guess it hasn't been cut yet ?14:07
dmsimardalso, KB was saying he was hitting a similar issue on liberty14:07
dmsimardI'm testing right now14:07
*** openstack has joined #puppet-openstack14:23
aleedegorenko, yup -- ok, in  https://review.openstack.org/#/c/296157/1/manifests/logging.pp14:24
aleeyo mention what needs to be done for deprecation ..14:24
degorenkoyes, this is exactly when we can use it14:24
degorenkosame for db class, if such present14:24
EmilienMdmsimard: I don't think so14:24
aleedegorenko, 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:25
degorenkoalee, do we have branches for barbican?14:26
degorenkono, we don't14:26
aleedegorenko, prob not yet ..14:26
degorenkoalee, then, since puppet-barbican was not released ever - you can simply remove that, yes, i forgot about that. This module wasn't ever released14:27
aleedegorenko, ok great14:27
aleedegorenko, in https://review.openstack.org/#/c/296157/1/manifests/api/logging.pp , there is a bunch of "pick" code there14:28
degorenkoalee, in this case we don't need that14:29
aleedegorenko, I think what you guys are saying is that I can just eliminate that code ..14:29
degorenkoalee, indeed14:29
aleedegorenko, and then those parameters would not be in the barbican::api manifest at all?14:29
degorenkoyep14:29
aleecool14:29
degorenkoso, you can add include for this class in api14:29
aleedegorenko, yes - will do14:30
mfischiurygregory: we need some unit tests for the ldap_backend class14:30
degorenkoand when you need override such parameters in non-hiera way you can simply declare this class with needed parameters14:30
*** dprince has quit IRC14:30
degorenkoit will not raise duplication error, because include is not actual declaring14:31
aleedegorenko, in https://review.openstack.org/#/c/296158/1/manifests/config.pp, you make comment about removing old config14:31
iurygregorymfisch ?14:31
degorenkoalee, example for logging and database for a new classes https://review.openstack.org/#/c/293524/14:31
aleedegorenko, thanks - a new example definitely helps :)14:32
degorenkoalee, yes, what's wrong with old barbican_config?14:32
mfischiurygregory: didnt you write the ldap_backend class in keystone?14:32
*** karimb has quit IRC14:32
iurygregorynops =P14:32
aleedegorenko, well - I renamed it to api_config --  the reason is that eventually I will be adding support for barbican -worker processes14:33
aleewhich will have their own logging14:33
iurygregorymfisch, I just said something about ldap at the meeting yesterday14:33
aleeand will be worker_config most likely14:33
degorenkoalee, i don't see patch, where you renamed current provider14:33
mfischiurygregory: k ;)14:33
mfischah chem  wrote it14:34
mfischsorry iurygregory14:34
iurygregorymaybe i can help with review =)14:34
iurygregoryno problem mfisch =D14:34
aleedegorenko, not sure I understand -- why can I not rename barbican_config to api_config ?14:34
chemmfisch: what's up ?14:34
degorenkoalee, oh, now i got it14:35
aleedegorenko, ok - so no objection then ?14:35
degorenkoalee, yes, no objection here. Just add new tests for new providers :)14:36
aleedegorenko, will do - thanks :)14:36
degorenkoalee, example is in patch above too :)14:36
aleedegorenko, excellent :)14:36
degorenkoalee, also you can see here: https://review.openstack.org/#/c/296390/ this is rspec tests14:37
openstackgerritMatt Fischer proposed openstack/puppet-keystone: Add the ability to control LDAP package mgmt  https://review.openstack.org/29649314:37
aleedegorenko, is there an example of the usage of ensure_resources() too ?14:37
degorenkoalee, hmmm14:37
aleedegorenko, I can play with it if not .. shouldn't be too hard to figure out.14:38
degorenkoalee, we have such examples, let me find them14:38
degorenkoalee, in which patch i dropped this comment?14:38
aleehttps://review.openstack.org/#/c/296156/114:39
aleefor one ..14:39
degorenkoalee, ah, ok, so14:39
degorenkofor example for keystone we have only include for openstackclient14:39
degorenkonot any calling class explicitly14:40
aleedegorenko, ok - I can do that :)14:40
degorenkoalee, ensure function, for example for package exactly: https://github.com/openstack/puppet-openstacklib/blob/master/manifests/openstackclient.pp#L1414:40
degorenkoif you want manage openstackclient - then you need to use such construction ^14:41
degorenkobut i'm not sure why you need that?14:41
aleedegorenko, yeah14:42
aleedegorenko, so maybe remove altogether?14:43
degorenkoalee, calling openstackclient?14:43
openstackgerritDimitri Savineau proposed openstack/puppet-tripleo: Enable HAProxy forwardfor option for Horizon.  https://review.openstack.org/29649714:44
aleeright -- there is barbican stuff in the openstack client -- but I'm not calling it anywhere14:44
aleealternatively I can do what keystone does and just include it14:44
degorenkoalee, second way is more better14:45
aleedegorenko, I suppose eventually we will be , so I 'll just include it like keystone does.14:45
degorenko:)14:45
*** mdorman has joined #puppet-openstack14:47
aleedegorenko, any idea what the test for the client class should look like?14:47
openstackgerritIvan Berezovskiy proposed openstack/puppet-oslo: Fix values for rabbit_hosts  https://review.openstack.org/29396414:47
degorenkoalee, https://github.com/openstack/puppet-keystone/blob/master/spec/classes/keystone_client_spec.rb14:48
aleedegorenko, perfect thanks14:49
aleedegorenko, can you tell me what I need to change for https://review.openstack.org/#/c/296159/1/manifests/api.pp@346  ?14:50
degorenkoalee, it's question probably for other reviewers, but for you: something like https://review.openstack.org/#/c/292309/14:52
aleedegorenko, ok14:54
mfischchem: was thinking we should have some unit tests for ldap_backend in keystone14:54
aleedegorenko, https://review.openstack.org/#/c/296162/1/manifests/api.pp , there are some comments there about some code which I added that was commented out14:55
chemmfisch: yep, I'm working on this, it would be ldap and multi-backend domain14:55
mfischk14:55
aleedegorenko, I plan to revisit that code with a follow up patch14:55
chemmfisch: https://review.openstack.org/#/c/296370/14:55
aleeafter this set lands14:55
mfischchem: I pushed up a small change for the LDAP code this rmoning14:55
chemmfisch: yep, I've seen14:55
degorenkoalee, yep, that's because this code will not work for puppet < 414:56
chemmfisch: 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 necessary14:56
chemmfisch: class should only use the define ... but that's another patch :014:56
chem:)14:56
mfischwhich define?14:56
aleedegorenko, right -- well at the time, we had no support for multicfg options - which its the original reason I commented it out.14:57
chemmfisch: in ldap you have a class and a define: ldap is a class and ldap_backend is a define14:57
aleedegorenko, so can I just leave it there, or should I just remove the whole thing?14:58
mfischchem: ah14:58
degorenkoalee, then remove this block at all :)14:58
mfischchem: I totally missed that ;)14:58
aleedegorenko, ok14:58
chemmfisch: 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
chemmfisch: for the testing ldap, I hope to have something working before the end of the week.14:59
*** trown|meeting is now known as trown15:00
aleedegorenko, 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
degorenkotake a look here: https://review.openstack.org/#/c/292309/ it is in puppet-oslo module15:00
degorenkowe are planning to switch all modules to a new scheme15:00
aleedegorenko, ok --I'll try out the changes there.15:02
mfischchem: restarting keystone to add an LDAP backend, why is that a problme?15:06
chemmfisch: 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:07
mfischis not a huge issue for me15:09
mfischchem: im on board with you switching15:10
mfischhaving the code twice is terrible15:10
chemmfisch: oki, I'll do that on top of your patch15:11
*** openstackstatus has joined #puppet-openstack15:13
*** ChanServ sets mode: +v openstackstatus15:13
*** ansmith has quit IRC15:18
*** baker has quit IRC15:21
*** baker has joined #puppet-openstack15:22
*** jaosorior has quit IRC15:27
*** ansmith has joined #puppet-openstack15:33
*** paramite is now known as paramite|afk15:33
*** paramite|afk is now known as paramite15:35
*** jmikolajczak has quit IRC15:36
openstackgerritMichael Polenchuk proposed openstack/puppet-nova: Autorequire nova security group  https://review.openstack.org/29654815:39
*** jaosorior has joined #puppet-openstack15:44
*** paramite is now known as paramite|afk15:45
*** ducttape_ has quit IRC15:50
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: (WIP) Explicit containment for more predictability  https://review.openstack.org/29655715:51
*** paramite|afk is now known as paramite15:59
*** ansmith has quit IRC16:05
EmilienMlooking at https://review.openstack.org/#/c/296557/ - I'm really afraid how puppet4 is not really backward compatible with puppet316:14
EmilienMspecially that code: https://review.openstack.org/#/c/296557/1/fixtures/scenario-aio.pp16:14
openstackgerritDmitry Burmistrov proposed openstack/puppet-heat: Use identity_uri and auth_uri by default  https://review.openstack.org/26132616:14
*** karimb has joined #puppet-openstack16:16
EmilienM_ody: does puppetlabs expect puppet3 users to update their manifests ordering for puppet4 migration?16:17
openstackgerritMerged openstack/puppet-vswitch: Add Solaris support  https://review.openstack.org/29601016:18
*** trown is now known as trown|lunch16:19
*** ansmith has joined #puppet-openstack16:20
*** Guest60842 has quit IRC16:21
_odyEmilienM: 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
openstackgerritIvan Berezovskiy proposed openstack/puppet-oslo: Fix values for rabbit_hosts  https://review.openstack.org/29396416:26
_odyIn 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:26
*** karimb has quit IRC16:27
*** skolekonov is now known as _skolekonov16:27
_odyEmilienM: 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:29
_odyDoing 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:30
EmilienM_ody: what do you suggest for Puppet OpenStack ?16:31
_odyWe go directly to Puppet 4.16:32
EmilienMand what about out users that still have puppet3?16:32
_odyLess work in my opinion.  Else you just touch all the code first.16:32
_odyEmilienM: I mean just upgrade the code to be compatible both ways.16:32
_odyIt is sill possible.16:32
EmilienM_ody: ok16:32
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
EmilienMif yes, I'm happy with that now16:33
_odyEmilienM: So far but I am still only in aio.16:33
EmilienMok16:35
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/29610216:35
*** paramite is now known as paramite|afk16:42
*** paramite|afk is now known as paramite16:42
openstackgerritDenis Egorenko proposed openstack/puppet-cinder: Update Cinder BDD backend configuration  https://review.openstack.org/29660616:43
*** rahulait has joined #puppet-openstack16:47
*** paramite has quit IRC16:47
*** foxhunt72 is now known as foxhunt_away16:50
*** ducttape_ has joined #puppet-openstack16:51
*** cdelatte has joined #puppet-openstack16:53
*** Guest63327 has quit IRC16:55
*** ducttape_ has quit IRC16:56
*** karimb has joined #puppet-openstack16:56
*** ducttape_ has joined #puppet-openstack16:56
*** aimon has quit IRC16:57
*** gfidente has quit IRC17:24
degorenkoEmilienM, i've answered here https://review.openstack.org/29660617:24
*** derekh has quit IRC17:25
*** brandon has joined #puppet-openstack17:26
*** aimon has joined #puppet-openstack17:28
*** rcernin has quit IRC17:29
*** aimon has quit IRC17:33
aleedegorenko, 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 has joined #puppet-openstack17:34
degorenkoalee, first - full path17:34
openstackgerritMerged openstack/puppet-neutron: Add ensure_absent_val behavior to Nuage neutron plugin type  https://review.openstack.org/29604317:34
*** trown|lunch is now known as trown17:36
aleedegorenko, thanks17:36
EmilienMdegorenko: will look after lunch :)17:36
degorenkoEmilienM, sure17:37
EmilienMdegorenko: back :)17:53
EmilienMdegorenko: damn, I missed that param :(17:53
degorenkono problem :)17:53
degorenkoEmilienM, also i will have one more patch for glare - i forgot to add endpoint :D17:54
*** vinsh has quit IRC17:54
*** angdraug has joined #puppet-openstack17:54
EmilienMdegorenko: it can help ;-)17:54
*** vinsh has joined #puppet-openstack17:54
degorenkoEmilienM, 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 :D17:55
*** social has quit IRC18:01
*** vinsh has quit IRC18:01
*** vinsh has joined #puppet-openstack18:02
*** jaosorior has quit IRC18:13
openstackgerritDenis Egorenko proposed openstack/puppet-glance: Configure endpoint for Glance Glare  https://review.openstack.org/29665618:15
*** iurygregory has quit IRC18:15
openstackgerritMerged openstack/puppet-manila: Fix typos  https://review.openstack.org/29437518:15
*** ericksonsantos has quit IRC18:16
*** karimb has quit IRC18:19
degorenkoEmilienM, 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
EmilienMlooking18:23
degorenkoso, the problem is that ensure_ function can't use dependencies18:24
EmilienMright18:24
EmilienMyou need to use a collector maybe18:24
EmilienMPackage['tgt' -> Service['tgtd']18:24
EmilienMsomething like that18:24
EmilienMmaybe a notify?18:24
degorenkowell, that's not a problem with notify18:25
degorenkowe must be sure, that such resources are present18:25
degorenkobut in case of multibackend with iscsi18:25
degorenkowe will get duplication error18:25
degorenkoif they will be ran in one catalog18:25
*** ericksonsantos has joined #puppet-openstack18:26
degorenko1) can we pass require/notify to ensure_ injected to passed params?18:26
degorenko2) call iscsi backend from bdd with proper configs - we have almost identical options between them, bdd has a few more18:27
degorenkomwhahaha, ^18:27
*** iurygregory has joined #puppet-openstack18:27
EmilienMdegorenko: I don't see why you can't use notify18:30
EmilienMor is it 1)?18:30
degorenkowe can't use collectors here, because there is no guarantee that we have iscsi also18:31
degorenkoas another option, use !defined(<resource>)18:31
degorenkoin 'if' structure18:31
EmilienMthe collector is secure because catalog won't fail if resource is not here18:32
EmilienMisn't?18:32
degorenkobut who will installed/run resources?18:32
*** angdraug has quit IRC18:33
EmilienMmhh ok I see18:34
EmilienM!defined(<resource>) maybe18:34
openstackEmilienM: Error: "defined(<resource>)" is not a valid command.18:34
EmilienMopenstack: hey bro18:34
degorenkoEmilienM, 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 error18:37
*** baker has quit IRC18:37
degorenkowe can have iscsi1 and iscsi2 backends with different volumes_dir or whatever18:38
degorenkoit will have different sections in cinder.conf and error :(18:38
EmilienMright, it's a bit tricky18:39
*** permalac has quit IRC18:39
degorenkoit's already common situation for me - digging into one bug and discover new one x)18:39
EmilienMok I'm going to spam18:42
degorenko:D branches are coming!18:42
EmilienMI hope I don't mess up18:43
EmilienMok I messed up :P18:44
*** angdraug has joined #puppet-openstack18:49
*** angdraug has quit IRC18:50
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Release 8.0.0  https://review.openstack.org/29666918:50
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Release 8.0.0  https://review.openstack.org/29667018:50
openstackgerritEmilien Macchi proposed openstack/puppet-designate: Release 8.0.0  https://review.openstack.org/29667118:50
openstackgerritEmilien Macchi proposed openstack/puppet-vswitch: Release 4.0.0  https://review.openstack.org/29667218:50
openstackgerritEmilien Macchi proposed openstack/puppet-glance: Release 8.0.0  https://review.openstack.org/29667318:51
openstackgerritEmilien Macchi proposed openstack/puppet-gnocchi: Release 8.0.0  https://review.openstack.org/29667418:51
openstackgerritEmilien Macchi proposed openstack/puppet-heat: Release 8.0.0  https://review.openstack.org/29667518:51
openstackgerritEmilien Macchi proposed openstack/puppet-horizon: Release 8.0.0  https://review.openstack.org/29667618:51
*** openstack has joined #puppet-openstack19:22
*** openstackstatus has joined #puppet-openstack19:24
*** ChanServ sets mode: +v openstackstatus19:24
*** dtantsur has joined #puppet-openstack19:24
*** Guest69253 is now known as zeus`19:25
*** sbadia has joined #puppet-openstack19:26
*** zeus` is now known as zeus19:33
*** zeus has quit IRC19:34
*** zeus has joined #puppet-openstack19:34
EmilienMmwhahaha: if you have time today, I prepared releases for modules https://review.openstack.org/#/q/topic:puppet/mitaka+status:open19:36
*** foxhunt_away is now known as foxhunt7219:36
mwhahahak19:36
EmilienMthe step after that will be to create branches and we're almost good19:37
*** alee has quit IRC19:45
*** jmikolajczak has joined #puppet-openstack19:50
mwhahahaEmilienM: you've got a trailing space in the 8.0.0 version number in the metadata.json19:54
EmilienMdamn19:55
EmilienMdamn !19:55
EmilienMeverywhere I guess19:55
mwhahahaso i could review and add to my stackalytics's profile ;) that's a lot of -1s19:55
EmilienMI used sed19:55
mwhahaha:D19:55
EmilienMlet me update... /me sigh19:55
*** ansmith has quit IRC19:57
*** jmikolajczak has quit IRC20:01
EmilienMok let's try again20:03
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Release 8.0.0  https://review.openstack.org/29672220:03
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Release 8.0.0  https://review.openstack.org/29672320:03
openstackgerritEmilien Macchi proposed openstack/puppet-designate: Release 8.0.0  https://review.openstack.org/29672420:03
openstackgerritEmilien Macchi proposed openstack/puppet-glance: Release 8.0.0  https://review.openstack.org/29672520:03
openstackgerritEmilien Macchi proposed openstack/puppet-gnocchi: Release 8.0.0  https://review.openstack.org/29672620:03
openstackgerritEmilien Macchi proposed openstack/puppet-heat: Release 8.0.0  https://review.openstack.org/29672720:03
openstackgerritEmilien Macchi proposed openstack/puppet-horizon: Release 8.0.0  https://review.openstack.org/29672820:03
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Release 8.0.0  https://review.openstack.org/29672920:03
openstackgerritEmilien Macchi proposed openstack/puppet-manila: Release 8.0.0  https://review.openstack.org/29673020:03
openstackgerritEmilien Macchi proposed openstack/puppet-mistral: Release 8.0.0  https://review.openstack.org/29673220:03
openstackgerritEmilien Macchi proposed openstack/puppet-murano: Release 8.0.0  https://review.openstack.org/29673320:03
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Release 8.0.0  https://review.openstack.org/29673420:03
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Release 8.0.0  https://review.openstack.org/29673520:04
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_extras: Release 8.0.0  https://review.openstack.org/29673620:04
openstackgerritEmilien Macchi proposed openstack/puppet-openstacklib: Release 8.0.0  https://review.openstack.org/29673720:04
openstackgerritEmilien Macchi proposed openstack/puppet-sahara: Release 8.0.0  https://review.openstack.org/29673820:04
openstackgerritEmilien Macchi proposed openstack/puppet-swift: Release 8.0.0  https://review.openstack.org/29673920:04
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: Release 8.0.0  https://review.openstack.org/29674020:04
openstackgerritEmilien Macchi proposed openstack/puppet-trove: Release 8.0.0  https://review.openstack.org/29674120:04
openstackgerritEmilien Macchi proposed openstack/puppet-zaqar: Release 8.0.0  https://review.openstack.org/29674220:04
EmilienMmwhahaha: better? :P20:05
mwhahahalet me see :D20:05
EmilienMdamn no20:05
EmilienMno no20:05
EmilienMI messed up again I should go sleep20:05
mwhahahaoops20:05
EmilienMok this time is good20:07
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Release 8.0.0  https://review.openstack.org/29674420:07
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Release 8.0.0  https://review.openstack.org/29674520:07
openstackgerritEmilien Macchi proposed openstack/puppet-designate: Release 8.0.0  https://review.openstack.org/29674620:07
EmilienM(I hope)20:07
openstackgerritEmilien Macchi proposed openstack/puppet-glance: Release 8.0.0  https://review.openstack.org/29674720:07
openstackgerritEmilien Macchi proposed openstack/puppet-gnocchi: Release 8.0.0  https://review.openstack.org/29674820:07
openstackgerritEmilien Macchi proposed openstack/puppet-heat: Release 8.0.0  https://review.openstack.org/29674920:07
openstackgerritEmilien Macchi proposed openstack/puppet-horizon: Release 8.0.0  https://review.openstack.org/29675020:07
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Release 8.0.0  https://review.openstack.org/29675120:07
openstackgerritEmilien Macchi proposed openstack/puppet-manila: Release 8.0.0  https://review.openstack.org/29675220:07
openstackgerritEmilien Macchi proposed openstack/puppet-mistral: Release 8.0.0  https://review.openstack.org/29675320:07
openstackgerritEmilien Macchi proposed openstack/puppet-murano: Release 8.0.0  https://review.openstack.org/29675420:07
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Release 8.0.0  https://review.openstack.org/29675520:07
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Release 8.0.0  https://review.openstack.org/29675620:07
mwhahahanow it's in tripicate20:07
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_extras: Release 8.0.0  https://review.openstack.org/29675720:07
mwhahahatriplicate20:07
openstackgerritEmilien Macchi proposed openstack/puppet-openstacklib: Release 8.0.0  https://review.openstack.org/29675820:07
openstackgerritEmilien Macchi proposed openstack/puppet-sahara: Release 8.0.0  https://review.openstack.org/29675920:07
mwhahahasince you got new change-ids20:07
openstackgerritEmilien Macchi proposed openstack/puppet-swift: Release 8.0.0  https://review.openstack.org/29676020:07
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: Release 8.0.0  https://review.openstack.org/29676120:08
mwhahahaabandon time20:08
openstackgerritEmilien Macchi proposed openstack/puppet-trove: Release 8.0.0  https://review.openstack.org/29676220:08
openstackgerritEmilien Macchi proposed openstack/puppet-zaqar: Release 8.0.0  https://review.openstack.org/29676320:08
mwhahahaEmilienM: you want to abandon the bad changes?20:09
EmilienMwait20:09
EmilienMI used amend20:09
mwhahahanope20:09
EmilienMok I go kill myself, I come back.20:10
mwhahahahttps://review.openstack.org/296669 https://review.openstack.org/296722 https://review.openstack.org/29674420:10
mwhahaha:D20:10
* EmilienM big facepalm20:10
*** ansmith has joined #puppet-openstack20:10
mwhahahait's one of those days20:10
EmilienMmwhahaha: exactly. Let me abandon them20:11
EmilienMmy script used amend but was overriding commit message.20:12
*** johnmilton has quit IRC20:15
*** openstack has joined #puppet-openstack20:33
*** rahulait has quit IRC20:41
*** haplo37 has quit IRC20:41
*** rahulait has joined #puppet-openstack20:41
*** rahulait has quit IRC20:42
*** ansmith has quit IRC20:45
*** rahulait has joined #puppet-openstack20:45
*** angdraug has joined #puppet-openstack20:50
*** imcsk8 has quit IRC20:56
*** rahulait has quit IRC21:03
mwhahahaEmilienM: puppet-keystone Release 8.0.0?21:04
EmilienMmwhahaha: looking21:04
*** alee has joined #puppet-openstack21:05
EmilienMdid I miss it?21:05
mwhahahai don't see it21:05
mwhahahawhich is why i asked :D21:05
EmilienMI missed it21:05
EmilienMit's 5pm, almost the end of this day.21:06
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: Release 8.0.0  https://review.openstack.org/29680221:08
EmilienMmwhahaha: vswitch also21:09
*** dgurtner has quit IRC21:09
mwhahahayea was going to ask about that one21:09
EmilienMI abandonned it21:10
EmilienMmwhahaha: https://review.openstack.org/#/c/296672/21:10
EmilienMit was the only one that is good lol21:10
mwhahaha:D21:10
openstackgerritMerged openstack/puppet-vswitch: Release 4.0.0  https://review.openstack.org/29667221:15
*** trown is now known as trown|outtypewww21:17
openstackgerritMerged openstack/puppet-zaqar: Release 8.0.0  https://review.openstack.org/29676321:19
*** mdorman has quit IRC21:21
openstackgerritMerged openstack/puppet-trove: Release 8.0.0  https://review.openstack.org/29676221:21
openstackgerritMerged openstack/puppet-tempest: Release 8.0.0  https://review.openstack.org/29676121:21
openstackgerritMerged openstack/puppet-swift: Release 8.0.0  https://review.openstack.org/29676021:22
openstackgerritMerged openstack/puppet-sahara: Release 8.0.0  https://review.openstack.org/29675921:23
openstackgerritMerged openstack/puppet-nova: Release 8.0.0  https://review.openstack.org/29675621:23
openstackgerritMerged openstack/puppet-openstacklib: Release 8.0.0  https://review.openstack.org/29675821:23
openstackgerritMerged openstack/puppet-murano: Release 8.0.0  https://review.openstack.org/29675421:23
openstackgerritMerged openstack/puppet-manila: Release 8.0.0  https://review.openstack.org/29675221:24
*** rahulait has joined #puppet-openstack21:24
openstackgerritMerged openstack/puppet-mistral: Release 8.0.0  https://review.openstack.org/29675321:25
mfischEmilienM: wrt vswitch21:25
EmilienMmfisch: why vswitch has some issue in your side but not in our CI?21:25
mfischit only errors on the first run21:25
mfischand perhaps you21:25
mfischits not using the same feature21:25
mfischI'm looking21:25
mfischyeah I dont see any vs_port references in the CI code21:26
openstackgerritMerged openstack/puppet-designate: Release 8.0.0  https://review.openstack.org/29674621:26
EmilienMyes we have, let me show you21:26
mfischso whenever all this code was redone, someone added an abstract function and only a subclass for redhat21:27
EmilienMmfisch: https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/provision.pp#L5321:27
mfischgive me a minute21:28
openstackgerritMerged openstack/puppet-openstack_extras: Release 8.0.0  https://review.openstack.org/29675721:28
openstackgerritMerged openstack/puppet-ironic: Release 8.0.0  https://review.openstack.org/29675121:28
openstackgerritMerged openstack/puppet-horizon: Release 8.0.0  https://review.openstack.org/29675021:28
openstackgerritMerged openstack/puppet-heat: Release 8.0.0  https://review.openstack.org/29674921:28
openstackgerritMerged openstack/puppet-gnocchi: Release 8.0.0  https://review.openstack.org/29674821:28
openstackgerritMerged openstack/puppet-neutron: Release 8.0.0  https://review.openstack.org/29675521:29
openstackgerritMerged openstack/puppet-glance: Release 8.0.0  https://review.openstack.org/29674721:29
openstackgerritMerged openstack/puppet-cinder: Release 8.0.0  https://review.openstack.org/29674521:29
openstackgerritMerged openstack/puppet-ceilometer: Release 8.0.0  https://review.openstack.org/29674421:29
mfischEmilienM: I will figure out why CI isnt failing21:31
mfischif you use this defined type directly it breaks21:32
mfischneutron::plugins::ovs::port21:32
EmilienMthis code is highly old21:33
EmilienMand I'm not sure it works21:33
mfischthe difference between liberty and mitaka is large21:34
mfischoddly though this code is older than liberty21:34
EmilienMI don't think this code works, we should drop it21:34
EmilienMwhy do you need it?21:34
mfischwill show you21:35
mfischml2-ovs uses it here21:35
mfischhttps://github.com/openstack/puppet-neutron/blob/master/manifests/agents/ml2/ovs.pp#L186-L21321:35
mfischI'm really just using that block of ml2 ovs code21:35
EmilienMok so you have bridge_mappings21:36
EmilienMsomething we don't have in our CI21:36
mfischyeah21:38
mfischmaybe I dont need them21:38
mfischwe use them at TWC21:38
mfischthough21:38
*** morazi has quit IRC21:40
*** dhollinger has quit IRC21:45
EmilienMmfisch: what is the bug report again in vswitch?21:46
*** xarses_ has joined #puppet-openstack21:52
*** xarses has quit IRC21:52
*** xarses_ is now known as xarses21:56
*** baker has quit IRC21:58
openstackgerritMerged openstack/puppet-keystone: Release 8.0.0  https://review.openstack.org/29680222:02
xarsesEmilienM: can you review https://review.openstack.org/#/c/296050/ ?22:06
EmilienMxarses: sure22:07
EmilienMcolonwq: fyi ^22:07
EmilienMcolonwq: it will likely break your patches in tripleo, eventually22:07
xarsesIt's passing CI currently, but I want to make we're on board before nuking the options22:08
xarsesif you want deprecation speak up22:08
EmilienMwe don't need it22:08
EmilienMsince no release, your comment is valid22:08
EmilienMxarses: +122:09
*** ducttape_ has quit IRC22:09
xarsesthanks22:10
*** rahulait has quit IRC22:11
*** rahulait has joined #puppet-openstack22:12
*** emccormick has quit IRC22:12
*** rahulait has quit IRC22:14
*** emccormick has joined #puppet-openstack22:24
mfischEmilienM: https://bugs.launchpad.net/puppet-vswitch/+bug/155409622:24
openstackLaunchpad bug 1554096 in puppet-vswitch "undefined method vlan? for vs_port on Ubuntu" [Undecided,Confirmed]22:24
EmilienMthanks22:24
*** aimon has quit IRC22:26
*** aimon has joined #puppet-openstack22:27
*** jmikolajczak has joined #puppet-openstack22:27
*** alee has quit IRC22:30
*** chem has quit IRC22:31
openstackgerritMerged openstack/puppet-cinder: Update Cinder BDD backend configuration  https://review.openstack.org/29660622:41
*** tristanC has quit IRC22:43
*** tristanC has joined #puppet-openstack22:44
EmilienMdmsimard: only one test is failing for linuxbridge :-)22:44
EmilienMI'm investigating22:44
EmilienM tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops22:44
EmilienMthe one that test VM access using floating ip22:45
EmilienMand I still have The external network bridge 'br-ex' does not exist22:45
*** jmikolajczak has quit IRC22:46
*** saneax_AFK is now known as saneax22:51
*** imcsk8 has joined #puppet-openstack22:55
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/29610222:55
*** angdraug has quit IRC23:02
*** aimon_ has joined #puppet-openstack23:04
openstackgerritRicardo Rocha proposed openstack/puppet-magnum: Add conductor install and configuration.  https://review.openstack.org/27522323:04
*** alee has joined #puppet-openstack23:05
colonwqEmilienM, Thanks for the FYI. I am locally testing how the change will impact next patch.23:05
*** aimon has quit IRC23:06
*** ducttape_ has joined #puppet-openstack23:10
*** ducttape_ has quit IRC23:16
*** foxhunt72 is now known as foxhunt_away23:18
*** openstack has joined #puppet-openstack23:24
*** openstackstatus has joined #puppet-openstack23:24
*** ChanServ sets mode: +v openstackstatus23:24
*** ansmith has joined #puppet-openstack23:30
*** angdraug has joined #puppet-openstack23:45
dmsimardEmilienM: yeah I made a couple comments on that patch23:45
EmilienMdmsimard: right, I take them in account23:46
EmilienMthe l3 agent logs don't have errors with br-ex anymore23:47
dmsimardwow, apevec actually found the tuned race in the code23:48
EmilienMdmsimard: what was it?23:48
dmsimardit 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#n7623:49
EmilienM:)23:49
openstackgerritMerged openstack/puppet-ceph: Don't clone scenario_node_terminus for acceptance tests anymore  https://review.openstack.org/29416723:57
*** ansmith has quit IRC23:58
*** aimon_ has quit IRC23:58
« Tuesday, 2016-03-22 Index Thursday, 2016-03-24 »

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