Monday, 2015-05-11

*** sdake_ has joined #tripleo00:08
*** sdake_ has joined #tripleo00:09
*** sdake has quit IRC00:12
*** weshay has quit IRC00:28
*** yamahata has quit IRC00:55
*** lsmola_ has joined #tripleo01:01
*** dtantsur|pto has quit IRC01:03
*** dtantsur has joined #tripleo01:03
*** lsmola has quit IRC01:04
*** julim has joined #tripleo01:05
*** yuanying has joined #tripleo01:10
*** dtantsur has quit IRC01:10
*** dtantsur has joined #tripleo01:11
*** lsmola_ has quit IRC01:11
*** lsmola_ has joined #tripleo01:25
*** julim has quit IRC01:48
*** rbrady_ has joined #tripleo02:03
*** rbrady has quit IRC02:05
*** untriaged-bot has joined #tripleo03:00
untriaged-botUntriaged bugs so far:03:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/145240003:00
openstackLaunchpad bug 1452400 in diskimage-builder "debian-minimal builds still busted, now on locale issues" [Undecided,New]03:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985203:00
openstackLaunchpad bug 1449852 in diskimage-builder "Buidling ramdisk with ironic-agent behind proxy fails" [Undecided,In progress] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)03:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985403:00
openstackLaunchpad bug 1449854 in diskimage-builder "Ironic agent ramdisk built using disk-image-create fails with iscsi_ilo driver" [Undecided,In progress] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)03:00
untriaged-bothttps://bugs.launchpad.net/tuskar/+bug/145275203:00
openstackLaunchpad bug 1452752 in tuskar "keystone_authtoken section is wrong in default shipped tuskar.conf.sample" [Undecided,Confirmed]03:00
*** untriaged-bot has quit IRC03:00
*** sdake has joined #tripleo03:05
*** sdake has joined #tripleo03:05
*** sdake_ has quit IRC03:06
*** sdake has quit IRC03:08
*** sdake has joined #tripleo03:09
*** panda|afk has quit IRC03:13
*** panda|afk has joined #tripleo03:14
*** gfidente has joined #tripleo03:37
*** subscope has joined #tripleo03:38
*** sdake_ has joined #tripleo03:56
*** sdake has quit IRC04:00
*** eghobo has joined #tripleo04:04
*** gfidente^2nd has joined #tripleo04:22
*** gfidente has quit IRC04:23
*** sdake has joined #tripleo04:34
*** sdake_ has quit IRC04:38
openstackgerritSteve Baker proposed openstack/tripleo-heat-templates: Config & deployments to update overcloud packages  https://review.openstack.org/17997404:59
*** chlong has quit IRC05:09
*** sdake_ has joined #tripleo05:13
*** sdake has quit IRC05:16
*** daneyon has joined #tripleo05:25
*** masco has joined #tripleo05:30
*** sdake has joined #tripleo05:57
*** sdake_ has quit IRC06:01
*** sdake_ has joined #tripleo06:15
*** sdake__ has joined #tripleo06:18
*** sdake has quit IRC06:18
*** sdake_ has quit IRC06:21
*** ukalifon1 has joined #tripleo06:23
*** athomas has joined #tripleo06:24
*** jcoufal has joined #tripleo06:31
*** yog__ has joined #tripleo06:35
*** mmagr has joined #tripleo06:36
openstackgerritMerged openstack/tripleo-incubator: devtest_overcloud.sh: split out image building  https://review.openstack.org/17364506:36
openstackgerritMerged openstack/tripleo-incubator: load-images: load images via YAML config file  https://review.openstack.org/17502806:37
*** shardy has joined #tripleo06:40
*** Marga_ has joined #tripleo07:01
*** Marga_ has quit IRC07:04
*** subscope has quit IRC07:05
*** eghobo has quit IRC07:11
*** mmagr has quit IRC07:11
*** panda|afk has quit IRC07:13
*** panda|afk has joined #tripleo07:14
*** jtomasek has joined #tripleo07:15
*** Marga_ has joined #tripleo07:19
*** Marga_ has quit IRC07:19
*** mmagr has joined #tripleo07:27
*** Goneri has joined #tripleo07:27
*** sdake__ has quit IRC07:28
*** hewbrocca has joined #tripleo07:30
*** aufi has joined #tripleo07:32
*** spredzy|afk is now known as spredzy07:32
*** gfidente^2nd is now known as gfidente07:44
*** gfidente has quit IRC07:44
*** gfidente has joined #tripleo07:44
openstackgerritDougal Matthews proposed openstack/python-tuskarclient: Implement Update Plan for the OpenStack client  https://review.openstack.org/17921907:48
openstackgerritDougal Matthews proposed openstack/python-tuskarclient: Implement show Plan for the OpenStack client  https://review.openstack.org/17922007:48
openstackgerritDougal Matthews proposed openstack/python-tuskarclient: Implement Plan add Role for the OpenStack client  https://review.openstack.org/17922107:48
openstackgerritDougal Matthews proposed openstack/python-tuskarclient: Implement Plan remove Role for the OpenStack client  https://review.openstack.org/17922207:48
openstackgerritDougal Matthews proposed openstack/python-tuskarclient: Implement download Plan for the OpenStack client  https://review.openstack.org/17922307:48
*** ifarkas has joined #tripleo07:49
openstackgerritDougal Matthews proposed openstack/python-tuskarclient: Implement download Plan for the OpenStack client  https://review.openstack.org/17922307:49
*** ishant has joined #tripleo07:56
*** jistr has joined #tripleo07:58
*** mcornea has joined #tripleo08:01
*** lucasagomes has joined #tripleo08:04
*** derekh has joined #tripleo08:22
*** yog__ has quit IRC08:28
*** yog__ has joined #tripleo08:45
*** untriaged-bot has joined #tripleo09:00
untriaged-botUntriaged bugs so far:09:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/145240009:00
openstackLaunchpad bug 1452400 in diskimage-builder "debian-minimal builds still busted, now on locale issues" [Undecided,New]09:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985209:00
openstackLaunchpad bug 1449852 in diskimage-builder "Buidling ramdisk with ironic-agent behind proxy fails" [Undecided,In progress] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)09:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985409:00
openstackLaunchpad bug 1449854 in diskimage-builder "Ironic agent ramdisk built using disk-image-create fails with iscsi_ilo driver" [Undecided,In progress] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)09:00
untriaged-bothttps://bugs.launchpad.net/tuskar/+bug/145275209:00
openstackLaunchpad bug 1452752 in tuskar "keystone_authtoken section is wrong in default shipped tuskar.conf.sample" [Undecided,Confirmed]09:00
*** untriaged-bot has quit IRC09:00
*** regebro has joined #tripleo09:01
*** jprovazn has joined #tripleo09:11
*** yog__ has quit IRC09:12
*** yog__ has joined #tripleo09:23
*** pblaho has joined #tripleo09:25
*** derekh has quit IRC09:27
*** derekh has joined #tripleo09:28
*** subscope has joined #tripleo09:43
*** rlandy has joined #tripleo09:53
*** dasm|afk is now known as dasm09:55
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Add Memcache as a Pacemaker resource when EnablePacemaker  https://review.openstack.org/17982009:57
*** athomas has quit IRC10:00
*** shardy_ has joined #tripleo10:04
*** shardy has quit IRC10:06
dasmgood afternoon :)10:07
*** athomas has joined #tripleo10:08
*** shardy_ has quit IRC10:10
*** shardy has joined #tripleo10:10
*** akrivoka has joined #tripleo10:11
*** akrivoka has quit IRC10:14
openstackgerritJan Provaznik proposed openstack/tripleo-common: Scale out heat stack  https://review.openstack.org/17328310:14
*** pdb_ has quit IRC10:14
*** pdb_ has joined #tripleo10:16
*** akrivoka has joined #tripleo10:17
*** gfidente has quit IRC10:19
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Add Galera as a Pacemaker resource when EnablePacemaker  https://review.openstack.org/17776510:24
*** pelix has joined #tripleo10:30
*** pdb_ has quit IRC10:31
*** pdb_ has joined #tripleo10:33
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Add Galera as a Pacemaker resource when EnablePacemaker  https://review.openstack.org/17776510:33
*** yuanying has quit IRC10:34
*** pdb_ has quit IRC10:34
jistrgood afternoon :)10:47
openstackgerritDougal Matthews proposed openstack/python-tuskarclient: Implement Update Plan for the OpenStack client  https://review.openstack.org/17921910:48
openstackgerritDougal Matthews proposed openstack/python-tuskarclient: Implement show Plan for the OpenStack client  https://review.openstack.org/17922010:48
openstackgerritDougal Matthews proposed openstack/python-tuskarclient: Implement Plan add Role for the OpenStack client  https://review.openstack.org/17922110:48
openstackgerritDougal Matthews proposed openstack/python-tuskarclient: Implement Plan remove Role for the OpenStack client  https://review.openstack.org/17922210:48
openstackgerritDougal Matthews proposed openstack/python-tuskarclient: Implement download Plan for the OpenStack client  https://review.openstack.org/17922310:48
d0ugalGood morning :)10:49
*** pdb has joined #tripleo10:50
dasmd0ugal: yeah... we're missing TZ on IRC ;)10:50
d0ugalhah10:52
d0ugalIndeed!10:52
openstackgerritMerged openstack/python-tuskarclient: Show Plan parameters with empty values only  https://review.openstack.org/17821010:53
*** gfidente has joined #tripleo10:55
jistrgfidente: heya, i'm +1 on making CI run in debug mode, but should this be a TOCI patch rather than switching the default in t-h-t? https://review.openstack.org/#/c/181332/10:56
gfidentejistr, ahm that was only meant to get a debug run to investigate into CI issues10:57
gfidenteso I have workflow -110:57
jistrah i see10:57
gfidentedo you think we should?10:57
gfidentethe issue was this: https://review.openstack.org/#/c/181357/ , see the depends-on10:58
jistrwell anyway, yeah i'm thinking we should have debug on always in CI... it's going to make a bit larger log files probably, but it might help debugging some issues10:58
jistrgfidente: seems like the patches ran CI before the toci temprevert was in place, gave +2s and rechecks11:07
*** panda|afk has quit IRC11:13
*** panda|afk has joined #tripleo11:14
*** EmilienM|afk is now known as EmilienM11:28
*** morazi has quit IRC11:31
*** lucasagomes is now known as lucas-hungry11:41
*** akrivoka has quit IRC11:41
*** subscope has quit IRC11:44
*** jdob has joined #tripleo11:45
*** subscope has joined #tripleo11:47
*** trown|outttypeww is now known as trown11:51
*** akrivoka has joined #tripleo11:54
*** dprince has joined #tripleo12:02
openstackgerritJiri Stransky proposed openstack/diskimage-builder: Make managing hosts entries optional  https://review.openstack.org/17720312:07
gfidentedprince, the change which switches .pp12:08
gfidenteI think we need to change the type of ControllerPuppetConfig in controller-post-puppet.yaml12:08
gfidentemaking it point to the registry resource12:08
*** tzumainn_ has joined #tripleo12:10
dprincegfidente: yes, we do12:11
*** weshay has joined #tripleo12:12
*** Marga_ has joined #tripleo12:12
dprincespredzy: morning, thanks for the feedback on https://review.openstack.org/#/c/180833/3. I'm willing to make the changes you suggest, but wanted to point out that initially overcloud_controller_pacemaker.pp was just a plain copy so as to make rebasing easier for the in-flight patches.12:13
mariosdprince: dude it is all borked now anyway just do it an this point, imo12:13
*** _dguerri is now known as dguerri12:13
mariosdprince: sooner than later let's get it done so we can rebase12:13
*** jayg|g0n3 is now known as jayg12:14
dprincemarios: okay, I'm up for that12:14
spredzydprince, morning. To me, inflight change will need to remove the $enable_pacemaker , else -1 until its done12:14
spredzysince it is the purpose of this split12:14
dprincespredzy: agree. lets do it12:14
mariosdprince: let's include that yeah spredzy +112:14
mariosdprince: if am rebasing anyway, will remove those in mine12:15
spredzyas gfidente mentioned earlier I tried the patch, edited the proper file but I m getting always the overcloud_controller.pp and not overcloud_controller_pacemaker.pp manifest in12:15
dprincespredzy: the end goal here is to (eventually) get us back to a single manifest.12:15
spredzyin == in the controller node, I applied locally the change you did in the tripleo-ci proj12:15
spredzydprince, yep I read the thread in the ML this morning12:15
spredzywill take more time to think it thoroughly and put my grain of salt there12:16
spredzybut I do get why we're doing it12:16
gfidentespredzy, the error is due to missing reference to OS::TripleO::ControllerConfig in post-puppet.yaml though12:17
mariosgfidente: am trying it now man12:21
*** Marga_ has quit IRC12:21
*** Marga_ has joined #tripleo12:22
d0ugalpblaho: Can you re-review https://review.openstack.org/#/c/179219/ please?12:22
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Puppet: Split out controller pacemaker manifest  https://review.openstack.org/18083312:23
spredzydprince, thanks :)12:28
dprincespredzy: sure, np12:30
*** rhallisey has joined #tripleo12:34
*** mcornea has quit IRC12:36
*** mcornea has joined #tripleo12:37
*** lucas-hungry is now known as lucasagomes12:37
*** sdake has joined #tripleo12:38
openstackgerritJiri Stransky proposed openstack/diskimage-builder: Make managing hosts entries optional  https://review.openstack.org/17720312:40
*** sdake_ has joined #tripleo12:40
jistrgfidente: ^ updated12:41
gfidentejistr, thanks +112:41
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Add metering agent as neutron-server plugin  https://review.openstack.org/17937012:42
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Add neutron-lbaas as neutron-server plugin & neutron-lbaas-agent  https://review.openstack.org/18053512:42
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Adds neutron-server and agents as pacemaker resources  https://review.openstack.org/18101512:42
mariosdprince: thanks, updated12:42
mariosdprince: am going to try these in a sec12:42
marios(i mean including your one)12:42
*** sdake has quit IRC12:44
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Add Galera as a Pacemaker resource when EnablePacemaker  https://review.openstack.org/17776512:45
openstackgerritGiulio Fidente proposed stackforge/puppet-tripleo: Add support for haproxy_service_manage  https://review.openstack.org/18048412:45
openstackgerritDan Prince proposed openstack-infra/tripleo-ci: Enable pacemaker via the resource registry  https://review.openstack.org/18152212:46
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Add Memcache as a Pacemaker resource when EnablePacemaker  https://review.openstack.org/17982012:50
pblahod0ugal: done12:52
*** sdake_ is now known as sdake12:55
*** dobson has quit IRC12:56
*** dobson has joined #tripleo12:57
d0ugalpblaho: thanks!12:59
pblahod0ugal: np12:59
*** radez_g0n3 is now known as radez13:01
*** ukalifon1 has quit IRC13:02
*** ukalifon has joined #tripleo13:02
spredzyshardy, dprince I'd like to know if it was possible t specify an entry file here conditionally https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/controller-puppet.yaml#L648-L658 (ie. if pacemaker enabled then include a file with predetermined conf) else not13:04
spredzythe only way I see so far is to generate it in controller-config-pacemaker.yaml, which is not elegant13:05
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Add Glance as a Pacemaker resource when EnablePacemaker  https://review.openstack.org/17815213:14
spredzyAnyone know Jason Guiditta IRC nick ?13:14
spredzyjayg, ^ ? is that you ?13:14
gfidentemaybe he is hiding13:15
jaygspredzy: yep, thats me13:15
gfidente:P13:15
spredzyahah never new what your last name was, you were always jayg to me :)13:15
jayg:)13:15
spredzyAbout https://review.openstack.org/#/c/177765/ and your comment. Is it still happening ? How did you run the deployment ?13:15
jaygI have been unable to get it to create the tables, but I think it worked for gfidente13:16
gfidenteit worked for me yes13:16
jaygI am using 3 controllers, gfidente - did you do the same on your test?13:16
gfidentejayg, none of the attempts succeeded for you?13:17
gfidenteor was it intermittent?13:17
gfidente3 controllers yes13:17
jayggfidente: I have never seen a table created on multiple tries13:17
gfidenteauch13:17
jaygand rabbit continues to be an issue, so perhaps it is just something with my setup...13:17
spredzyjayg, are the Exec[sync_db] run on the master node ?13:18
jaygI was going to hack how puppet appy is run so I can get more info, planning to do that today13:18
gfidenterabbit continues to be an issue with the config settings applied?13:18
jayggfidente: yes, still multiple pcs fails with it13:18
jaygspredzy: is there an easy way to find out which one is master based on config?13:19
spredzyjayg, master is node 013:19
gfidentehiera bootstrap_nodeid13:19
jaygI can look in that journalctl mess for it for now..13:19
gfidentewill tel13:19
jaygok13:19
spredzyalso, but from the undercloud, if you do a nova list once deployed (even in failed state) log into the controller-0 node, it will be the master13:19
spredzyjayg, ^13:20
gfidentejayg, so I am going to try and will ping you13:20
jaygok, I'll look - gfidente sounds good, thx13:21
*** chlong has joined #tripleo13:22
*** cinerama has quit IRC13:22
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Ensure haproxy is not enabled by puppet-tripleo when EnablePacemaker  https://review.openstack.org/18049013:23
*** masco has quit IRC13:23
jaygspredzy: nope, grepping 'journalctl -u os-collect-config' on node 0 shows no sync_db calls at all13:23
*** cinerama has joined #tripleo13:26
jistrjayg: i think it's s/sync_db/db_sync/13:26
gfidentedprince, I know there is a lot wip already, wanted to know if the 'environments' dir you mentioned is something you have a change up already13:27
*** morazi has joined #tripleo13:27
jistr(though $sync_db is variable in puppet modules, so it's a bit confusing which is which... :) )13:28
gfidenteceph and pacemaker are the two which could use it to start with13:28
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Use pacemaker::resource::service instead of ::systemd  https://review.openstack.org/18050613:28
spredzyjayg, jistr is right, when I mentionned sync_db I mentionned the action itself13:29
spredzyjayg, look at the log for the 4th puppet apply run13:29
spredzythis is when they happen13:29
jayggrep for db_sync also return nada13:29
shardyspredzy: Hi, atm Heat doesn't support template conditionals13:30
spredzyshardy, thank you for the answer. At least it is clear13:31
shardyspredzy: it's been discussed, and AWS supports it, so we'll probably end up adding it at some point, but right now I think your options are either unconditionally include the file and do the conditional in puppet, or have a subset of the config defined in a different template13:31
jistrjayg, spredzy: it could be that the overcloud heat stack creation didn't get to the 4th run then... `heat stack-list` on seed/undercloud shows create_failed?13:31
hewbroccaI still don't understand why we're not using Tuskar to assemble a complete template from a set of more-or-less immutable fragments13:32
hewbrocca... but maybe that's just me13:32
jaygjistr: yes, it always shows create failed on multi-node cluster setup to date13:33
spredzyshardy, adding the content of the file directly in the controller-config-pacemaker.yaml file would be a bad idea ? Something similar to that https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/all-nodes-config.yaml#L49-L72 but with hardcoded string for now13:33
jistrjayg: then grepping `journalctl -u os-collect-config` for 'Error' might reveal more13:34
jaygjistr: gfidente explained to me that had something to do with configuration being incomplete from heats view, though I may be remembering that explanation wrong13:34
shardyhewbrocca: you could do that via the "subset of the config" approach, which is the same pattern we use already e.g to switch between puppet and element based implementations13:34
shardyhewbrocca: however if you also want to allow overcloud deployments directly via heat, you have to stop short of actual pre-preprocesing in Tuskar IMO, or it becomes merge.py as a service ;)13:35
jaygjistr: hmm, that node does show some cib-related errors, like Error: unable to get cib13:35
jayghowever, the cluster is running at the end13:36
hewbroccahmm I see13:36
hewbroccawell, I'll let you guys figure this out13:36
jaygthis will be easier to debug once I have puppet output go to a file...13:37
shardyspredzy: yeah, you could do that, and switch the StructuredConfig in the parent template to an alias mapped via the resource_registry13:37
dprincegfidente: https://review.openstack.org/#/c/181271/13:38
shardye.g like we do for various other *SoftwareConfig resources already where you need to switch between implementations13:38
dprincegfidente: I sort of think that belongs in t-h-t now though13:38
* dprince has mixed feelings on the location of these13:38
shardyspredzy: maybe you end up with a ha_environment.yaml which specifies the mappings needed to pull in and enable the HA pieces?13:39
*** prad has joined #tripleo13:39
gfidentedprince, yeah I see reasons for having it in both as well13:39
dprincespredzy: if what you are going for is a custom hiera file I think we can do that. And it would be a great way to consolidate some of the pacemaker stuff (like ensure => disabled", etc.)13:40
shardyspredzy: similar to what dprince just posted actually, e.g you could have parameters, resource_registry mappings and potentially parameter_defaults for nested templates as required to enable the pacemaker configuration13:40
shardyhttps://review.openstack.org/#/c/181271/1/environments/ceph.yaml13:40
gfidentedprince, so I am plusvoting it there for now13:41
gfidentedprince, shardy and how close is support for multiple envs ?13:42
*** julim has joined #tripleo13:42
jdobgfidente: multiple envs works now; we are using it for the satellite integration13:43
jdobunlesss i'm misunderstanding what you're asking13:43
gfidentejayg, hack the cmdline in /var/lib/heat-config/hook13:43
shardygfidente: Yeah, as jdob says, heatclient has supported multiple environments for a while now13:44
gfidentejdob, passing multiple -e to heatclient?13:44
jdobright13:44
gfidenteI thought it couldn't, thanks :)13:44
shardyso it should be fairly easiy to pass multiple environment "snippets" which enable a specific configuration13:44
jdobnot sure the policy on conflicts or what not13:44
jdobbut i know it can be used for aggregation13:45
shardyjdob: last one wins, basically13:45
jdobmeaning -e first -e second and second is consdiered "last"?13:45
jayggfidente: that is on my host/bare metal machine, right?13:45
shardywe just do a merge of each environment in sequence13:45
shardyjdob: right13:45
jdobkk13:45
jdobeasy enough :)13:45
gfidentejayg, in the image actually, so you get it when redeploy13:45
*** prad has quit IRC13:47
*** yog__ has quit IRC13:47
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Use optimized config for RabbitMQ cluster  https://review.openstack.org/18141313:47
gfidentejayg, ^^ this I had to rebase but is same you are testing13:48
spredzyshardy, the issue I see with the snipett you sent is that it would require to create new template parameter wouldnt it ? When really what is needed is if pacemaker manage_service => false, else manage_service => true for example13:48
*** prad has joined #tripleo13:48
jaygok, fair enough13:49
shardyspredzy: it's possible to just define e.g a ManageService parameter in your nested pacemaker-specific template, then pass it in via parameter_defaults without modifying the parent template(s)13:49
shardyhttps://github.com/openstack/tripleo-heat-templates/blob/master/overcloud-resource-registry-puppet.yaml#L2613:50
shardye.g like EnablePackageInstall13:50
spredzyhmmm I haven't thought of doing it that way13:51
*** rhallisey has quit IRC13:52
spredzyshardy, thanks. That might just work :)13:52
*** rhallisey has joined #tripleo13:55
*** openstackgerrit has quit IRC14:06
*** openstackgerrit has joined #tripleo14:07
*** jprovazn has quit IRC14:12
openstackgerritDan Prince proposed openstack/tripleo-incubator: Support multiple custom overcloud heat envs  https://review.openstack.org/18190914:14
dprincestevebaker: your idea ^^^. But we'll need it soon I think14:15
dprincestevebaker: oops. Wrong Steve ;)14:15
dprinceshardy: your idea ^^^ :)14:15
*** sdake_ has joined #tripleo14:16
jdobdprince: does that mean we're definitely going to be needing multiple envs to define the HA stuff?14:16
dprincegfidente: ^^^ this is a simple way to support multiple envs in devtest14:16
jdobor is this just so you can try something out in devtest?14:16
dprincejdob: for pacemaker we are already okay I think. SO here: https://review.openstack.org/#/c/181522/14:17
jdobdprince: i'm thinking in terms of tuskar14:17
*** ishant has quit IRC14:17
jdobif we need to be able to load in multiple envs14:17
dprincejdob: I want this... thinking ahead to how I would document how to use more complex overlays like the network architecture14:17
*** ishant has joined #tripleo14:18
dprincejdob: which make use of the resource registry to enable these things very cleanly/easily14:18
*** openstack has joined #tripleo14:20
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Add Galera as a Pacemaker resource when EnablePacemaker  https://review.openstack.org/17776514:20
dprincejdob: the network stuff will, yes. I think tuskar gets a pass on not supporting it initialy though14:20
jdobfrom an upstream perspective, a pass is fine. i'm trying to figure out if this is gonna impact OSP 714:21
dprincejdob: not sure... I'm the one to ask about that. I think this was slated as an advanced CLI only feature there14:22
jdobi'm not asking you as a PM, I'm asking you in the sense of if functionality in the next 3-4 weeks will require it, which it sounds like it will for networking14:23
*** dasm is now known as dasm|afk14:24
*** bnemec has joined #tripleo14:25
*** rhallisey has quit IRC14:26
*** rhallisey has joined #tripleo14:27
openstackgerritOpenStack Proposal Bot proposed openstack/os-cloud-config: Updated from global requirements  https://review.openstack.org/17841814:28
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Add Galera as a Pacemaker resource when EnablePacemaker  https://review.openstack.org/17776514:29
*** sdake has joined #tripleo14:32
EmilienMspredzy: I think we should really start to write rspec unit tests for puppet-tripleo14:32
*** ukalifon has quit IRC14:33
*** sdake_ has quit IRC14:35
*** prad has quit IRC14:37
hewbroccaEmilienM: +114:37
*** jprovazn has joined #tripleo14:38
*** mestery has quit IRC14:38
hewbroccajdob: yes, but I think what dprince is saying is that Tuskar doesn't need to support advanced network management in the next sprint14:39
*** ukalifon1 has joined #tripleo14:41
spredzyEmilienM, def. could be a story for next sprint better start sooner than later, less to cope with14:42
EmilienMspredzy: cool, wfm14:43
openstackgerritOpenStack Proposal Bot proposed openstack/tuskar-ui: Updated from global requirements  https://review.openstack.org/18192114:47
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Provide RabbitMQ clients with a list of servers instead of VIP  https://review.openstack.org/18108114:48
*** rameshg87 has joined #tripleo14:54
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Provide RabbitMQ clients with a list of servers instead of VIP  https://review.openstack.org/18108114:56
rameshg87hello tripleo cores - can someone please review - https://review.openstack.org/#/c/179992/ https://review.openstack.org/#/c/178512/ - already a +2 on it14:59
*** untriaged-bot has joined #tripleo15:00
untriaged-botUntriaged bugs so far:15:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/145240015:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985215:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985415:00
openstackLaunchpad bug 1452400 in diskimage-builder "debian-minimal builds still busted, now on locale issues" [Undecided,New]15:00
untriaged-bothttps://bugs.launchpad.net/tuskar/+bug/145275215:00
*** untriaged-bot has quit IRC15:00
openstackLaunchpad bug 1449852 in diskimage-builder "Buidling ramdisk with ironic-agent behind proxy fails" [Undecided,In progress] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)15:00
openstackLaunchpad bug 1449854 in diskimage-builder "Ironic agent ramdisk built using disk-image-create fails with iscsi_ilo driver" [Undecided,In progress] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)15:00
openstackLaunchpad bug 1452752 in tuskar "keystone_authtoken section is wrong in default shipped tuskar.conf.sample" [Undecided,Confirmed]15:00
*** daneyon_ has joined #tripleo15:06
*** masco has joined #tripleo15:06
*** daneyon has quit IRC15:09
*** panda|afk has quit IRC15:13
*** yamahata has joined #tripleo15:14
*** ishant has quit IRC15:14
*** panda|afk has joined #tripleo15:14
openstackgerritMerged openstack/tuskar-ui: Updated from global requirements  https://review.openstack.org/18192115:21
*** masco has quit IRC15:21
gfidentedprince, shardy, I remember you did not like the & * notation15:31
gfidentebut I couldn't find a much better way of doing this https://review.openstack.org/#/c/181081/15:31
gfidentebecause interpolation from hiera is turning arrays into strings anyway (making the include in .pp to fail)15:32
gfidenteso can you think of alternatives or is that good to go?15:32
dprincegfidente: I would defer to steve on this. If it works I think I'm fine w/ it15:33
gfidenteit works as long as the param is passed along in the yaml15:34
*** dasm|afk is now known as dasm15:35
gfidentesorry, in the template, but not if you try the aliasing the hieradata using %{}15:35
gfidenteso I had to do the & * thing15:35
gfidentebut looks like that is by design, as per note from https://docs.puppetlabs.com/hiera/1/variables.html it says. "  Hiera can only interpolate variables whose values are strings.  "15:37
*** alop has joined #tripleo15:40
*** daneyon_ has quit IRC15:41
*** daneyon has joined #tripleo15:41
*** dasm is now known as dasm|afk15:46
*** rwsu has joined #tripleo15:48
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Define service behavior in hiera file  https://review.openstack.org/18195115:50
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Add Cinder services as Pacemaker resources  https://review.openstack.org/18106715:51
*** mmagr is now known as mmagr|afk15:51
d0ugaljdob: Could you re-review https://review.openstack.org/#/c/179219/12? I fixed a typo you spotted and rebased.15:51
d0ugalI just started a recheck for the failure.15:51
gfidentespredzy, ^^ I think to not pollute the hieradata, you might want to add those straight into the controller-config-pacemaker thing15:52
jdobd0ugal: looking15:52
d0ugaljdob: Thanks!15:52
gfidentespredzy, triple(o) check with dprince, but I think that was his thinking ^^15:52
jdobd0ugal: done15:52
shardygfidente: I'm fine with it, although you're right I do prefer to minimise the use of yaml references where possible15:52
*** ifarkas has quit IRC15:52
gfidenteshardy, yeah I remember that, I don't know how to replace it though15:53
spredzygfidente, well the idea was to have it there. and drive the service_manage and service_enabled via heat15:53
openstackgerritOpenStack Proposal Bot proposed openstack/tuskar-ui: Updated from global requirements  https://review.openstack.org/18195415:53
spredzythat what I understood at least15:53
gfidentespredzy, yeah I got that but not sure if dprince is okay with that in the static hieradata15:53
spredzyIm gonna have to hope off for a moment please leave a comment on the review if there is an agreement Ill pick it up from there15:54
spredzy++15:54
*** pdb has quit IRC15:54
dprincegfidente: static hiera data (in a pacemaker specific YAML) is a great way to encapsulate pacemaker bits15:55
jistrmarios: re my review on the neutron + pacemaker change -- here's an example how the constraints can be done. I tested it and they actually got created. https://review.openstack.org/#/c/181067/2/puppet/manifests/overcloud_controller_pacemaker.pp15:55
gfidentedprince, sure it is, I referred to this: https://review.openstack.org/#/c/181951/15:56
shardygfidente: the alternative is to do the transformation in the parent template, e.g pass in the list pre str_replaced as a string15:56
gfidenteyou had in mind to pass ensure/managed from the controller-config-pacemaker.yaml or to provide for the params theirselves from there?15:56
shardyI'm not sure it makes sense to change the interface to do that in this case though15:56
openstackgerritMerged openstack/python-tuskarclient: Implement Update Plan for the OpenStack client  https://review.openstack.org/17921915:57
*** tzumainn_ is now known as tzumainn15:59
spredzygfidente, just to clarify something this is just a first review, another review is coming where we create a new file that will contain those value specifically for the pacemaker case15:59
spredzyso the lookup will go and pick it up from this file when the file exist16:00
gfidentespredzy, yeah I get that and it doesn't need to be an additional file, you probably can paste those from controller-config-pacemaker.yaml16:00
gfidenteso given the settings are pacemaker specific, I was thinking to move them there, instead of having them in the non-pacemaker hieradata (even though they flip based on pacemaker.yaml)16:01
dprincespredzy: YES! https://review.openstack.org/#/c/181951/116:02
dprincespredzy: but a minor suggestion on using %{hiera...} instead16:02
spredzydprince, ahah you right. like this it would look a facter16:02
dprincespredzy: because we have enable_factor, and enable_hiera support in the Heat puppet hook already. So you can do it either way... but we like Hiera16:03
spredzyI just wanted to push it, but it needs another review to acually work :)16:03
*** aufi has quit IRC16:03
gfidentedprince, I still don't get you16:03
gfidenteeven though, I try16:03
* spredzy afk &16:03
*** spredzy is now known as spredzy|afk16:03
gfidentedidn't you want to have the pacemaker related config in pacemaker related yamls?16:04
dprincegfidente: the goal is this is to support pacemaker in a stylistically clean way16:04
mariosjistr: thanks16:04
gfidentesure, so in this case16:04
gfidentedon't you want those params, which only apply to the pacemaker scenario16:05
gfidenteto go in the pacemaker yaml?16:05
dprincegfidente: I don't want them separated for any reason other than for now (today) maintaining separate files is cleaner than many/many $enable_pacemaker blocks16:05
*** cwolferh has joined #tripleo16:05
dprincegfidente: what spredzy|afk did is a nice generic feature16:05
dprincegfidente: which we can use to support pacemaker16:05
dprincegfidente: and it is exactly the direction which we should go w/ this16:05
gfidentebut that means16:06
gfidenteif I read it correctly16:06
*** eghobo has joined #tripleo16:06
openstackgerritOpenStack Proposal Bot proposed openstack/python-tuskarclient: Updated from global requirements  https://review.openstack.org/18124216:06
gfidentewe have in the hieradata files16:06
gfidentesupposedly static16:06
gfidentesome parameters16:06
gfidenteonly needed for pacemaker16:06
dprincegfidente: We do make some exception for conditionals in our Overcloud manifest. I think those are bad examples...16:06
mariosjistr: getting ready to call it a day but will update tomorrow16:06
gfidentewhich are flipped , only when deploying with pacemaker16:06
dprincegfidente: exactly16:06
jistrmarios: sure thing, i'll call it a day in a few mins too :)16:06
dprincegfidente: or, like I pointed out they could go into a pacemaker.yaml too16:06
openstackgerritOpenStack Proposal Bot proposed openstack/tuskar-ui: Updated from global requirements  https://review.openstack.org/18195416:07
gfidentedprince, yeah that is what I was trying to get to16:07
dprincegfidente: How we construct the Hiera doesn't matter as much to me16:07
gfidentedon't you want those into pacemaker.yaml?16:07
dprincegfidente: they could, but I like that spredzy is thinking generically here16:07
mariosjistr: wrt the system dependent service names, are you sure?16:07
mariosjistr: i can easily change but seems bad to me16:07
dprincegfidente: not thinking $enable_pacemaker is what we want...16:07
mariosjistr: i mean i had this running clean with nice pcs status at the end of it16:08
jistrmarios: you mean for the dependencies using arrows?16:08
gfidentedprince, so instead of passing mongodb::service::service_ensure from pacemaker.yaml16:08
gfidentedprince, you move that into a generic file16:08
mariosjistr: yeah that one and the one above for the dhcp agent notification16:08
mariosjistr: (where you commented)16:08
gfidentedprince, where you don't need it16:08
gfidenteUNLESS when using pacemaker16:09
gfidentein which case, you pass some service_ensure which will flip all those values16:09
jistrmarios: yeah it depends how the service is defined in the low-level manifest16:10
mariosjistr: let's talk some more tomorrow mate, sorry, got to run, am on kid duty in - 45 mins and need to get there first16:10
mariosjistr: thanks have a good one16:10
*** rameshg87 has quit IRC16:10
marioso/16:10
jistrmarios: ack, you too :)16:10
*** subscope has quit IRC16:12
*** trown is now known as trown|lunch16:16
*** dasm|afk is now known as dasm16:17
*** jistr has quit IRC16:17
*** jcoufal has quit IRC16:18
*** jcoufal has joined #tripleo16:18
*** jcoufal has quit IRC16:23
*** jcoufal has joined #tripleo16:23
*** mmagr|afk is now known as mmagr16:24
*** sdake_ has joined #tripleo16:25
*** gothicmindfood has quit IRC16:29
*** sdake has quit IRC16:29
*** gothicmindfood has joined #tripleo16:31
*** mcornea has quit IRC16:34
*** sdake has joined #tripleo16:36
*** alop has quit IRC16:37
*** alop has joined #tripleo16:39
*** sdake_ has quit IRC16:40
*** Marga_ has quit IRC16:42
*** derekh_ has joined #tripleo16:44
*** derekh has quit IRC16:47
*** derekh_ has quit IRC16:51
openstackgerritgreghaynes proposed openstack/diskimage-builder: Simple-init should disable cloud-init  https://review.openstack.org/18197716:51
*** pdb has joined #tripleo16:51
greghaynesmordred: ^16:51
mordredgreghaynes: I'm not sure how that disables cloud-init - although I think it's a good patch16:53
greghaynesmordred: sets the only data source to be None, which I assume means dont use any datasources16:53
greghaynesor, does cloud init still do things when None is a datasource?16:54
mordredgreghaynes: well ... so I don't think this will set it to None16:54
greghaynesoh. hah, forgot that file16:55
mordredI think you'd need to add an environment.d entry that woudl set DIB_CLOUD_INIT_DATASOURCES to None16:55
mordred:)16:55
openstackgerritgreghaynes proposed openstack/diskimage-builder: Simple-init should disable cloud-init  https://review.openstack.org/18197716:55
alopah16:55
greghaynesmaybe this is a sign I need coffee16:56
alophad not heard of simple-init16:56
*** yamahata has quit IRC16:58
*** yamahata has joined #tripleo16:58
*** daneyon_ has joined #tripleo17:05
*** Marga_ has joined #tripleo17:06
*** regebro has quit IRC17:07
*** daneyon has quit IRC17:08
*** lucasagomes is now known as lucas-dinner17:14
*** jcoufal has quit IRC17:15
*** Marga_ has quit IRC17:16
*** trown|lunch is now known as trown17:17
*** akrivoka has quit IRC17:23
*** Marga_ has joined #tripleo17:25
*** vipul has quit IRC17:31
*** mmagr has quit IRC17:32
*** ukalifon1 has quit IRC17:35
openstackgerritMerged openstack/tripleo-image-elements: Add ability to disable stunnel  https://review.openstack.org/13605217:40
-openstackstatus- NOTICE: We have discovered post-upgrade issues with Gerrit affecting nova (and potentially other projects). Some changes will not appear and some actions, such as queries, may return an error. We are continuing to investigate.17:40
*** ChanServ changes topic to "We have discovered post-upgrade issues with Gerrit affecting nova (and potentially other projects). Some changes will not appear and some actions, such as queries, may return an error. We are continuing to investigate."17:40
*** mestery has joined #tripleo17:41
*** mestery has quit IRC17:44
*** mestery has joined #tripleo17:46
*** Goneri has quit IRC17:47
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Compute: os-net-config for isolated networks  https://review.openstack.org/18199917:50
*** mestery has quit IRC17:56
*** pelix has quit IRC17:58
dprincehttps://review.openstack.org/#/c/178023/18:03
dprincethat looks good and passes CI18:03
dprincefixes an issue in os-net-config so it would be nice to land18:03
*** dprince has quit IRC18:03
*** eghobo has quit IRC18:07
openstackgerritDan Prince proposed openstack/tripleo-incubator: devtest_testenv.sh: support for multiple bridges  https://review.openstack.org/17788018:12
openstackgerritDan Prince proposed openstack/tripleo-incubator: Allow libvirt to assign PCI slots for us  https://review.openstack.org/18084118:12
openstackgerritDan Prince proposed openstack/tripleo-incubator: cleanup-env: Allow configurable bridge names.  https://review.openstack.org/17787718:12
openstackgerritDan Prince proposed openstack/tripleo-incubator: configure-vm: allow multiple baremetal-interface's  https://review.openstack.org/17787618:12
openstackgerritDan Prince proposed openstack/tripleo-incubator: create-nodes: allow multiple bridge names  https://review.openstack.org/17787918:12
openstackgerritDan Prince proposed openstack/tripleo-incubator: setup-network: allow creation of multiple bridges  https://review.openstack.org/17787818:12
openstackgerritDan Prince proposed openstack/tripleo-incubator: devtest_testenv.sh: support for multiple bridges  https://review.openstack.org/17788018:18
*** funzo_ is now known as funzo18:21
*** bnemec has quit IRC18:25
*** lsmola_ has quit IRC18:25
*** morganfainberg has quit IRC18:25
*** dasm_ has joined #tripleo18:25
*** larsks|alt has joined #tripleo18:26
*** radez_g0n3 has joined #tripleo18:26
*** fabbione has quit IRC18:26
*** larstobi has quit IRC18:26
*** larsks has quit IRC18:26
*** dmellado has quit IRC18:26
*** dasm has quit IRC18:26
*** slagle has quit IRC18:26
*** phschwartz has quit IRC18:26
*** zaneb has quit IRC18:26
*** bkero has quit IRC18:26
*** lan has quit IRC18:26
*** radez has quit IRC18:26
*** larstobi_ has joined #tripleo18:27
*** zaneb has joined #tripleo18:27
*** slagle has joined #tripleo18:27
*** thegodfather has joined #tripleo18:27
*** bnemec has joined #tripleo18:27
*** lsmola_ has joined #tripleo18:27
*** morganfainberg has joined #tripleo18:27
*** dmellado has joined #tripleo18:27
*** radez_g0n3 is now known as radez18:27
*** slagle has quit IRC18:27
*** slagle has joined #tripleo18:27
*** larstobi_ is now known as larstobi18:27
*** lan has joined #tripleo18:28
*** phschwartz has joined #tripleo18:28
*** larsks|alt is now known as larsks18:28
*** athomas has quit IRC18:28
*** bkero has joined #tripleo18:30
*** bkero has quit IRC18:30
*** bkero has joined #tripleo18:30
*** eghobo has joined #tripleo18:31
*** yamahata has quit IRC18:40
openstackgerritMerged openstack/os-net-config: Mark ENI files as autogenerated  https://review.openstack.org/16587818:57
jdobslagle: ping19:02
*** yamahata has joined #tripleo19:03
*** ukalifon1 has joined #tripleo19:05
openstackgerritDan Sneddon proposed openstack/os-net-config: Add flag for DEFROUTE=no in ifcfg files to ignore DHCP gateway  https://review.openstack.org/17802319:05
*** ukalifon1 has quit IRC19:06
openstackgerritDan Sneddon proposed openstack/os-net-config: Modify ordering of interface restarts so vlans are always restarted last.  https://review.openstack.org/18202319:11
*** Marga_ has quit IRC19:11
*** dguerri is now known as _dguerri19:12
*** panda|afk has quit IRC19:13
*** panda|afk has joined #tripleo19:13
openstackgerritJay Dobies proposed openstack/tuskar: Avoid MODULEPATH environment var in config generator  https://review.openstack.org/18202519:14
openstackgerritJay Dobies proposed openstack/tuskar: Change default auth type to keystone  https://review.openstack.org/18202719:19
openstackgerritJan Provaznik proposed openstack/tripleo-common: Add package update support  https://review.openstack.org/17986219:19
openstackgerritJan Provaznik proposed openstack/tripleo-common: Add a step-through heat stack update module  https://review.openstack.org/17815019:19
openstackgerritMerged openstack/tuskar-ui: Updated from global requirements  https://review.openstack.org/18195419:20
*** spzala has joined #tripleo19:26
*** jtomasek has quit IRC19:27
*** rlandy_ has joined #tripleo19:28
*** jprovazn has quit IRC19:29
*** subscope has joined #tripleo19:29
*** rlandy has quit IRC19:30
*** larstobi has quit IRC19:31
slaglejdob: hi19:32
jdobslagle: hey, should I be doing a release for tripleo-puppet-elements?19:33
slaglejdob: has it ever been released before?19:35
jdobdoesnt look like it19:35
slaglejdob: i think you could skip it until it's asked for19:36
jdobkk19:36
slaglei know of no one wanting to consume a release of it19:36
openstackgerritMerged openstack/diskimage-builder: Address comments on virtual media device label commit  https://review.openstack.org/17867219:36
*** _dguerri is now known as dguerri19:38
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Define service behavior in hiera file  https://review.openstack.org/18195119:41
*** Marga_ has joined #tripleo19:42
*** mcornea has joined #tripleo19:46
*** Marga_ has quit IRC19:46
*** eghobo has quit IRC19:55
*** spredzy|afk is now known as spredzy20:02
*** eghobo has joined #tripleo20:04
*** jayg is now known as jayg|g0n320:07
*** eghobo has quit IRC20:08
*** sdake_ has joined #tripleo20:10
*** mcornea has quit IRC20:12
*** daneyon_ has quit IRC20:12
*** daneyon has joined #tripleo20:13
*** sdake has quit IRC20:13
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Define service behavior in hiera file  https://review.openstack.org/18195120:24
*** Marga_ has joined #tripleo20:29
*** larstobi has joined #tripleo20:29
openstackgerritDan Sneddon proposed openstack/os-net-config: Modify ordering of interface restarts so vlans are always restarted last.  https://review.openstack.org/18202320:36
*** radez is now known as radez_g0n320:36
*** larstobi has quit IRC20:38
*** larstobi has joined #tripleo20:38
*** openstackgerrit_ has joined #tripleo20:43
*** eghobo has joined #tripleo20:50
*** Marga_ has quit IRC20:58
*** Marga_ has joined #tripleo20:59
shardydsneddon: Hey, looks like dprince isn't around, but FYI I added a comment to https://review.openstack.org/#/c/178715/21:00
*** untriaged-bot has joined #tripleo21:00
untriaged-botUntriaged bugs so far:21:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/145240021:00
openstackLaunchpad bug 1452400 in diskimage-builder "debian-minimal builds still busted, now on locale issues" [Undecided,New]21:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985221:00
openstackLaunchpad bug 1449852 in diskimage-builder "Buidling ramdisk with ironic-agent behind proxy fails" [Undecided,In progress] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)21:00
untriaged-bothttps://bugs.launchpad.net/diskimage-builder/+bug/144985421:00
openstackLaunchpad bug 1449854 in diskimage-builder "Ironic agent ramdisk built using disk-image-create fails with iscsi_ilo driver" [Undecided,In progress] - Assigned to Ramakrishnan G (rameshg87) (rameshg87)21:00
untriaged-bothttps://bugs.launchpad.net/tuskar/+bug/145275221:00
openstackLaunchpad bug 1452752 in tuskar "keystone_authtoken section is wrong in default shipped tuskar.conf.sample" [Undecided,Confirmed]21:00
*** untriaged-bot has quit IRC21:00
shardyI think potentially using a json parameter map specified via the environment may be cleaner, but I've not tested it21:00
shardyhappy to hear your thoughts on that, I'll try to catch up with dprince to discuss it tomorrow21:01
dsneddonshardy, It seems to make sense to me, but I want to hear dprince chime in21:01
dsneddonshardy, I'm not crazy about moving it out of the templates and into an environment file, but those lookups get a lot cleaner.21:01
*** rhallisey has quit IRC21:02
shardydsneddon: I think the key thing for me is heat templates aren't a container for data, that's what the environment is for21:02
shardyso, to me, if we can avoid it, we should try not to use templates like that21:02
dsneddonshardy, Yes, if you think about it like data, that's true. I was thinking of it as template defaults that could be overridden, but I see your point.21:03
shardydsneddon: Yeah, again, template defaults which can be overridden is exactly what parameter_defaults is for21:03
shardyhopefully the combination of that and the path-based get_param will provide a solution we're all happy with21:03
shardybut happy to chat again about it tomorrow if needed :)21:04
*** trown is now known as trown|outttypeww21:04
dsneddonI'm with you, unless dprince has a strong objection21:04
*** sdake has joined #tripleo21:04
shardydsneddon: Ok, cool, thanks21:05
* shardy heads for some sleep21:05
*** shardy is now known as shardy_z21:06
*** sdake_ has quit IRC21:08
*** Marga_ has quit IRC21:10
*** Marga_ has joined #tripleo21:10
*** Marga_ has quit IRC21:17
*** rlandy_ has quit IRC21:17
*** Marga_ has joined #tripleo21:18
*** eghobo has quit IRC21:31
*** julim has quit IRC21:31
*** Slower has quit IRC21:32
*** Slower has joined #tripleo21:39
*** eghobo has joined #tripleo21:40
*** gfidente has quit IRC21:40
*** morazi has quit IRC21:44
*** funzo has quit IRC21:49
*** sdake has quit IRC21:50
*** sdake has joined #tripleo21:52
*** Marga_ has quit IRC22:04
*** Marga_ has joined #tripleo22:04
*** subscope has quit IRC22:09
*** subscope has joined #tripleo22:18
openstackgerritDan Sneddon proposed openstack/os-net-config: Add flag for DEFROUTE=no in ifcfg files to ignore DHCP gateway  https://review.openstack.org/17802322:23
*** rwsu has quit IRC22:25
*** lucas-dinner has quit IRC22:28
openstackgerritDan Sneddon proposed openstack/os-net-config: Modify ordering of interface restarts so vlans are always restarted last.  https://review.openstack.org/18202322:33
*** BadCub is now known as BadCub-Hiding22:56
*** funzo has joined #tripleo23:04
*** yuanying has joined #tripleo23:04
*** subscope has quit IRC23:10
stevebakerslagle: hey, still about?23:13
*** panda|afk has quit IRC23:13
*** panda|afk has joined #tripleo23:14
*** alop has quit IRC23:25
slaglestevebaker: i am now, for a few23:51
stevebakerslagle: I wanted to tag this as a TripleO session too, but the scheduling site won't let me http://libertydesignsummit.sched.org/event/02f197c0beadad5eb354ab7518dd9363#.VVE4sHU4Z4s23:51
slaglehmm, i'm not sure how to do that23:53
-openstackstatus- NOTICE: Gerrit is going offline while we perform an emergency downgrade to version 2.8.23:56
*** ChanServ changes topic to "Gerrit is going offline while we perform an emergency downgrade to version 2.8."23:56
*** sdake_ has joined #tripleo23:57

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