Wednesday, 2016-06-15

*** xarses has joined #puppet-openstack00:10
*** ducttape_ has joined #puppet-openstack00:12
*** saneax is now known as saneax_AFK00:15
openstackgerritMerged openstack/puppet-nova: Move nova to use OSDefaults - Part1  https://review.openstack.org/32622100:16
*** rcernin has joined #puppet-openstack00:17
*** ducttape_ has quit IRC00:17
*** Lokesh_Jain has quit IRC00:17
*** rcernin has quit IRC00:22
openstackgerritMerged openstack/puppet-tripleo: add plumgrid neutron profile  https://review.openstack.org/31725900:35
*** rcernin has joined #puppet-openstack00:48
*** cwolferh has quit IRC00:50
*** haaj has quit IRC00:53
*** rcernin has quit IRC00:53
*** ducttape_ has joined #puppet-openstack00:54
*** saneax_AFK is now known as saneax01:00
*** rcernin has joined #puppet-openstack01:05
*** rcernin has quit IRC01:10
*** cwolferh has joined #puppet-openstack01:11
*** furlongm_ is now known as furlongm01:45
*** ilbot3 has quit IRC01:46
*** ilbot3 has joined #puppet-openstack01:48
*** haaj has joined #puppet-openstack01:54
*** saneax is now known as saneax_AFK01:56
*** haaj has quit IRC02:00
*** haaj has joined #puppet-openstack02:56
*** haaj has quit IRC03:01
*** iurygregory_ has quit IRC03:05
openstackgerritMerged openstack/puppet-glance: Switch $service_name to 'glance' for keystone  https://review.openstack.org/31946403:09
*** richm has quit IRC03:11
*** saneax_AFK is now known as saneax03:14
*** morazi has quit IRC03:35
*** ducttape_ has quit IRC03:37
*** ducttape_ has joined #puppet-openstack03:38
*** ducttape_ has quit IRC03:43
*** haaj has joined #puppet-openstack03:57
*** haaj has quit IRC04:02
*** furlongm has quit IRC04:13
*** furlongm has joined #puppet-openstack04:13
*** jmikolajczak has joined #puppet-openstack04:31
*** ducttape_ has joined #puppet-openstack04:38
*** ducttape_ has quit IRC04:43
*** jmikolajczak has quit IRC04:44
*** haaj has joined #puppet-openstack04:58
*** haaj has quit IRC05:04
*** jaosorior has joined #puppet-openstack05:04
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: profile/base/nova: declare nova class properly  https://review.openstack.org/32834705:19
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add fact to get the fqdn for a host in the different networks  https://review.openstack.org/32929905:23
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Drop galera_bootstrapped fact  https://review.openstack.org/32897905:24
*** leanderthal|afk is now known as leanderthal05:30
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Split Heat pacemaker roles into separate services  https://review.openstack.org/32770805:51
openstackgerritSanjay Upadhyay proposed openstack/puppet-openstack-integration: WIP - Adding SR-IOV in puppet openstack intigration  https://review.openstack.org/32976705:52
*** haaj has joined #puppet-openstack06:00
*** saneax is now known as saneax_AFK06:04
*** haaj has quit IRC06:05
openstackgerritzhongshengping proposed openstack/puppet-manila: Provide default service_name for keystone endpoint  https://review.openstack.org/32866706:06
*** jmikolajczak has joined #puppet-openstack06:07
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/32927506:12
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/32859106:12
*** yolanda has joined #puppet-openstack06:13
*** yolanda_ has joined #puppet-openstack06:13
*** yolanda_ has quit IRC06:14
*** openstackgerrit has quit IRC06:18
*** openstackgerrit has joined #puppet-openstack06:18
*** rcernin has joined #puppet-openstack06:29
*** ducttape_ has joined #puppet-openstack06:39
openstackgerritVasyl Saienko proposed openstack/puppet-ironic: Allow to configure cleaning_network_uuid  https://review.openstack.org/32891606:42
openstackgerritVasyl Saienko proposed openstack/puppet-ironic: Allow to set provisioning_network_uuid  https://review.openstack.org/32892106:42
openstackgerritVasyl Saienko proposed openstack/puppet-ironic: Allow to configure enabled_network_interfaces  https://review.openstack.org/32977506:42
*** ducttape_ has quit IRC06:44
*** ifarkas has joined #puppet-openstack06:49
*** dgurtner has joined #puppet-openstack06:58
*** dgurtner has quit IRC06:58
*** dgurtner has joined #puppet-openstack06:58
*** maestro has joined #puppet-openstack06:59
*** amoralej|off is now known as amoralej06:59
*** haaj has joined #puppet-openstack07:01
*** cwolferh has quit IRC07:02
*** tesseract has joined #puppet-openstack07:03
*** rcernin has quit IRC07:04
*** rcernin has joined #puppet-openstack07:04
*** maestro has quit IRC07:05
*** haaj has quit IRC07:07
*** saneax_AFK is now known as saneax07:13
*** shardy has joined #puppet-openstack07:24
*** danpawlik has joined #puppet-openstack07:25
*** jpena|off is now known as jpena07:26
*** hjensas__ has joined #puppet-openstack07:31
*** openstackgerrit has quit IRC07:33
*** openstackgerrit has joined #puppet-openstack07:33
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for keystone  https://review.openstack.org/32702907:34
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for heat  https://review.openstack.org/32706907:34
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for glance API and registry  https://review.openstack.org/32747307:35
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for RabbitMQ  https://review.openstack.org/32748207:35
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for cinder-api  https://review.openstack.org/32885907:35
*** shardy has quit IRC07:43
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for keystone  https://review.openstack.org/32702907:46
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for heat  https://review.openstack.org/32706907:46
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for glance API and registry  https://review.openstack.org/32747307:46
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for RabbitMQ  https://review.openstack.org/32748207:46
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for cinder-api  https://review.openstack.org/32885907:46
*** dtantsur|afk is now known as dtantsur07:50
*** jaosorior is now known as jaosorior_brb07:53
*** jtomasek_ has joined #puppet-openstack07:53
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Add association of the admin user to a domain.  https://review.openstack.org/32980707:55
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Add association of the admin user to a domain.  https://review.openstack.org/32980708:02
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP  https://review.openstack.org/31072508:06
*** jtomasek_ has quit IRC08:10
*** shardy has joined #puppet-openstack08:10
*** dmk0202 has joined #puppet-openstack08:22
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/30906908:22
*** paramite has joined #puppet-openstack08:34
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP  https://review.openstack.org/31072508:35
openstackgerritDaniel Pawlik proposed openstack/puppet-watcher: Implement api.pp  https://review.openstack.org/32983008:38
*** jaosorior_brb has quit IRC08:40
*** jaosorior_brb has joined #puppet-openstack08:41
*** jaosorior_brb is now known as jaosorior08:41
*** evidex has joined #puppet-openstack08:43
*** derekh has joined #puppet-openstack08:45
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP  https://review.openstack.org/31072508:50
*** igajsin has joined #puppet-openstack08:55
*** evidex has quit IRC08:56
*** igajsin has left #puppet-openstack08:57
*** jaosorior has quit IRC08:57
*** foxhunt_away is now known as foxhunt7208:58
*** jtomasek_ has joined #puppet-openstack09:07
*** jtomasek_ has quit IRC09:13
openstackgerritVasyl Saienko proposed openstack/puppet-ironic: Introduce openstack_resource_id_getter function  https://review.openstack.org/32984009:13
*** cdelatte has joined #puppet-openstack09:17
*** evidex has joined #puppet-openstack09:18
*** jaosorior has joined #puppet-openstack09:21
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for keystone  https://review.openstack.org/32702909:29
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for heat  https://review.openstack.org/32706909:29
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for glance API and registry  https://review.openstack.org/32747309:29
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for RabbitMQ  https://review.openstack.org/32748209:29
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for cinder-api  https://review.openstack.org/32885909:29
*** cdelatte has quit IRC09:38
*** permalac has joined #puppet-openstack09:58
openstackgerritVasyl Saienko proposed openstack/puppet-ironic: Allow to configure swift_account  https://review.openstack.org/32985310:01
*** evidex has quit IRC10:04
*** iberezovskiy|off is now known as iberezovskiy10:19
*** dgurtner has quit IRC10:22
*** xb_ has joined #puppet-openstack10:27
openstackgerritMerged openstack/puppet-ceph: Move enable_sig and release to ceph::params  https://review.openstack.org/32954410:33
*** ducttape_ has joined #puppet-openstack10:41
*** ducttape_ has quit IRC10:45
*** jmikolajczak has quit IRC10:46
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Add association of the admin user to a domain.  https://review.openstack.org/32980710:55
*** jmikolajczak has joined #puppet-openstack11:01
*** evidex has joined #puppet-openstack11:01
openstackgerritDmitry Tantsur proposed openstack/puppet-tripleo: Add base ironic profiles  https://review.openstack.org/31929711:03
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: Force symlink creation  https://review.openstack.org/32987611:13
*** dprince has joined #puppet-openstack11:14
*** johnmilton has joined #puppet-openstack11:16
*** johnmilton has quit IRC11:21
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/30906911:27
*** cdelatte has joined #puppet-openstack11:27
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767811:30
*** johnmilton has joined #puppet-openstack11:34
*** johnmilton has quit IRC11:38
*** johnmilton has joined #puppet-openstack11:38
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767811:45
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767811:49
*** paramite has quit IRC11:50
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-heat: Enforce use_ssl flag  https://review.openstack.org/32989011:51
*** dgurtner has joined #puppet-openstack11:53
*** dgurtner has joined #puppet-openstack11:53
*** ducttape_ has joined #puppet-openstack11:53
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: [WIP] Keystone doesn't have service script anymore in UCA  https://review.openstack.org/32956711:56
*** morazi has joined #puppet-openstack12:00
*** jpena is now known as jpena|lunch12:01
*** jayg|g0n3 is now known as jayg12:02
*** ducttape_ has quit IRC12:06
*** ducttape_ has joined #puppet-openstack12:07
*** ducttape_ has quit IRC12:07
*** evidex has quit IRC12:14
*** furlongm_ has joined #puppet-openstack12:16
*** furlongm has quit IRC12:16
iurygregorymorning12:17
*** paramite has joined #puppet-openstack12:17
*** evidex has joined #puppet-openstack12:18
jaosoriorskolekonov: ping12:20
skolekonovjaosorior, I'm here12:21
EmilienMhello12:21
iurygregory\o12:22
*** trown|outtypewww is now known as trown12:23
skolekonovmorning12:23
jaosoriorskolekonov: Hey man, I responded to your comment here https://review.openstack.org/#/c/329890/112:24
jaosoriordo you have any other alternatives for how to check if cert_file and keyfiles should/should not be provided there?12:24
skolekonovjaosorior, thanks, we can use is_service_default for parameters and check use_ssl12:25
jaosoriorI see12:25
jaosoriorskolekonov: can you pass a reference to where that function is used?12:25
jaosoriorEmilienM: Can you take a quick look at https://review.openstack.org/#/c/329890/1 ?12:26
skolekonovIn the similar block for example https://github.com/openstack/puppet-heat/blob/master/manifests/api.pp#L6612:26
jaosoriorlol, funny how I missed that12:27
jaosoriorthanks man12:27
skolekonovnp12:27
EmilienMjaosorior: I'll a bit later12:27
degorenkochem``, ping12:33
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-heat: Enforce use_ssl flag  https://review.openstack.org/32989012:34
*** evidex has quit IRC12:36
iberezovskiyhi12:37
iberezovskiyEmilienM, I was able to pass integration jobs locally (for puppet 4), but currently I'm trying to figure out what dependencies are missed when job is running on CI slaves12:39
iberezovskiyjfyi :)12:40
EmilienMiberezovskiy: excellent progress12:43
*** jmikolajczak has quit IRC12:44
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767812:45
iberezovskiy:)12:45
*** jmikolajczak has joined #puppet-openstack12:56
EmilienMjaosorior: https://review.openstack.org/#/c/329890/ commented13:02
*** dprince has quit IRC13:02
jaosoriorEmilienM: The reason I added those else clauses was pretty much to have those values in the catalog, even if it's the default ones. If you think it's not appropriate to have that else, I can remove it.13:05
EmilienMjaosorior: it's useless, remove it.13:05
jaosoriorwill do13:06
EmilienMdegorenko: thx for https://review.openstack.org/#/c/329694/13:07
EmilienMdegorenko: really I could have added the tests but not sure it's super useful13:07
degorenkoEmilienM, no problem, just was confused.13:08
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-heat: Enforce use_ssl flag  https://review.openstack.org/32989013:08
degorenkoif we have such in nova::libvirt - ok13:08
EmilienMthanks13:08
*** ducttape_ has joined #puppet-openstack13:08
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Fix Blueprints url  https://review.openstack.org/32991413:10
iurygregoryEmilienM, done13:10
EmilienMiurygregory: thx man13:10
iurygregorynp =D13:10
*** jpena|lunch is now known as jpena13:10
*** rcernin has quit IRC13:14
openstackgerritBrad P. Crochet proposed openstack/puppet-tripleo: Add Mistral profiles  https://review.openstack.org/32343113:15
*** amoralej is now known as amoralej|lunch13:16
*** richm has joined #puppet-openstack13:16
*** ducttape_ has quit IRC13:17
*** rcernin has joined #puppet-openstack13:28
*** johnmilton has quit IRC13:35
openstackgerritMerged openstack/puppet-ironic: Allow to configure swift_account  https://review.openstack.org/32985313:37
openstackgerritMerged openstack/puppet-swift: swiftinit provider write file directly  https://review.openstack.org/32966613:38
*** v1k0d3n has quit IRC13:45
vinshyippie13:46
EmilienMyeah!13:47
EmilienMvinsh: thx13:47
vinsh:)13:47
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-heat: Enforce use_ssl flag  https://review.openstack.org/32989013:49
*** ducttape_ has joined #puppet-openstack13:51
*** rcernin has quit IRC13:54
*** amoralej|lunch is now known as amoralej13:56
iberezovskiyEmilienM, xenial jobs are green https://review.openstack.org/#/c/327678/ :)13:58
iberezovskiyplease take a look before I recheck failed centos job13:58
*** dprince has joined #puppet-openstack13:59
*** xarses has quit IRC14:00
*** rcernin has joined #puppet-openstack14:09
*** dgurtner has quit IRC14:11
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/30906914:12
*** foxhunt72 is now known as foxhunt_away14:15
*** rcernin has quit IRC14:18
*** trown is now known as trown|mtg14:18
*** johnmilton has joined #puppet-openstack14:21
*** hjensas__ has quit IRC14:22
*** xarses has joined #puppet-openstack14:23
*** dprince has quit IRC14:25
EmilienMiberezovskiy: excellent new14:27
EmilienMlet me look14:27
EmilienMiberezovskiy: prob related to https://review.openstack.org/#/c/327678/33/run_tests.sh14:28
EmilienMwhy https://review.openstack.org/#/c/327678/33/run_tests.sh?14:28
EmilienMiberezovskiy: did you report this thing to james?14:28
iberezovskiyddt is test requirement, it's not runtime dependency14:28
iberezovskiysame for oslotest14:28
iberezovskiyddt is required for neutron tests, oslotest is test-requirement for tempest14:29
EmilienMiberezovskiy: manage it with puppet then, no?14:29
*** dfisher has joined #puppet-openstack14:29
EmilienMiberezovskiy: for ddt, can't we use apt?14:29
iberezovskiywe can use apt, but I failed to install python-ddt on centos14:30
EmilienMjust install it on ubuntu then14:30
EmilienMuse puppet please, and use tempest.pp14:30
iberezovskiyk14:30
EmilienMthx14:30
*** rcernin has joined #puppet-openstack14:31
EmilienMiberezovskiy: we've never been so close14:31
iberezovskiy:D14:31
dfisherEmilienM: do you have a few minutes to continue to hand-hold me through this neutron quagmire?14:32
*** v1k0d3n has joined #puppet-openstack14:33
openstackgerritMerged openstack/puppet-nova: Enable compute node micro-services  https://review.openstack.org/32969414:34
EmilienMdfisher: go ahead14:34
*** severion has joined #puppet-openstack14:35
dfisherEmilienM: what I need is for manifests/agents/l3.pp to optionally take a router_name (instead of a router_id) and then do a lookup to translate that name to the id for the conf file14:35
dfishersince l3_agent.ini can only use the ID14:35
dfisherbut neutron router_create takes a name14:36
dfisheryou pointed me at https://github.com/openstack/puppet-tempest/blob/master/lib/puppet/provider/tempest_neutron_net_id_setter/openstack.rb last week14:36
dfisherbut i'm not sure how to do something like this.14:36
dfishershould it be a simple Puppet function?  an entire type/provider?  something in lib/puppet/provider/neutron_l3_agent_config/neutron?  none of the above?14:37
*** v1k0d3n has quit IRC14:37
EmilienMdfisher: like I said, you'll need a provider that takes a router name and lookup for you the id and write it in neutron config14:38
EmilienMwe do that for temepst, you have the link14:38
EmilienMyou need to do the same, for the router.14:38
dfishershould I do the same lines() function to just read the whole l3_agent.ini file in?14:39
dfisherthankfully it's not very large14:39
openstackgerritMerged openstack/puppet-monasca: Ensure all servers are listed (beyond osapi_max_limit)  https://review.openstack.org/32969314:40
*** ducttape_ has quit IRC14:41
*** jmikolajczak has quit IRC14:42
*** ducttape_ has joined #puppet-openstack14:43
*** ducttape_ has quit IRC14:45
*** ducttape_ has joined #puppet-openstack14:48
*** v1k0d3n has joined #puppet-openstack14:52
openstackgerritBartek Żurawski proposed openstack/puppet-nova: Add support for MultiStrOpt for nova_config  https://review.openstack.org/32999114:53
*** severion has quit IRC14:55
*** severion has joined #puppet-openstack14:59
*** v1k0d3n has quit IRC15:00
*** v1k0d3n has joined #puppet-openstack15:06
*** severion has quit IRC15:07
*** dgurtner has joined #puppet-openstack15:14
*** dgurtner has quit IRC15:14
*** dgurtner has joined #puppet-openstack15:14
*** ducttape_ has quit IRC15:17
openstackgerritDaniel Pawlik proposed openstack/puppet-watcher: Implement api.pp  https://review.openstack.org/32983015:17
*** danpawlik has quit IRC15:20
*** hjensas__ has joined #puppet-openstack15:22
*** ifarkas has quit IRC15:23
*** leanderthal is now known as leanderthal|afk15:24
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Add support for creating and managing Sahara templates  https://review.openstack.org/30779615:27
*** xarses_ has joined #puppet-openstack15:28
*** severion has joined #puppet-openstack15:29
*** xarses has quit IRC15:29
*** ducttape_ has joined #puppet-openstack15:30
*** v1k0d3n has quit IRC15:31
openstackgerritBrad P. Crochet proposed openstack/puppet-tripleo: Add Mistral profiles  https://review.openstack.org/32343115:31
*** v1k0d3n has joined #puppet-openstack15:31
*** severion has quit IRC15:34
*** shardy has quit IRC15:35
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767815:37
*** saneax is now known as saneax_AFK15:37
EmilienMiberezovskiy: reviewed ^15:44
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767815:45
EmilienMiberezovskiy: +215:46
EmilienMlet's see how it works :P15:46
iberezovskiyI didnt perform manual testing on centos, may be oslotest isn't requred. let's see :)15:46
EmilienMdfisher: someone also wants this kind of thing https://review.openstack.org/#/c/329840/15:48
* dfisher looks15:49
dfisheryeah, something like that :)15:50
*** trown|mtg is now known as trown15:52
*** dmk0202 has quit IRC15:52
*** rcernin has quit IRC15:53
*** ducttape_ has quit IRC15:54
*** tesseract has quit IRC15:55
*** acabot has quit IRC15:57
*** severion has joined #puppet-openstack15:58
*** v1k0d3n has quit IRC15:58
*** ducttape_ has joined #puppet-openstack15:58
openstackgerritIvan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767816:01
iberezovskiyit's not good to hurry, forget to update file titles16:02
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Limit CentOS SIG Storage support to Ceph hammer  https://review.openstack.org/33004116:06
*** dgurtner has quit IRC16:18
*** severion has quit IRC16:18
*** jaosorior has quit IRC16:20
*** v1k0d3n has joined #puppet-openstack16:20
*** derekh has quit IRC16:22
*** v1k0d3n has quit IRC16:23
*** v1k0d3n has joined #puppet-openstack16:24
*** v1k0d3n has quit IRC16:27
*** dprince has joined #puppet-openstack16:31
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Limit CentOS SIG Storage support to Ceph hammer  https://review.openstack.org/33004116:33
*** dgurtner has joined #puppet-openstack16:34
*** jpena is now known as jpena|off16:35
*** cwolferh has joined #puppet-openstack16:36
*** trown is now known as trown|lunch16:39
openstackgerritMerged openstack/puppet-heat: Enforce use_ssl flag  https://review.openstack.org/32989016:44
*** yolanda has quit IRC16:46
colonwqEmilienM, I think you need a "$release = 'hammer" added here: https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/repos.pp#L5516:49
EmilienMI need to take lunch16:49
* EmilienM afk16:50
iberezovskiybon appetit16:50
iberezovskiydegorenko, mwhahaha let's land it, finally it works https://review.openstack.org/#/c/327678/ :D16:52
degorenko\o/16:53
degorenkois it ok, btw, that puppet4 broken?16:54
iberezovskiyyep, I'm on it16:54
iberezovskiyfix for puppet-keystone is required16:54
degorenkokk16:54
iberezovskiywill be published today I believe16:54
iurygregorygtz iberezovskiy16:54
iberezovskiy:)16:55
*** petems has quit IRC16:59
*** igorbelikov has quit IRC16:59
*** dtantsur is now known as dtantsur|afk17:00
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: Update keystone module to support UCA Newton  https://review.openstack.org/33008817:01
*** igorbelikov has joined #puppet-openstack17:01
*** petems has joined #puppet-openstack17:01
iberezovskiydegorenko, please follow up this one ^17:01
degorenkoalready on it17:01
openstackgerritBrad P. Crochet proposed openstack/puppet-tripleo: Add Mistral profiles  https://review.openstack.org/32343117:09
*** amoralej is now known as amoralej|off17:10
*** queretaro has quit IRC17:13
*** dgurtner has quit IRC17:23
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: Update keystone module to support UCA Newton  https://review.openstack.org/33008817:25
iberezovskiyEmilienM, I'll continue to work on this ^ for puppet 417:31
iberezovskiytomorrow17:31
*** iberezovskiy is now known as iberezovskiy|off17:31
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: Update keystone module to support UCA Newton  https://review.openstack.org/33008817:36
*** v1k0d3n has joined #puppet-openstack17:39
beaglesskolekonov: in https://github.com/openstack/puppet-neutron/commit/0d7a2ca5aa6c7be0f5e73d2d4bfd766f63e1e3ac you let the firewall_driver be empty but it still has a puppet specified default value in neutron::agents::ml2::ovs. Was this an oversight or deliberate?17:41
beaglesit creates an interesting situation where the ml2 plugin can have an incompatible configuration with the ovs agent17:42
beagles"by default" actually17:42
*** v1k0d3n has quit IRC17:44
openstackgerritBrad P. Crochet proposed openstack/puppet-tripleo: Add Mistral profiles  https://review.openstack.org/32343117:44
degorenkobeagles, well, that's was added probably with os_service_default - because by default enable_security_group is not true by default in neutron17:45
degorenkooops, many by default :D17:45
degorenkoand also it was a new parameter for ml2 class17:46
degorenkobut not sure17:46
beaglesdegorenko: ah okay, so really the ovs.pp should also have the firewall_driver set to os_service_default as well, correct?17:47
degorenkobeagles, i guess yes17:47
degorenkoand probably should have same warning as ml2.pp if needed17:47
degorenkoos_service_default means - that will be added project default value17:48
degorenkoand ovs.pp just has old *puppet* default value17:48
beaglesright17:48
beaglesI thought os_service_default was kind of a sentinel value that ultimately meant "don't set this"17:49
degorenkoso, i guess we can switch ovs.pp firewall to os_service_default (if needed)17:49
degorenkobeagles, right - it will not be set in config17:49
degorenkoand we rely on project default value17:49
beaglesdegorenko: to fix the problem I am seeing I have to hand in values anyways, I just figured it might be good to have the puppet consistent.17:50
degorenkoso if we have service_default for some parameter - that means, that service will use his own default for this parameter. If service doesn't have such - will be nothing at all17:50
degorenkobeagles, you can upload patch with changing ovs.pp firewall default to os_service_default17:50
degorenkoi think it will ok17:51
beaglesdegorenko: cool17:51
beaglesdegorenko: thanks!17:51
degorenkojust be sure, that we don't have some restrictions for this default like in ml2.pp :)17:51
degorenkosure, no problem17:51
*** trown|lunch is now known as trown17:51
beaglesdegorenko: yeah.. I think I'll have to be pretty careful here actually because people may be relying on this behavior and to alter this would break security groups17:52
degorenkoyeah :)17:52
*** degorenko is now known as _degorenko|afk17:52
beaglesdegorenko: ie. they are setting the firewall_driver in ml2 but relying on the puppet default in ovs - I'll mention that in the commit msg.17:52
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/32859117:55
*** shaikapsar has joined #puppet-openstack18:12
*** cwolferh has quit IRC18:15
*** shaikapsar has quit IRC18:15
*** shaikapsar has joined #puppet-openstack18:16
EmilienMmwhahaha: I see lot of inconsistencies on keystone authtoken18:21
EmilienMI think we should revive https://github.com/openstack/puppet-keystone/blob/master/manifests/resource/authtoken.pp18:21
EmilienMand use it like puppet-oslo18:21
mwhahahayes18:21
mwhahahathat's terrible :D18:21
EmilienMiurygregory: wdyt? ^18:22
*** chem``` has joined #puppet-openstack18:23
*** chem`` has quit IRC18:25
*** cwolferh has joined #puppet-openstack18:26
openstackgerritMerged openstack/puppet-openstack-integration: Force symlink creation  https://review.openstack.org/32987618:31
*** hjensas__ has quit IRC18:33
*** jaosorior has joined #puppet-openstack18:44
colonwqEmilienM, I think you need a "$release = 'hammer" added here: https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/repos.pp#L5518:47
EmilienMcolonwq: go ahead18:47
colonwqEmilienM, Wil do.18:47
EmilienMthx18:47
_odyEmilienM: Last minute before the family got home yesterday, https://github.com/puppetlabs/puppet/pull/5019.18:54
EmilienM_ody: oh nice18:54
EmilienMfinally some progress on this thing18:54
EmilienMit was for virtlog and virtlockd iirc18:55
_odyyeah. Those two were what I used to validate.18:55
_odySomething that might come up, which I don't think is totally fixable.18:55
_odyIf you stry to "disable" those services it will disable them on every run because the are by definition not disableable.18:55
*** xb_ has quit IRC19:02
openstackgerritAndy Smith proposed openstack/puppet-sahara: Add transport_url parameters for oslo.messaging  https://review.openstack.org/32293419:09
*** johnmilton has quit IRC19:16
iurygregoryEmilienM, sorry i was in a meeting i'll take a look in a few minutes brb o/19:17
iurygregoryoh i like the idea to revive the authtoken o/ i can help =D19:17
*** dprince has quit IRC19:21
EmilienMiurygregory: great, let's cleanup the class first, so it's a bit like we do in puppet-oslo19:25
EmilienMiurygregory: and start using it in puppet-nova for ex19:25
EmilienMiurygregory: we can also ask richm to review the params in this define, to make sure we're updated with latest keystonemiddleware thing19:26
richmwhich review?19:27
EmilienMrichm: see https://github.com/openstack/puppet-keystone/blob/master/manifests/resource/authtoken.pp19:28
EmilienMI think we can make it revive and used by our puppet-* modules19:28
EmilienMto configure keystone authtoken19:28
EmilienMinstead of everytime inventing the wheel19:28
richmEmilienM: yeah, that was my original intention19:29
EmilienM++19:29
EmilienMlet's do it19:29
EmilienMrichm: somethhing blocked you? just to make sure we're not missing something19:29
richmEmilienM: 1) the only place I used it was in my glance review that has since rotted - https://review.openstack.org/#/c/176150/ "Updated 10 months ago"19:31
openstackgerritMerged openstack/puppet-openstack-integration: repos: deploy newton repo on ubuntu xenial  https://review.openstack.org/32767819:31
richm2) I have not reviewed the latest keystonemiddleware parameters to see if manifests/resource/authtoken.pp is up-to-date19:31
EmilienMwe need to make 2)19:32
EmilienMbefore doing the modules19:32
richmI'm not sure what the plan is for adding keystone v3 support to all of the modules19:32
EmilienMwell, we need it19:32
*** openstackgerrit has quit IRC19:33
EmilienMiurygregory: let me know if you need help19:33
EmilienMrichm: we'll probably ask you to review, or maybe ayoung19:33
EmilienMiurygregory: we'll probably help you in the integration (modules)19:33
*** openstackgerrit has joined #puppet-openstack19:33
richmI'll see if I can find someone on my team who can help with this - not sure if ayoung is the right person - we have some other installer guys now19:33
ayoungNot It!19:34
ayoungin the middle of a debugging session at the moment19:34
EmilienMayoung is always the best person19:34
ayounglies19:34
jmiulol19:34
ayoungI'm the worst19:34
EmilienMI have a problem at home, need someone to fix water19:34
EmilienMayoung? ^19:34
ayoungEmilienM, there I can't help you19:34
openstackgerritKeith Schincke proposed openstack/puppet-openstack-integration: Update to limit CentOS SIG to hammer  https://review.openstack.org/33016819:36
mwhahahaworkaround: move19:36
EmilienM# TODO(emilien): fix later19:37
EmilienMit would be great if you could pin your life, like we pin repositories19:37
*** ducttape_ has quit IRC19:38
openstackgerritBrad P. Crochet proposed openstack/puppet-tripleo: Add Mistral profiles  https://review.openstack.org/32343119:40
openstackgerritEmilien Macchi proposed openstack/puppet-nova: migration: split libvirt & nova configs  https://review.openstack.org/33017219:46
richmiurygregory: a good place to start is the https://review.openstack.org/#/c/176150/19:48
EmilienMrichm, iurygregory: the thing I would like to see first is https://review.openstack.org/#/c/176150/11/manifests/api.pp19:49
EmilienMbefore providers19:49
EmilienMbut yeah, we need providers too19:49
richmEmilienM: do you mean - that review should be split into separate reviews - one which is just for the keystone::resource::authtoken changes, and one for the other v3 changes?19:52
EmilienMyes19:52
EmilienMrichm: I want to see config first19:52
EmilienMand once we have it, ok let's move providers19:52
EmilienMspecially recently, we have new modules19:52
EmilienMand they're all inventing a new wheel for keystone authtoken19:52
EmilienMso before we get crazy, let's fix it19:52
*** jaosorior has quit IRC19:57
*** vinsh_ has joined #puppet-openstack20:02
*** vinsh has quit IRC20:06
*** dprince has joined #puppet-openstack20:07
*** xarses_ has quit IRC20:08
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/32927520:14
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001/ubuntu: re-enable telemetry  https://review.openstack.org/33018820:21
EmilienMiberezovskiy|off: ^20:21
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration:  scenario002/ubuntu: re-enable ironic  https://review.openstack.org/33018920:23
*** xarses_ has joined #puppet-openstack20:24
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003/ubuntu: re-enable trove  https://review.openstack.org/33019120:26
dfisherEmilienM: thank you for your help.  I think I have something working.20:34
EmilienMdfisher: cool don't forget to send a patch20:34
dfisherso, router_id is deprecated in Mitaka20:34
dfisherbecause use_namespaces is the default behavior20:35
dfisherSolaris doesn't yet support IP namespaces (/sigh)20:35
dfisherso, it's really only a < Mitaka and Solaris patch20:35
openstackgerritDan Radez proposed openstack/puppet-congress: Populating init.pp and parameters  https://review.openstack.org/33019520:35
EmilienMJun 15 19:52:56 localhost pengine[11455]: warning: Processing failed op start for ip-fd00.fd00.fd00.3000..18 on overcloud-controller-0: unknown error (1)20:35
EmilienMdamn20:35
EmilienMwrong window sorry20:35
EmilienMok our beaker jobs are broken20:47
EmilienMwe need https://review.openstack.org/#/c/330088/20:47
*** EmilienM changes topic to "CI status: red - all beaker jobs broken - we need https://review.openstack.org/#/c/330088/ - Puppet OpenStack modules http://docs.openstack.org/developer/puppet-openstack-guide/"20:49
*** shaikapsar has quit IRC20:52
*** v1k0d3n has joined #puppet-openstack20:57
iurygregoryi'm alive \o/21:02
iurygregoryone day monasca will kill me lol21:02
iurygregoryEmilienM, richm ack =D will take a look tonight21:07
richmiurygregory: thanks21:07
iurygregoryi'm leaving the office right now but i'll be back latter =D21:07
*** dprince has quit IRC21:13
*** trown is now known as trown|outtypewww21:15
*** vinsh_ has quit IRC21:20
*** vinsh has joined #puppet-openstack21:22
*** jayg is now known as jayg|g0n321:24
openstackgerritDan Radez proposed openstack/puppet-congress: Populating init.pp and parameters  https://review.openstack.org/33019521:28
openstackgerritMerged openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/32927521:44
*** openstackgerrit has quit IRC22:02
*** openstackgerrit has joined #puppet-openstack22:06
*** paramite has quit IRC22:08
*** dfisher has quit IRC22:23

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