Monday, 2015-04-27

*** Goneri has quit IRC00:01
*** bhunter has quit IRC00:15
*** bhunter has joined #tripleo00:16
*** pcrews has joined #tripleo00:25
*** zhiwei has joined #tripleo00:39
*** zhiwei has left #tripleo00:40
*** eghobo has joined #tripleo01:05
*** eghobo has quit IRC01:08
*** victor_lowther has quit IRC01:20
*** victor_lowther has joined #tripleo01:22
*** sdake has joined #tripleo01:32
*** sdake_ has joined #tripleo01:48
*** sdake has quit IRC01:51
*** eghobo has joined #tripleo02:03
*** sdake_ has quit IRC02:35
*** sdake has joined #tripleo02:52
*** wuhg has joined #tripleo02:52
*** untriaged-bot has joined #tripleo03:00
untriaged-botUntriaged bugs so far:03:00
untriaged-bothttps://bugs.launchpad.net/python-tuskarclient/+bug/144794303:00
openstackLaunchpad bug 1447943 in python-tuskarclient "Missing /v2 path of TUSKAR_URL example in README" [Undecided,In progress] - Assigned to Marek Aufart (maufart)03:00
*** untriaged-bot has quit IRC03:00
*** zaneb has joined #tripleo03:08
*** tzumainn has quit IRC03:14
*** zaneb has quit IRC03:17
*** eghobo has quit IRC03:18
*** sdake_ has joined #tripleo03:28
*** sdake has quit IRC03:32
*** eghobo has joined #tripleo03:58
*** subscope_ has joined #tripleo05:30
*** subscope_ has quit IRC05:31
*** dasm|afk is now known as dasm05:36
*** eghobo_ has joined #tripleo05:51
*** masco has joined #tripleo05:52
*** eghobo has quit IRC05:55
*** sdake_ has quit IRC05:56
*** sdake has joined #tripleo05:56
*** dtantsur|afk is now known as dtantsur06:07
*** eghobo has joined #tripleo06:09
*** eghobo_ has quit IRC06:12
*** BadCub_Hiding has quit IRC06:24
*** BadCub_Hiding has joined #tripleo06:25
*** jprovazn has joined #tripleo06:26
*** regebro has joined #tripleo06:35
*** ukalifon1 has joined #tripleo06:49
*** rdopiera has joined #tripleo06:56
*** bvandenh has joined #tripleo06:56
*** sdake has quit IRC06:58
*** jcoufal has joined #tripleo07:06
*** yog__ has joined #tripleo07:06
*** jcoufal is now known as 7JTAAIHSL07:06
*** jcoufal has joined #tripleo07:06
*** jcoufal has quit IRC07:06
*** jcoufal has joined #tripleo07:06
*** mcornea has joined #tripleo07:08
*** shardy_ has joined #tripleo07:09
*** shardy has quit IRC07:11
*** yog__ has quit IRC07:13
*** shardy_ has quit IRC07:14
*** shardy has joined #tripleo07:15
*** Nakato has quit IRC07:16
*** Nakato has joined #tripleo07:16
*** yog__ has joined #tripleo07:17
*** eghobo has quit IRC07:17
*** hewbrocca has joined #tripleo07:37
*** sdake has joined #tripleo07:42
*** sdake has quit IRC07:42
*** sdake has joined #tripleo07:42
*** ifarkas has joined #tripleo07:45
*** aufi has joined #tripleo07:50
*** jistr has joined #tripleo07:57
*** openstackstatus has joined #tripleo08:07
*** ChanServ sets mode: +v openstackstatus08:07
-openstackstatus- NOTICE: Restarting gerrit because it stopped sending events (ETA 15 mins)08:11
*** athomas has joined #tripleo08:13
*** openstackgerrit has quit IRC08:13
*** hewbrocca has quit IRC08:14
*** hewbrocca has joined #tripleo08:15
*** dtantsur is now known as dtantsur|brb08:16
*** openstackgerrit has joined #tripleo08:16
*** lucasagomes has joined #tripleo08:17
*** russell_h has quit IRC08:17
*** derekh has joined #tripleo08:19
*** russell_h has joined #tripleo08:20
*** mmagr has joined #tripleo08:22
*** jtomasek has joined #tripleo08:27
*** yog__ has quit IRC08:32
*** shardy_ has joined #tripleo08:33
*** pdb has joined #tripleo08:34
*** shardy has quit IRC08:34
*** shardy_ has quit IRC08:38
*** Goneri has joined #tripleo08:39
*** shardy has joined #tripleo08:39
*** aufi_ has joined #tripleo08:44
*** maufart_ has joined #tripleo08:45
*** yog__ has joined #tripleo08:46
*** al has quit IRC08:47
*** aufi has quit IRC08:47
*** al has joined #tripleo08:47
*** aufi_ has quit IRC08:49
*** gfidente has joined #tripleo08:50
*** gfidente has joined #tripleo08:50
*** sdake_ has joined #tripleo08:51
*** Goneri has quit IRC08:51
*** Goneri has joined #tripleo08:52
*** aufi has joined #tripleo08:55
*** sdake has quit IRC08:55
*** dasm is now known as dasm|afk08:55
*** maufart_ has quit IRC08:59
*** untriaged-bot has joined #tripleo09:00
untriaged-botUntriaged bugs so far:09:00
untriaged-bothttps://bugs.launchpad.net/python-tuskarclient/+bug/144794309:00
*** untriaged-bot has quit IRC09:00
openstackLaunchpad bug 1447943 in python-tuskarclient "Missing /v2 path of TUSKAR_URL example in README" [Undecided,In progress] - Assigned to Marek Aufart (maufart)09:00
*** Goneri has quit IRC09:01
*** Goneri has joined #tripleo09:03
*** sdake_ has quit IRC09:13
*** sdake has joined #tripleo09:15
*** pelix has joined #tripleo09:16
*** dtantsur|brb is now known as dtantsur09:19
gfidentemorning jistr , saw your reviews09:26
gfidentejistr, would you re-vote this https://review.openstack.org/#/c/173773/7/puppet/manifests/overcloud_controller.pp,cm if I fix the indent? spredzy_ << :)09:26
spredzy_gfidente, ahah sure, I already voted +1 code looks good to me, so if you resubmt i will revote ;)09:27
jistrmorning :)09:28
jistrgfidente: sure i would09:28
*** dmellado has quit IRC09:28
gfidentedeal ;)09:28
jistri don't care about the alignments much -- aligned looks better, but when one needs to add another parameter which is longer than the others, then all lines need to be changed, which breaks git history (the commit now changes unrelated lines). Personally i prefer non-aligned arrows a bit more to keep git clean, but i know the usual convention is the other way round :)09:31
*** aufi_ has joined #tripleo09:32
*** aufi has quit IRC09:33
*** aufi_ is now known as aufi09:33
*** yog__ has quit IRC09:38
*** yog__ has joined #tripleo09:51
*** yog__ has quit IRC09:54
gfidentejistr, I wanted to ask09:59
gfidentegive we have pcsd09:59
gfidenteshall we add the resources to pacemaker in a single node?09:59
gfidente*given09:59
*** yog__ has joined #tripleo10:08
openstackgerritGonéri Le Bouder proposed openstack/diskimage-builder: force arch amd64 for EL7 elements  https://review.openstack.org/17770510:08
*** lucasagomes has quit IRC10:09
*** lucasagomes has joined #tripleo10:11
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Add RabbitMQ as a Pacemaker resource when EnablePacemaker  https://review.openstack.org/17592610:23
spredzy_gfidente, for Galera this is what I am doing. If enable_pacemaker, then even in single node mysql is managed by pacemaker (cc jistr)10:25
gfidentespredzy_, yeah that seems valid to me10:25
gfidenteI was wondering if we shouldn't make puppet configure only a single pacemaker instance and let pcsd propagate the changes10:25
*** tserong has quit IRC10:27
*** tserong has joined #tripleo10:28
*** tserong has joined #tripleo10:28
jistrgfidente: re pcsd -- yeah i think we should add the resources on a single node10:39
jistrgfidente: otherwise race conditions can happen10:39
jistrpcsd is going to take care of replicating the setup on all nodes in the cluster10:39
gfidenteyeah so I was thinking to use a if enable_pacemake and hostname == bootstrap_nodeid10:40
gfidentespredzy_, jistr ^^ makes sense?10:40
jistrgfidente: yup10:40
gfidentewill update my submissions then10:41
gfidentethanks10:41
*** sdake has quit IRC10:41
*** mmagr is now known as mmagr|afk10:44
*** sdake has joined #tripleo10:52
*** yuanying has quit IRC10:55
jistrhuh the change which should fix our Pacemaker setup actually breaks the non-Puppet setup https://review.openstack.org/#/c/177203/10:56
jistrthis will be fun :D10:57
jistrhmm ok so i guess it'll be patch to DIB to introduce cloud-init-hosts element -> patch to incubator to make the non-Puppet jobs use that element -> patch to DIB to remove that from core element10:58
* jistr on it10:58
*** hewbrocca has quit IRC11:01
*** rhallisey has joined #tripleo11:02
*** maufart has joined #tripleo11:07
*** sdake has quit IRC11:10
*** dprince has joined #tripleo11:11
openstackgerritRadomir Dopieralski proposed openstack/tuskar-ui: Add radix parameter to parseInt  https://review.openstack.org/17771311:12
*** dasm|afk is now known as dasm11:22
openstackgerritJiri Stransky proposed openstack/diskimage-builder: Extract cloud-init-hosts element from core element  https://review.openstack.org/17771711:24
*** hewbrocca has joined #tripleo11:30
*** tzumainn has joined #tripleo11:32
openstackgerritJiri Stransky proposed openstack-infra/tripleo-ci: Add cloud-config-hosts element for non-puppet jobs  https://review.openstack.org/17772211:35
*** pblaho has joined #tripleo11:37
*** mmagr|afk is now known as mmagr11:39
*** athomas has quit IRC11:40
*** mmagr is now known as mmagr|afk11:45
openstackgerritJiri Stransky proposed openstack/diskimage-builder: Do not manage host entries by default  https://review.openstack.org/17720311:46
*** athomas has joined #tripleo11:47
*** chmouel has quit IRC11:48
openstackgerritJiri Stransky proposed openstack/diskimage-builder: Do not manage host entries by default  https://review.openstack.org/17720311:48
*** thrash|wknd is now known as thrash11:49
*** chmouel has joined #tripleo11:50
*** lucasagomes is now known as lucas-hungry11:56
openstackgerritJiri Stransky proposed openstack/tripleo-incubator: Add cloud-config-hosts element  https://review.openstack.org/17772811:56
*** chmouel has quit IRC11:57
*** chmouel_ has joined #tripleo11:57
*** athomas has quit IRC11:58
*** jdob has joined #tripleo11:59
openstackgerritRadomir Dopieralski proposed openstack/tuskar-ui: Override submit_url on images.UpdateView  https://review.openstack.org/17773011:59
*** rhallisey has quit IRC12:00
*** lucas-hungry has quit IRC12:02
*** mmagr|afk is now known as mmagr12:05
*** openstackgerrit has quit IRC12:06
*** openstackgerrit has joined #tripleo12:06
*** athomas has joined #tripleo12:09
openstackgerritGonéri Le Bouder proposed openstack/diskimage-builder: force arch amd64 for EL7 elements  https://review.openstack.org/17773412:09
*** tserong has quit IRC12:09
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Set up corosync using hostnames rather than IPs  https://review.openstack.org/17773512:09
openstackgerritGonéri Le Bouder proposed openstack/diskimage-builder: force arch amd64 for EL7 elements  https://review.openstack.org/17770512:12
*** weshay has joined #tripleo12:13
*** lucas-hungry has joined #tripleo12:14
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: puppet: install Horizon on overcloud-controller  https://review.openstack.org/16439212:16
*** jehb has joined #tripleo12:16
*** zaneb has joined #tripleo12:17
*** jayg|g0n` is now known as jayg12:19
openstackgerritGonéri Le Bouder proposed openstack/diskimage-builder: force arch amd64 for EL7 elements  https://review.openstack.org/17770512:19
*** dprince has quit IRC12:19
openstackgerritJiri Stransky proposed openstack-infra/tripleo-ci: Add cloud-init-hosts element for non-puppet jobs  https://review.openstack.org/17772212:26
openstackgerritJiri Stransky proposed openstack/tripleo-incubator: Add cloud-init-hosts element  https://review.openstack.org/17772812:27
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Switch VIP management from Keepalived to Pacemaker  https://review.openstack.org/17377312:29
*** masco has quit IRC12:31
gfidentespredzy_, jistr ^^ that adds the if master12:32
*** chmouel_ is now known as chmouel12:33
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Configures haproxy as a Pacemaker resource  https://review.openstack.org/17446512:34
*** trown|outttypeww has quit IRC12:36
*** dprince has joined #tripleo12:36
*** openstackgerrit has quit IRC12:37
*** openstackgerrit has joined #tripleo12:37
spredzy_gfidente, ack12:37
jistrgfidente: +2'd both12:39
gfidentejistr, spredzy_, so I can use pacemaker_master outside the step 1 scope right?12:41
gfidentewould you try to kill me then?12:42
gfidente(that would be for the other services to be added)12:42
spredzy_gfidente, using $pacemaker_master outside step 1 would be fine. Technically it will work and I am ok with the use of it outside step112:43
jistrgfidente, spredzy_: i'd like it a bit more if we moved the $enable_pacemaker and $pacemaker_master definitions above step 1, but not a blocker12:45
gfidenteyou can do that later! :)12:45
gfidenteauhauhauh12:45
gfidenteno sorry, wasn't so fun, I know12:45
spredzy_jistr, you mean at the same level of enable_package_install ?12:45
jistrspredzy_: yeah12:46
*** lucas-hungry is now known as lucasagomes12:54
dprinceshardy: so I'm creating Neutron::Ports and I'd like to add subnet information to them12:57
dprinceshardy: there are properties for subnet_id, subnet, etc. off of the fixed_ip property but only the subnet_id is set (I really want the subnet instead).12:57
dprinceshardy: any ideas on how I might get a handle to a subnet, or should I perhaps just pass it into the template since I need it...12:59
*** rhallisey has joined #tripleo13:01
*** yog__ has quit IRC13:01
shardydprince: you should be able to pass either "subnet" or "subnet_id" in via the fixed_ips list13:02
*** trown has joined #tripleo13:02
shardydprince: are you creating the subnet in the template, or passing it in?13:02
spredzy_Does this issue ring a bell to anybody ?  ResourceUnknownStatus: Resource failed - Unknown status FAILED due to "Resource CREATE failed: Error: Server ov-2hmwmepmvg6-0-pmktfmzro4hm-Controller-kqza6myv5yfz delete failed: (500) No valid host was found. Exceeded max scheduling attempts 3 for instance 8a07d8fd-f48b-4339-a510-9f2a201a4c92.13:04
spredzy_during a heat stack-create13:04
dprinceshardy: got it http://paste.openstack.org/show/208282/13:05
dprinceshardy: the subnet parameter then contains something like 172.0.19.0/2413:05
shardydprince: you need to pass in both the network and subnet (either by name or ID) when creating the port13:07
*** yog__ has joined #tripleo13:07
shardyhttps://github.com/openstack/heat-templates/blob/master/hot/servers_in_existing_neutron_network_no_floating_ips.yaml#L3813:07
dprinceshardy: I'm using the network name13:08
dprinceshardy: It is an optional parameter and I think is what we want13:08
*** subscope has quit IRC13:09
dprinceshardy: okay, so is there any way to make 172.0.19.0/24 just /24?13:10
dprinceshardy: What I actually want as an output parameter of this template is <ipaddress>/<cidr>13:12
dprinceshardy: so something like 172.0.19.18/2413:13
dprinceshardy: what I've got is 172.0.19.18 (the ip_address) and 172.0.19.0/24 (the full subnet)13:13
dprinceshardy: I'm not opposed to rolling a custom function if we have to BTW13:13
dprinceshardy: I've got 8 or so unpushed patches at this point. Maybe I should just push everything and you can see where I'm att13:16
* hewbrocca salt-n-pepa13:17
dprinceshardy: I'd like to show dsneddon what I've got from the weekend too...13:17
dprincehewbrocca: ha, took me 15 seconds or so to get that reference13:18
hewbroccaLOL13:18
shardydprince: Ok, let me think about the cleanest way to do this, I'm not sure atm13:18
*** maufart has quit IRC13:19
dprinceshardy: well, a custom function would do it. slice, or split or something13:19
dprinceshardy: another option is I could use a wrapper resource (a custom resource) if we find that approach more palitable13:19
dprinceshardy: now that I've been show the art of custom things it is only a matter of time before I going to find some use for something custom anyways ;)13:20
dprinceshardy: but I do think there may be some value in sticking w/ the stock Heat resources/functions too13:21
*** yamahata has quit IRC13:21
dprinceshardy: but still, customizations is a feature and we have (probably) one of the most complex set of templates upstream so we might need to use them occasionally13:21
*** openstackgerrit has quit IRC13:21
shardydprince: there is already Fn::Split which we inherited from CFN, but we removed it from recent HOT versions, so there must be a native way to do it now13:22
*** aufi1 has joined #tripleo13:22
*** openstackgerrit has joined #tripleo13:22
shardydprince: Yeah, normally if TripleO needs something, it's something other folks will want sooner or later13:22
*** subscope has joined #tripleo13:24
*** 7JTAAIHSL has quit IRC13:26
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Add Galera support with Pacemaker  https://review.openstack.org/17776513:31
*** athomas has quit IRC13:31
*** mestery_ is now known as mestery13:32
spredzy_jistr, do you agree with the 4 uses cases listed in this commit message ^^ ?13:32
spredzy_Real question is. should we manage the use case HA setup / Non Pacemaker setup ?13:33
jistrspredzy_: i agree with the commit message. I wouldn't aim for HA-non-pacemaker scenario. Getting Galera running without pacemaker seemed quite difficult and not very clean (i'd like to avoid tweaking the systemd unit files if possible).13:36
dprinceshardy: http://paste.openstack.org/show/208332/13:39
spredzy_jistr, ack. Thank you.13:39
jistrspredzy_: btw i skimmed through the patch to get a high level view, looks good!13:39
dprinceshardy: so that works... assuming we only have 2 digit cidrs. I can't imaging anyone is going to drop a /8 on these templates anyways!13:40
spredzy_jistr, hehe thanks ;)13:40
dprinceshardy: what I get is "this_is_what_I_want?": "172.0.19.12/24"13:40
dprinceshardy: I'm gonna call this good enough and move on :)13:40
shardydprince: hehe, ugly but gets the job done ;)13:41
*** lucasagomes has quit IRC13:41
shardydprince: I'll follow up on the removal of Fn::Split - it's odd that we removed it with (AFAICT) no native replacement13:42
*** lucasagomes has joined #tripleo13:43
*** morganfainberg has quit IRC13:44
*** jroll has quit IRC13:44
*** jroll has joined #tripleo13:44
*** morganfainberg has joined #tripleo13:47
-openstackstatus- NOTICE: gerrit has been restarted to clear a problem with its event stream. change events between 13:09 and 13:36 utc should be rechecked or have approval votes reapplied as needed to trigger jobs13:48
*** athomas has joined #tripleo13:52
EmilienMshardy: do you want me to rebase https://review.openstack.org/#/c/174150 ?13:53
EmilienMi'm depending on this patch to test serverspec13:53
shardyEmilienM: sure, please do13:55
*** subscope has quit IRC13:57
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Add support for generic controller validations  https://review.openstack.org/17415013:57
*** jtomasek has quit IRC13:57
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: puppet/controller: validate RabbitMQ with serverspec  https://review.openstack.org/17642113:58
EmilienMshardy: done, thx. I'll play with that today13:58
*** funzo has joined #tripleo13:59
*** lblanchard has joined #tripleo14:02
openstackgerritGonéri Le Bouder proposed openstack/diskimage-builder: force arch amd64 for EL7 elements  https://review.openstack.org/17770514:10
*** subscope has joined #tripleo14:13
*** dmellado has joined #tripleo14:14
*** BadCub_Hiding is now known as BadCub14:19
*** mmagr is now known as mmagr|afk14:20
openstackgerritMerged openstack/tripleo-heat-templates: Separate the network configuration per flavor.  https://review.openstack.org/17362414:20
openstackgerritMerged openstack/tripleo-heat-templates: Make all default values match overcloud defaults  https://review.openstack.org/17141114:21
*** jprovazn has quit IRC14:22
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Add RabbitMQ as a Pacemaker resource when EnablePacemaker  https://review.openstack.org/17592614:24
*** ChuckC_ has joined #tripleo14:25
*** ChuckC has quit IRC14:26
*** michchap has quit IRC14:28
*** ChuckC_ has quit IRC14:30
*** weshay has quit IRC14:35
*** weshay has joined #tripleo14:39
*** russell_h has quit IRC14:39
*** russell_h has joined #tripleo14:39
*** dmellado has quit IRC14:42
openstackgerritMerged openstack/os-collect-config: Updated from global requirements  https://review.openstack.org/17684714:42
*** spzala has joined #tripleo14:48
*** mmagr|afk is now known as mmagr14:49
*** aufi1 has quit IRC14:50
openstackgerritAntoine Musso proposed openstack/diskimage-builder: Turn docs warnings into error and fix issues  https://review.openstack.org/17780114:52
*** untriaged-bot has joined #tripleo15:00
untriaged-botUntriaged bugs so far:15:00
untriaged-bothttps://bugs.launchpad.net/python-tuskarclient/+bug/144794315:00
*** untriaged-bot has quit IRC15:00
openstackLaunchpad bug 1447943 in python-tuskarclient "Missing /v2 path of TUSKAR_URL example in README" [Undecided,In progress] - Assigned to Marek Aufart (maufart)15:00
*** ChuckC_ has joined #tripleo15:01
greghaynesCan I get a review on https://review.openstack.org/#/c/166650/19 ?15:02
greghaynesid really like to cut a dib release with that in it15:02
openstackgerritAntoine Musso proposed openstack/diskimage-builder: Turn docs warnings into errors and fix issues  https://review.openstack.org/17780115:03
*** zaneb has quit IRC15:04
*** rdopiera has quit IRC15:08
*** kbyrne has quit IRC15:12
*** kbyrne has joined #tripleo15:16
openstackgerritGiulio Fidente proposed openstack-infra/tripleo-ci: Bump up delorean to allow for openstack-dashboard installation  https://review.openstack.org/17717615:18
*** bvandenh has quit IRC15:20
*** dtantsur is now known as dtantsur|afk15:24
openstackgerritgreghaynes proposed openstack/diskimage-builder: Initial element tests  https://review.openstack.org/17468115:25
*** Marga_ has joined #tripleo15:29
openstackgerritgreghaynes proposed openstack/diskimage-builder: Support building ACIs  https://review.openstack.org/15548415:29
*** spredzy_ is now known as spredzy_|afk15:29
*** Marga__ has joined #tripleo15:29
*** penick has joined #tripleo15:31
gfidentejistr, spredzy_|afk so I'm adding the if $pacemaker_master in the top part here: https://review.openstack.org/#/c/175926/7/puppet/manifests/overcloud_controller.pp,cm15:32
*** Marga_ has quit IRC15:33
gfidentejistr, is that what you had in mind?15:34
jistrgfidente: yeah exactly, that feels a bit cleaner to me than defining it in "if step1" and using it in "if step2". thanks :)15:35
*** Marga__ has quit IRC15:37
*** aufi has quit IRC15:38
*** jroll has quit IRC15:44
openstackgerritAntoine Musso proposed openstack/diskimage-builder: Test elements have README.rst file  https://review.openstack.org/17783215:45
*** jroll has joined #tripleo15:46
*** sdake has joined #tripleo15:47
*** jroll has quit IRC15:53
*** jroll has joined #tripleo15:53
greghaynesjistr: Hey, Id like to chat about https://review.openstack.org/#/c/177203/15:53
jistrgreghaynes: hi, go ahead please :)15:54
greghaynesjistr: Im concerned that its a backwards incompat change - were making our users opt in to a new element to get the current default behavior15:54
*** jtomasek has joined #tripleo15:54
*** yog__ has quit IRC15:54
jistrgreghaynes: yeah that's true...15:54
greghaynesan alternative idea - if we make that config value configurable from an env var, and then make the default be "localhost", youall will be able to set it to something else and default behavior will be preserved15:55
greghaynesmanage_etc_hosts: ${DIB_CLOUD_INIT_ETC_HOSTS:-localhost} essentially15:55
regebroI get the error "No Presto metadata available for rhelosp-rhel-7-z".15:56
regebroAt least I think that's an error. Hard to say.15:56
*** rhallisey has quit IRC15:56
regebroI have no idea what that means.15:56
*** alop has joined #tripleo15:57
jistrgreghaynes: ok that sounds good to me, i'll amend the patch. Feel free to paste that comment on the review so it's visible it needs an amendment. Thanks for the feedback :)16:00
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Create split out neutron networks via Heat.  https://review.openstack.org/17784316:00
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Wire in optional network creation for overcloud  https://review.openstack.org/17784416:00
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add a ports (ip address) abstraction layer  https://review.openstack.org/17784516:00
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Wire in split out ports to all roles  https://review.openstack.org/17784616:00
dprincedsneddon: ^^^16:00
dprinceshardy: ^^^16:00
jistrgreghaynes: i'm EOD so i'll amend it tomorrow morning16:00
greghaynesjistr: ok, I left a bunch of comments on the child patch before noticing that about the parent, basically those two could probably become one patch which does that :)16:00
*** penick has quit IRC16:01
greghaynesjistr: np, just wanted to sync up since a giant wall of comments I figured would be a bit hard to get what im saying16:01
jistryeah if we keep the default behavior, we won't need to do some other changes around16:01
*** david-ly_ is now known as david-lyle16:01
gfidentegreghaynes, I can see why it changes the default behaviour16:03
gfidentejistr, greghaynes, what is the reason why we need to pass that option to cloud-init though?16:04
greghaynesWe already pass it in the base element16:05
jistrgfidente: some things already depend on having the FQDN mapped to loopback interface (for some reason anyway :) )16:05
greghaynesYes, so the issue is you cant opt in or out of that because its part of base16:06
jistryeah16:06
greghaynesTheres been a recent thinking that we should break all the things out of base for similar reasons, so im totally fine with that btw16:07
gfidentejistr, you know of something which depends on such a configuration or guessing?16:07
*** dmellado has joined #tripleo16:08
jistrgfidente: e.g. check-tripleo-ironic-undercloud-precise-nonha :)16:08
*** ChuckC_ has quit IRC16:09
gfidenteso it is specific to the job? or to the distro?16:09
gfidentecan we add the element only for that job or that distro then?16:09
jistrgfidente: re cause - i don't know, there wasn't much to lean on in the logs, and i won't get that setup running locally16:10
gfidentegreghaynes, ?16:10
greghaynesSure, thats what the patches do. The problem is if its opt-in (which is what splitting it out to a new element is) then its backwards incompat.16:11
*** SpamapS has quit IRC16:12
jistryeah i agree on the backwards compatibility point. I'm ok with just making it settable instead of hardcoded for now.16:12
*** SpamapS has joined #tripleo16:12
*** trown is now known as trown|lunch16:12
greghaynesyep, its easy enough and we get a new feature out of it16:12
jistri think i could still keep it extracted to its own element and just make the core element depend on it16:13
jistrso if/when we want to drop it from the behavior of the core element, we can just remove the dep16:13
*** cwolferh has joined #tripleo16:14
gfidenteso instead I do not think we should change by default the actual default of cloud-init16:14
gfidentewhich means that we should change the default dib behaviour to not mess with the default cloud-init config16:14
greghaynesYes. I left a comment about that - I think thats fine (like I said, splitting things out of base is something we want to do) but making an element per cloud-init config option seems like a lot. Instead we could make a single element that configures various cloud-init config options16:14
gfidenteand if that causes issues, workaround those only where appropriate16:14
*** jcoufal has joined #tripleo16:15
greghaynesIf we want to do that then youd need to deprecate and wait then change16:15
*** regebro is now known as regebro|afk16:17
*** regebro|afk has quit IRC16:17
*** ukalifon1 has quit IRC16:17
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Wire in split out ports to all roles  https://review.openstack.org/17784616:19
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Add a ports (ip address) abstraction layer  https://review.openstack.org/17784516:19
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Wire in optional network creation for overcloud  https://review.openstack.org/17784416:19
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Create split out neutron networks via Heat.  https://review.openstack.org/17784316:19
*** jistr has quit IRC16:21
gfidentegreghaynes, to be honest, I was trying to figure what is that the default cloud-init behaviour breaks16:23
SpamapSgfidente: Would you humor me as to what issue with cloud-init we're dicussing? I am without backscroll at the moment.16:26
greghaynesSpamapS: https://review.openstack.org/#/c/177203/16:27
greghayneshttps://review.openstack.org/#/c/177043/ might need some others to weight their opinion in on16:28
*** mmagr is now known as mmagr|afk16:29
SpamapSSo, AFAIK, that is the only sane answer for openstack's weirdness regarding DNS16:30
gfidenteSpamapS, ah yeah you mean nova adding .novalocal despite the neutron dns suffic?16:31
gfidente*suffix16:31
SpamapSYep16:31
SpamapSSo while I _do_ think it should be its own element.. I also think it's right probably 90% of the time to include it.16:31
gfidenteehehe only thing I found is that it adds that from some dhcp_domain default which can be customized in nova.conf16:32
SpamapSgfidente: right, if cloud providers have gone through the trouble to setup reverse-DNS for the IPs they give you, _and_ give them to you with the right hostname..........16:32
gfidenteand indeed does not look correct when the real dhcp is providing indeed some domain name16:32
SpamapSgfidente: or you've named your instances properly and are dynamically setting DNS to the IP assigned..16:33
SpamapSgfidente: EVEN if dhcp_domain is "right" .. the name comes from the user, so dhcp_domain is almost always wrong.16:33
SpamapSIt's actually a huge mess.16:34
gfidenteyeah16:34
gfidenteat least now I see where it is useful to map the name to loopback though16:34
gfidente*when16:34
SpamapSSo while I sympathise with "in this one case where I did things right, it gets in the way" ..16:34
SpamapSI'd almost rather have a no-cloud-init-hosts than a cloud-init-hosts16:35
SpamapSOr a runtime thing that turns it off when it sees that the hostname resolves to an IP.16:35
greghaynesI think we can just make a cloud-init-config element16:36
greghaynesand let people pass in an arbitrary string to that config line16:36
greghaynesand default to 'localhost'16:36
SpamapSharumph16:36
SpamapSmore options means more surface area16:36
greghaynesyea, here its a template string though so it kind of is the right thing16:37
gfidentegreghaynes, so for configuration of cloud-init you can use this https://review.openstack.org/#/c/170137/16:37
shardyIs there a reason this needs to be baked into the image, vs passed in via user_data?16:37
*** ifarkas has quit IRC16:37
shardythat patch should make passing any global cloud-config very easy16:37
greghaynesits cloud-init16:37
gfidenteshardy, indeed I was pointing at that, I would not make the element generic16:37
gfidentegreghaynes, SpamapS it really seems a single setting which is meant to make hostname map to loopback or not16:38
greghaynesif you notice, we have a cloud-init-datasources element too16:38
greghaynesso seems like were going down the path of element per cloud-init config option we want to mess with16:39
greghayneswhich is a bit silly16:39
gfidentenah I am for using the shardy's patch16:39
greghaynessure, I imagine the reason it is a problem is that cloud-init runs before you can do other configuration16:40
*** eghobo has joined #tripleo16:40
*** eghobo has quit IRC16:40
greghaynesand then also reboots16:40
gfidenteyeah so if we really need hostname mapped to loopback early, a single thing to deal with this specific problem might be worth16:41
gfidentebut I would not change by default the default cloud-init behaviour16:41
SpamapSgreghaynes: no this isn't a slippery slope. We're taking the most common things and making them easy to select.16:44
*** eghobo has joined #tripleo16:46
greghaynesAre we asserting that the only two reasonable options here are 'localhost' and not setting that field at all?16:47
gfidentegreghaynes, I think I am, but that is basing on the idea that one can use the template to pass cloud-init params16:48
greghaynesIf we want to support the template string in that option then I dont see the benefit in making the element separate - it just makes it harder to identify which element you need to use16:48
greghaynesSure, we have a fair amount of non tripleo users now too16:48
*** derekh has quit IRC16:49
greghaynesMy thinking is that if were going to go the route of using an env var then the two workflows a user would have to go through are either 1) find the specific cloud-init-foo element, find the var in there, set var, or 2) find the cloud-init-config element, find var in there, set it... the second one is less code and ever so slightly less work16:52
greghaynesbut if its just a simple opt in or not then yes, elements are the way to control that16:52
gfidenteso as per shardy's initial question, is there a reason why this can't be managed via the template?16:53
gfidentein that case one can default to not hacking the cloud-init config but provide it via userdata, as with any other option, if needed16:53
greghaynesthe cases I can think of are 1) not a tripleo user 2) you want the value to be correct before your templating runs 3) if you dont reassert templating on reboot and want your nodes to come back correctly16:54
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Add RabbitMQ as a Pacemaker resource when EnablePacemaker  https://review.openstack.org/17592616:56
openstackgerritDan Prince proposed openstack/tripleo-incubator: configure-vm: allow multiple baremetal-interface's  https://review.openstack.org/17787616:59
openstackgerritDan Prince proposed openstack/tripleo-incubator: Allow configurable bridge names and clean all.  https://review.openstack.org/17787716:59
openstackgerritDan Prince proposed openstack/tripleo-incubator: setup-network: allow creation of multiple bridges  https://review.openstack.org/17787816:59
openstackgerritDan Prince proposed openstack/tripleo-incubator: create-nodes: allow multiple bridge names  https://review.openstack.org/17787916:59
openstackgerritDan Prince proposed openstack/tripleo-incubator: devtest_testenv.sh: support for multiple bridges  https://review.openstack.org/17788016:59
*** yamahata has joined #tripleo17:07
greghayneshttp://logs.openstack.org/81/174681/10/check/gate-dib-dsvm-functests-devstack-f21/493b436/console.html#_2015-04-27_15_26_02_419 is ominious17:09
*** wuhg has quit IRC17:09
*** mmagr|afk is now known as mmagr17:14
*** rhallisey has joined #tripleo17:15
*** lucasagomes is now known as lucas-dinner17:16
*** EmilienM is now known as EmilienM|afk17:17
*** trown|lunch is now known as trown17:21
*** penick has joined #tripleo17:23
*** Goneri has quit IRC17:29
*** bvandenh has joined #tripleo17:31
*** penick has quit IRC17:32
*** sdake_ has joined #tripleo17:34
*** sdake has quit IRC17:36
*** jtomasek has quit IRC17:42
*** sdake has joined #tripleo17:45
*** bvandenh has quit IRC17:46
*** sdake_ has quit IRC17:48
*** dprince has quit IRC17:50
*** athomas has quit IRC17:55
*** EmilienM|afk is now known as EmilienM18:06
*** yamahata has quit IRC18:14
*** gfidente has quit IRC18:20
*** pelix has quit IRC18:20
*** jcoufal has quit IRC18:20
*** jprovazn has joined #tripleo18:25
*** mmagr has quit IRC18:48
*** penick has joined #tripleo19:11
*** dprince has joined #tripleo19:11
*** dguerri is now known as _dguerri19:11
*** spredzy_|afk is now known as spredzy_19:16
spredzy_When I run 'tripleo-incubator/scripts/devtest.sh --trash-my-machine --no-mergepy --no-undercloud' I end up with a service catalog is empty, does that ring a bell to nayone ?19:16
*** _dguerri is now known as dguerri19:17
openstackgerritgreghaynes proposed openstack/diskimage-builder: Initial element tests  https://review.openstack.org/17468119:18
*** jcoufal has joined #tripleo19:21
*** yamahata has joined #tripleo19:21
*** sdake_ has joined #tripleo19:22
*** sdake has quit IRC19:26
dprincespredzy_: Empty seed catalog, or overcloud?19:28
dprincespredzy_: I usually run things manually myself19:28
*** Goneri has joined #tripleo19:29
dprincespredzy_: so devtest_seed first, then source seed env, then devtest_overcloud.sh --no-mergepy19:29
*** dguerri is now known as _dguerri19:32
*** zaneb has joined #tripleo19:34
*** sdake_ has quit IRC19:35
*** sdake has joined #tripleo19:35
greghaynesAre we considering the f20puppet-ha job "voting"?19:39
greghaynesalso, the missing - in that job name is making my OCD go nuts19:39
dprincegreghaynes: no, I don't think it is voting yet19:40
dprincegreghaynes: I haven't seen it ever pass yet...19:40
greghaynesoh, I misread, this is check-tripleo-ironic-overcloud-f20puppet-nonha19:41
greghaynesthat one I imagine is voting19:42
*** Goneri has quit IRC19:46
*** eghobo has quit IRC19:49
*** tserong has joined #tripleo19:55
*** jcoufal has quit IRC19:55
*** eghobo has joined #tripleo20:08
dprincegreghaynes: yes, that one votes20:16
*** dprince has quit IRC20:16
*** _dguerri is now known as dguerri20:20
*** jprovazn has quit IRC20:25
*** eghobo has quit IRC20:30
*** eghobo has joined #tripleo20:32
*** dguerri is now known as _dguerri20:39
*** _dguerri is now known as dguerri20:39
*** jayg is now known as jayg|g0n320:42
openstackgerritMerged openstack/diskimage-builder: Updated from global requirements  https://review.openstack.org/17682620:45
*** Marga_ has joined #tripleo20:47
*** shardy is now known as shardy_z20:53
*** yamahata has quit IRC20:55
*** yamahata has joined #tripleo20:55
*** Marga_ has quit IRC20:59
*** untriaged-bot has joined #tripleo21:00
untriaged-botUntriaged bugs so far:21:00
untriaged-bothttps://bugs.launchpad.net/python-tuskarclient/+bug/144794321:00
openstackLaunchpad bug 1447943 in python-tuskarclient "Missing /v2 path of TUSKAR_URL example in README" [Undecided,In progress] - Assigned to Marek Aufart (maufart)21:00
*** untriaged-bot has quit IRC21:00
*** sdake_ has joined #tripleo21:07
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Make mongodb retrieve conf from proper file  https://review.openstack.org/17796921:08
spredzy_EmilienM, jdob ^21:09
EmilienMnice21:09
spredzy_will fully test it tomorrow, but I am pretty sure that is your error/fix21:09
jdobspredzy_: awesome, I'll check it out in the morning21:10
EmilienMspredzy_: I wonder if it's something we can solve in puppetlabs-mongodb21:11
*** sdake has quit IRC21:11
spredzy_EmilienM, technically both version are 2.621:11
spredzy_only one from EPEL and the other from OSP21:11
EmilienMyeah, it's tricky21:11
spredzy_so if we start make if from epel elsif from osp else21:12
spredzy_will start being messy21:12
*** sdake has joined #tripleo21:15
*** trown is now known as trown|outttypeww21:17
*** mmagr has joined #tripleo21:18
*** sdake_ has quit IRC21:19
*** lblanchard has quit IRC21:22
*** mmagr has quit IRC21:23
openstackgerritAntoine Musso proposed openstack/diskimage-builder: Test elements have README.rst file  https://review.openstack.org/17783221:24
*** lucas-dinner has quit IRC21:29
*** eghobo has quit IRC21:33
*** thrash is now known as thrash|g0ne21:43
*** sdake_ has joined #tripleo21:46
*** Marga_ has joined #tripleo21:47
*** sdake has quit IRC21:49
*** eghobo has joined #tripleo21:49
*** eghobo has quit IRC21:59
*** penick has quit IRC22:01
*** Marga_ has quit IRC22:01
*** eghobo has joined #tripleo22:03
openstackgerritgreghaynes proposed openstack/diskimage-builder: Initial element tests  https://review.openstack.org/17468122:05
*** penick has joined #tripleo22:07
openstackgerritgreghaynes proposed openstack/diskimage-builder: Initial element tests  https://review.openstack.org/17468122:18
*** mcornea has quit IRC22:26
*** eghobo has quit IRC22:35
openstackgerritgreghaynes proposed openstack/diskimage-builder: Initial element tests  https://review.openstack.org/17468122:40
openstackgerritgreghaynes proposed openstack/diskimage-builder: Initial element tests  https://review.openstack.org/17468122:48
*** eghobo has joined #tripleo22:53
*** sdake_ has quit IRC22:57
openstackgerritgreghaynes proposed openstack/diskimage-builder: Initial element tests  https://review.openstack.org/17468123:00
openstackgerritgreghaynes proposed openstack/diskimage-builder: Initial element tests  https://review.openstack.org/17468123:04
*** yuanying has joined #tripleo23:06
openstackgerritgreghaynes proposed openstack/diskimage-builder: Initial element tests  https://review.openstack.org/17468123:07
openstackgerritgreghaynes proposed openstack/diskimage-builder: Initial element tests  https://review.openstack.org/17468123:15
*** penick has quit IRC23:18
*** panda has quit IRC23:26
*** panda has joined #tripleo23:26
*** Marga_ has joined #tripleo23:32
SpamapSgreghaynes: can I assume 174681 is WIP, you're just iterating on the buildfails?23:40
greghaynesyes23:40
greghaynesI am totally baffled23:40
SpamapSbash is good for that23:41
greghaynesIt looks like f21 no longer has /bin in the path by default, and our logic for adding /bin to $PATH in the chroot is broken23:42
greghaynesoh my23:51
openstackgerritgreghaynes proposed openstack/diskimage-builder: Initial element tests  https://review.openstack.org/17468123:55
greghaynesI think theres about 4 layers of bugs that was preventing our static path setting from actually doing what we thought it did23:56
greghaynes\O/ test23:56
greghaynesyep23:57
greghaynespasses now23:57
* greghaynes splits the fix out into different patch23:57
*** noslzzp has quit IRC23:57

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