Monday, 2015-12-14

*** dmacpher has quit IRC00:21
*** dmacpher has joined #tripleo00:22
*** jcoufal has quit IRC00:41
*** chlong has joined #tripleo00:48
*** bvandenh has joined #tripleo02:02
*** bvandenh has quit IRC02:21
*** tserong has joined #tripleo03:11
*** tserong has joined #tripleo03:11
*** mgagne has quit IRC04:36
*** larstobi has quit IRC04:36
*** dsneddon_biab has quit IRC04:37
*** mgagne has joined #tripleo04:37
*** mgagne is now known as Guest7323304:37
*** larstobi has joined #tripleo04:38
*** dsneddon_biab has joined #tripleo04:38
*** masco has joined #tripleo05:11
*** devvesa has joined #tripleo05:28
*** jrist has quit IRC05:36
*** jrist has joined #tripleo05:38
*** jrist has joined #tripleo05:38
*** yuanying has joined #tripleo06:11
*** shivrao has joined #tripleo06:13
*** shivrao_ has joined #tripleo06:16
*** shivrao has quit IRC06:18
*** shivrao_ has quit IRC06:19
*** shivrao has joined #tripleo06:19
*** shivrao has quit IRC06:26
*** shivrao has joined #tripleo06:39
*** shivrao has quit IRC06:43
*** shivrao has joined #tripleo06:45
*** olap has joined #tripleo06:52
*** rcernin has joined #tripleo07:01
*** devvesa has quit IRC07:04
*** openstackstatus has joined #tripleo07:07
*** ChanServ sets mode: +v openstackstatus07:07
*** openstackgerrit has joined #tripleo07:10
*** rasca has joined #tripleo07:10
*** rasca has quit IRC07:11
openstackgerritIan Wienand proposed openstack/diskimage-builder: yum-minimal: Don't create eth0 & eth1  https://review.openstack.org/25717207:16
openstackgerritIan Wienand proposed openstack/diskimage-builder: yum-minimal: leave behind dummy /etc/resolv.conf  https://review.openstack.org/25717307:16
openstackgerritsonu proposed openstack/python-tripleoclient: Removes MANIFEST.in as it is not needed explicitely by PBR  https://review.openstack.org/25717807:25
*** ukalifon has joined #tripleo07:45
openstackgerritSteve Martinelli proposed openstack/tuskar: use keystonemiddleware instead of keystoneclient.middleware  https://review.openstack.org/25718607:45
*** chlong has quit IRC07:46
*** jprovazn has joined #tripleo08:00
openstackgerritSteve Martinelli proposed openstack/tuskar: use keystonemiddleware instead of keystoneclient.middleware  https://review.openstack.org/25718608:00
*** jcoufal has joined #tripleo08:01
*** olap has quit IRC08:06
*** gchamoul- is now known as gchamoul08:18
*** rasca has joined #tripleo08:21
*** liverpooler has joined #tripleo08:21
*** pino|work has joined #tripleo08:24
*** devvesa has joined #tripleo08:24
*** shardy has joined #tripleo08:25
*** aufi has joined #tripleo08:25
openstackgerritMerged openstack/python-tripleoclient: Set NeutronMetadataProxySharedSecret  https://review.openstack.org/25485708:25
*** paramite has joined #tripleo08:35
*** shivrao has quit IRC08:37
devvesaHi Anyone with more experience than me on Puppet can tell me why the gates fail now on this patch:08:41
devvesahttps://review.openstack.org/#/c/251319/08:41
openstackgerritMarek Aufart proposed openstack/tripleo-common: Add environment capabilities  https://review.openstack.org/23253408:41
*** paramite has quit IRC08:43
*** ifarkas has joined #tripleo08:45
*** shardy has quit IRC08:52
*** gfidente has joined #tripleo08:53
*** gfidente has quit IRC08:53
*** gfidente has joined #tripleo08:53
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: WIP: Allow predictable IPs for Controllers on the ctlplane  https://review.openstack.org/25600308:54
openstackgerritDougal Matthews proposed openstack/tripleo-common: [WIP] Add the Plan deploy endpoint to start a Heat deploy  https://review.openstack.org/25638408:55
gfidentemarios, christmas gift? https://review.openstack.org/#/c/222497/08:58
mariosgfidente: bah! humbug!09:00
gfidenteI had to rebase it, but it's same you voted on before!09:01
gfidenteand I fixed what you asked about oac09:01
mariosgfidente: k but later. have bumped reviews. trying to understand wtf with the floating ip issue i read about this morning (looking at an env now)09:01
gfidenteauch, haven't got there yet I think myself09:02
*** mcornea has joined #tripleo09:06
*** jistr has joined #tripleo09:12
*** nico_auvv has joined #tripleo09:13
*** mbound_ has joined #tripleo09:14
*** shardy has joined #tripleo09:19
*** devvesa has quit IRC09:23
*** devvesa has joined #tripleo09:23
*** akrivoka has joined #tripleo09:24
*** jrist has quit IRC09:26
*** leanderthal has joined #tripleo09:26
*** jrist has joined #tripleo09:30
*** jrist has joined #tripleo09:30
*** morazi has joined #tripleo09:30
*** yamahata has quit IRC09:32
*** athomas has joined #tripleo09:37
*** leanderthal has quit IRC09:42
*** jrist has quit IRC09:46
*** jrist has joined #tripleo09:47
*** jrist has joined #tripleo09:47
*** olap has joined #tripleo09:50
*** bandini has quit IRC09:56
*** bandini has joined #tripleo09:56
*** mkovacik has joined #tripleo09:56
*** jrist has quit IRC09:57
*** jrist has joined #tripleo09:59
*** jrist has joined #tripleo09:59
*** dtantsur|afk is now known as dtantsur10:00
*** leanderthal has joined #tripleo10:01
nico_auvvhello, could somebody merge this one please? it's a blocker for another patch to be reviewed in openstack-tripleo-heat-templates... https://review.openstack.org/#/c/251920/110:05
*** jtomasek has joined #tripleo10:06
*** ifarkas has quit IRC10:06
*** paramite has joined #tripleo10:16
*** electrofelix has joined #tripleo10:19
*** leanderthal has quit IRC10:21
*** hewbrocca has joined #tripleo10:21
*** rcernin has quit IRC10:25
devvesaHi jisrt: Do you have a second to look at this review https://review.openstack.org/#/c/251319/?10:30
devvesaI have no idea why the puppet gates fail10:30
*** pleia2 has quit IRC10:32
*** nico_auvv has quit IRC10:41
*** nico_auvv has joined #tripleo10:41
openstackgerritJaume Devesa proposed openstack/tripleo-heat-templates: MidoNet heat templates  https://review.openstack.org/23899810:43
*** rleander has joined #tripleo10:43
*** tosky has joined #tripleo10:45
shardynico_auvv: done, thanks!10:47
openstackgerritMerged openstack/tripleo-puppet-elements: Install puppet-timezone via puppet-modules element  https://review.openstack.org/25192010:48
*** rleander has quit IRC10:49
hewbroccadevvesa: ITYM jistr ^^^10:49
devvesahewbrocca: Yes, I do10:51
devvesa:)10:52
*** mgould has joined #tripleo10:56
nico_auvvthx shardy !10:56
*** leanderthal has joined #tripleo10:57
* jistr multitasking :)10:58
jistrhi devvesa10:58
devvesaHi jistr10:59
jistrthe log seems like a problem with the job itself, i just glanced over the latest patches submitted to puppet modules and they have the same problem https://review.openstack.org/#/c/255913/10:59
jistrthese jobs are maintained by puppet folks i think10:59
jistrso perhaps #puppet-openstack channel would be a good place to ask about that failure. I'd assume they'll have to fix it soon as it looks like no puppet-<anything> changes can land at the moment.11:00
devvesaSo that can be caused by a change on the job definition in project-config11:00
jistryes, or a new version of some rubygem11:00
devvesaOk. Thanks! I'll ask them11:00
*** rleander has joined #tripleo11:04
*** rasca has quit IRC11:11
*** ifarkas has joined #tripleo11:35
*** chlong has joined #tripleo11:39
*** leanderthal has quit IRC11:43
*** derekh has joined #tripleo11:47
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: Adding psych on Gemfile explicitly  https://review.openstack.org/25729411:53
*** slagle has quit IRC11:53
devvesajistr: I found this: https://github.com/bundler/bundler/issues/206811:54
devvesaSo I pushed the review ^^11:54
devvesaThe bug is quite TL;DR :D But I think I got the idea11:54
*** slagle has joined #tripleo11:54
openstackgerritMerged openstack/python-tuskarclient: Update README.md for repo project  https://review.openstack.org/21442711:54
devvesaIt is old, but let the jenkins run if it solves the issue11:55
devvesajistr, shardy. The patch fixes the issue12:03
devvesaCan you review it, please? It is quick12:03
*** hewbrocca has quit IRC12:04
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency  https://review.openstack.org/25131912:07
jistrthanks devvesa, i +2'd the Gemfile change12:23
*** rasca has joined #tripleo12:29
openstackgerritOpenStack Proposal Bot proposed openstack/python-tripleoclient: Updated from global requirements  https://review.openstack.org/25446312:35
openstackgerritOpenStack Proposal Bot proposed openstack/python-tuskarclient: Updated from global requirements  https://review.openstack.org/23290412:35
*** dprince has joined #tripleo12:38
*** trown|outttypeww is now known as trown12:38
openstackgerritMerged openstack/tripleo-common: Set a default timeout for 240 minutes for Heat Updates  https://review.openstack.org/25495412:39
*** weshay_xchat has joined #tripleo12:48
*** weshay_xchat is now known as weshay12:48
*** thrash|wknd is now known as thrash12:50
*** mburned has joined #tripleo12:56
dtantsurbnemec, dprince, hi, any plans to backport https://review.openstack.org/#/c/247088/ to liberty?13:00
dprincedtantsur: I'd be fine with that13:01
dtantsurif nobody is working on it already, I'll do it13:01
*** jdob has joined #tripleo13:03
*** rebrego has joined #tripleo13:05
*** regebro is now known as Guest9342313:05
*** rebrego is now known as regebro13:05
trowndtantsur: I am not sure we would want to change liberty defaults...13:05
dtantsurtrown, well, we're using the deprecated ramdisk13:06
dtantsurand as downstream, we're going to switch to IPA anyway13:06
trowndtantsur: btw something is broken with trunk IPA... working on gathering more info13:06
trowndtantsur: dprince, derekh, I think we need IPA in the list of 'current' packages... we will get broken if we dont vote there... but in my view we are broken if we are not able to move the 'current-tripleo' pin13:08
dtantsurnot sure I understood what is broken..13:08
trowndtantsur: for tripleoci, I would say broken is not being able to land anything because CI cant pass13:09
dtantsuryeah, but I don't understand how IPA is related right now.. is something broken with it?13:10
openstackgerritDmitry Tantsur proposed openstack/python-tripleoclient: Deprecate bash ramdisks  https://review.openstack.org/25733813:10
dtantsurI've put a patch ^^ do whatever you feel like with it :)13:10
trowndtantsur: something with it, DIB, or tripleoclient... still not havent totally narrowed down what13:10
dtantsurtrown, how does it manifest?13:10
trowndtantsur: boot loop on deploy13:10
trowndtantsur: ramdisk boots... then just shuts off over and over13:11
dtantsurtrown, at what stage does it gets shut off?13:11
trowndtantsur: images here have the issue13:11
trownhttps://ci.centos.org/artifacts/rdo/images/mitaka/delorean/testing/13:11
dprinceEmilienM: morning, hey I did some manual testing on this and posted some feedback https://review.openstack.org/#/c/234704/1113:12
trowndtantsur: I am testing a fix in THT for another issue on trunk... but then I will boot up that undercloud.qcow2 and work on getting more info13:13
openstackgerritDmitry Tantsur proposed openstack/python-tripleoclient: Allow assigning profiles based on XXX_profile capabilities  https://review.openstack.org/25040513:14
dtantsurquick question folks, does 'overcloud deploy' have something like --dry-run mode to only run validations?13:15
*** jayg|g0n3 is now known as jayg13:16
EmilienMdprince: ok I will13:16
dprinceEmilienM: cool. for reference this is what created the old service/endpoint (before your patch) http://git.openstack.org/cgit/openstack/os-cloud-config/tree/os_cloud_config/keystone.py#n7013:17
dprinceEmilienM: I actually think this fix would let us re-land the overcloud patch too.13:17
dprinceEmilienM: I'm looking into that...13:17
EmilienMdprince: in meeting now.13:17
openstackgerritBrent Eagles proposed openstack/tripleo-heat-templates: Enable setting Neutron QoS feature  https://review.openstack.org/24705613:20
trownderekh: that swift replica issue also happens in THT...13:20
dprincetrown: I'd like to send a mail to the list about the replicas thing13:20
dprincetrown: https://review.openstack.org/#/c/256631/13:20
openstackgerritDougal Matthews proposed openstack/tripleo-common: Add the missing plan name to the PlanAlreadyExistsError  https://review.openstack.org/25734313:20
dprincetrown: for the undercloud we can simply set to 1 (for now)13:20
EmilienMdprince: I'll look in a few13:21
dprincetrown: for the overcloud we are now used for SAIO13:21
trowndprince: SAIO?13:21
*** liverpooler has quit IRC13:21
dprincetrown: Single All In One13:21
d0ugaldtantsur: I don't think so, that would be a good idea tho'13:21
dprincetrown: like if you deploy an overcloud with just a single swift storage node13:22
dtantsurok, I'll try to do it13:22
dprincetrown: the new code wants replicas to be <= devices. And we hardcode it to 313:22
*** tzumainn has joined #tripleo13:23
trowndprince: right... derekh has a similar patch for undercloud btw... https://review.openstack.org/25599513:23
trowndprince: though yours has a bug... so I think you win :)13:23
dprincetrown: oh, I see13:24
dprincetrown: yeah, I missed the patch w/ no bug13:24
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates: Pacemaker maintenance mode for the duration of Puppet run on update  https://review.openstack.org/24509313:24
trowndprince: we see the same thing in a default `overcloud deploy --templates` as well... I imagine we would not see it on HA though13:25
dprincetrown: right, and before we make a change there I'd like to ask Clay13:25
dprincetrown: on openstack-dev how to best handle this13:25
dprincetrown: because it seems to me this is a breaking sort of change13:26
*** rhallisey has joined #tripleo13:26
trowndprince: fair enough... trunk is broken until that is resolved though there are a couple other issues blocking trunk CI too13:26
dprincetrown: I'm wondering if in the end we might not have to make any changes for this... because it is a swift bug13:27
trowndprince: hmmm... we are meant to be able to have multiple replicas per device? I thought we were just getting away with using old behavior that was deprecated?13:28
trownand now the deprecated was removed13:28
dprincetrown: if that is the case what would you set replicas to for our overcloud (now)13:29
trown(# of controllers)?13:29
dprincetrown: I think we'd go for a max of 3 perhaps13:30
dprincetrown: so #controllers, until it hits 3 and then leave it at that13:30
trownso min(3, # of controllers)13:31
dprincebasically13:31
trowndprince: cool, it would be good to confirm expected swift behavior on ML13:31
openstackgerritDmitry Tantsur proposed openstack/python-tripleoclient: Add --dry-run argument to overcloud deploy command  https://review.openstack.org/25734813:31
dtantsurd0ugal, ^^13:31
trownmy perception is just based on conversation with derekh last week13:32
dprincetrown: yeah, I am just a little fuzzy on what exactly happened. That patch was rather large13:32
d0ugaldtantsur: thanks13:34
dtantsurd0ugal, btw is there a particular reason a lot of errors in this command are logged with tracebacks? does not look too user friendly IMO13:37
d0ugaldtantsur: Not that I remember.13:37
d0ugaldtantsur: but I don't think any of the logging output is displayed by default, is it?13:37
dtantsurd0ugal, log.exception is displayed, at least for me13:38
d0ugalhmm, actually, that makes sense that it would be13:38
d0ugalWe should probably improve that13:38
d0ugalcc: regebro ^13:38
dtantsurd0ugal, regebro, here how it looks for me: http://paste.openstack.org/show/481804/13:40
dtantsur(this is a fresh devstack, so failures are quite expected)13:40
*** julim has joined #tripleo13:42
d0ugaldtantsur: Yeah, that's just lazy on our part I think.13:43
d0ugal.error() would make more sense13:44
dtantsur++13:44
*** jaosorior has joined #tripleo13:47
regebrodtantsur, d0ugal I like that it failes with tracebacks.13:47
dtantsurregebro, it should not be shown to users13:47
d0ugalregebro: Why? I can't see when an attributeerror would be useful13:48
dtantsurwith --debug - yes13:48
dtantsurbut not by default13:48
regebroIt should definitely not show a traceback unless it's an actual failure, ie, the command stops.13:48
openstackgerritDmitry Tantsur proposed openstack/python-tripleoclient: Add --dry-run argument to overcloud deploy command  https://review.openstack.org/25734813:49
dtantsurregebro, right, and here we have a user failure, not our failure13:49
regebrodtantsur: Yeah, this case is a good example, because you have multiple errors, and it also says "Error finding 'bm-deploy-ramdisk' in glance."13:50
regebroI guess a general rule could be "Don't show a traceback if you are handling the error in some way".13:51
*** mcornea has quit IRC13:53
*** mcornea has joined #tripleo13:54
openstackgerritDmitry Tantsur proposed openstack/python-tripleoclient: Allow assigning profiles based on XXX_profile capabilities  https://review.openstack.org/25040513:55
derekhdprince: trown sorry was at lunch13:57
derekh<dprince> trown: yeah, I am just a little fuzzy on what exactly happened. That patch was rather large13:57
derekhdprince: RE ^, swift used to warn about having less devices then replica's, it now errors instead13:57
dprincederekh: okay, so maybe we have no choice here13:58
derekhtrown: you mentioned another bug on current did you?14:02
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: Adding psych on Gemfile explicitly  https://review.openstack.org/25729414:02
*** fabbione has quit IRC14:03
dprincederekh: anyways, sorry I missed your patch. +2 although note that instack had a bunch of useless tripleo::ringbuilder stuff in it14:03
derekhdprince: yup, I deliberately didn't change it as it wasn't the bug I was trying to fix. My thoughts were that they should be removed in a followup patch14:05
dprincederekh: don't care who's patch lands, but it might be worth updating your patch to address the bug though14:05
*** fabbione has joined #tripleo14:05
derekhdprince: either way, doesn't make much difference, will +2 yours now14:05
dprincederekh: then I'll rebase mine to just remove the cruft14:05
derekhdprince: ok, will do that so14:05
*** pblaho has joined #tripleo14:06
trownderekh: ya there are two more... but I need to actually file bugs for them... probably wont get to that until late afternoon today14:07
derekhtrown: ok14:07
trowntripleoclient deploy command does not work with latest openstackclient, and there is some bug in the IPA disk we build14:07
openstackgerritDerek Higgins proposed openstack/instack-undercloud: Change undercloud replica count to 1  https://review.openstack.org/25599514:07
derekhdprince: ^14:08
derekhAlso it would be great to get the periodic job working, if somebody could look at  https://review.openstack.org/#/c/244519/  https://review.openstack.org/#/c/244526/214:09
dprincetrown: sounds good, ping me so I don't miss reviews of these things. Or maybe create an etherpad to organize it if it is too many patches14:09
*** paramite is now known as paramite|afk14:11
trowndprince: cool will do14:12
openstackgerritDan Prince proposed openstack/instack-undercloud: Remove tripleo::ringbuilder cruft  https://review.openstack.org/25663114:12
dprincederekh: ^^14:12
*** paramite|afk is now known as paramite14:12
trowndprince: I have an etherpad... it is pretty sparse but I will be filling in bugs/patches as I file them14:13
trowndprince: https://etherpad.openstack.org/p/delorean_master_current_issues14:13
openstackgerritLennart Regebro proposed openstack/python-tripleoclient: Make the undercloud install test discoverable  https://review.openstack.org/25641014:14
jaosoriortrown: Do you have a launchpad id for item number 2 in https://etherpad.openstack.org/p/delorean_master_current_issues ?14:14
openstackgerritDmitry Tantsur proposed openstack/python-tripleoclient: Allow assigning profiles based on XXX_profile capabilities  https://review.openstack.org/25040514:14
trownjaosorior: I do not, on my list for today to file it14:15
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency  https://review.openstack.org/25131914:15
openstackgerritLucas Alvares Gomes proposed openstack/python-tripleoclient: Add dynamic-login element to the AGENT_IMAGE_ELEMENT list  https://review.openstack.org/25736914:15
trownjaosorior: it is because of the switch to the openstack sdk14:15
*** rlandy has joined #tripleo14:16
jaosoriortrown: On which API of the openstack sdk? (Sorry, lacking context here :/ )14:16
trownjaosorior, ya it might be better if I just put all the info in a bug :) there is quite a bit of context not in the etherpad but just in my head :)14:17
jaosoriortrown: Seems legit14:18
*** jcoufal_ has joined #tripleo14:19
*** jcoufal has quit IRC14:22
*** rbrady has joined #tripleo14:22
*** regebro has quit IRC14:26
openstackgerritMerged openstack/puppet-tripleo: Adding psych on Gemfile explicitly  https://review.openstack.org/25729414:26
*** lblanchard has joined #tripleo14:26
shardyHey folks, couple of easy +A's if anyone has a moment:14:27
shardyhttps://review.openstack.org/#/c/255407/14:27
shardyhttps://review.openstack.org/#/c/254667/14:27
jaosoriorshardy: Wouldn't https://review.openstack.org/#/c/255407/ need https://review.openstack.org/#/c/253354/ ??14:28
jaosoriorshardy: Ah, nevermind, I now saw that it was the one without the -without-mergepy suffix14:29
* jaosorior is extremely jetlagged14:29
shardyjaosorior: Yeah, it's just removing the old now-unused registry14:30
jaosoriorshardy: Yeah, still need coffee haha. Greetings from Mexico by the way.14:30
shardyjaosorior: there was some discussion re backwards compatibility and tripleoclient recently14:30
*** NobodyCa1 is now known as NobodyCam14:30
shardyit may turn out that we need to support falling back to the liberty naming of things, so that e.g Mitaka can deploy both Liberty and Mitaka overclouds14:30
*** regebro has joined #tripleo14:31
jaosoriorshardy: well, I could modify that CR to open the overcloud.yaml and then if that fails, fall back to the without-mergepy naming14:31
*** rpothier has joined #tripleo14:31
shardyjaosorior: Yeah, if it's not too much work that'd be cool14:31
jaosoriorshardy: No problem14:32
*** lucasagomes is now known as lucas-hungry14:34
*** trown is now known as trown|outttypeww14:40
openstackgerritBrent Eagles proposed openstack/tripleo-heat-templates: Enable setting Neutron QoS feature  https://review.openstack.org/24705614:46
*** masco has quit IRC14:51
openstackgerritDmitry Tantsur proposed openstack/python-tripleoclient: Allow assigning profiles based on XXX_profile capabilities  https://review.openstack.org/25040514:53
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: Adding psych on Gemfile explicitly  https://review.openstack.org/25742014:56
*** akrivoka has quit IRC14:58
*** jprovazn has quit IRC14:59
*** akrivoka has joined #tripleo15:00
*** eggmaster has joined #tripleo15:05
*** pradk has joined #tripleo15:08
openstackgerritMerged openstack/tripleo-heat-templates: Remove deprecated overcloud-resource-registry.yaml  https://review.openstack.org/25540715:14
openstackgerritMerged openstack/tripleo-heat-templates: Enable per-role SchedulerHints  https://review.openstack.org/25466715:20
openstackgerritMerged openstack/tripleo-heat-templates: Add pcmk constraints against haproxy-clone only if applicable  https://review.openstack.org/25530115:22
dprincetrown|outttypeww: https://bugs.launchpad.net/tripleo/+bug/152595715:33
openstackLaunchpad bug 1525957 in tripleo "nova neutronclient fails due to deprecated auth options" [Critical,New]15:33
dprincetrown|outttypeww: added to the etherpad15:34
*** lucas-hungry is now known as lucasagomes15:37
*** rpothier has left #tripleo15:48
*** mcornea has quit IRC15:50
*** masco has joined #tripleo15:51
*** rleander is now known as leanderthal|afk15:55
*** athomas has quit IRC15:56
*** 21WAAGP61 has quit IRC15:57
*** aufi has quit IRC15:57
*** andreaf_ has joined #tripleo15:57
*** mkovacik has quit IRC15:57
*** dtantsur is now known as dtantsur|afk15:59
*** sthillma has joined #tripleo16:00
*** akrivoka has quit IRC16:00
openstackgerritEmilien Macchi proposed openstack/instack-undercloud: Manage keystone related configuration via Puppet  https://review.openstack.org/23470416:01
openstackgerritBen Swartzlander proposed openstack/tripleo-heat-templates: Enable Manila integration  https://review.openstack.org/18813716:02
*** jistr has quit IRC16:03
openstackgerritBen Swartzlander proposed openstack/tripleo-heat-templates: Add NetApp integration to Manila  https://review.openstack.org/18813816:03
*** olap has quit IRC16:04
openstackgerritBen Swartzlander proposed openstack/tripleo-heat-templates: Add Gluster integration to Manila  https://review.openstack.org/23093616:04
*** Goneri has quit IRC16:06
*** morazi has quit IRC16:06
*** ifarkas has quit IRC16:06
*** Goneri has joined #tripleo16:09
*** shivrao has joined #tripleo16:14
*** Goneri has quit IRC16:16
*** shivrao_ has joined #tripleo16:17
*** shivrao has quit IRC16:18
*** shivrao_ is now known as shivrao16:18
*** Goneri has joined #tripleo16:19
*** andreaf has quit IRC16:21
*** andreaf_ is now known as andreaf16:21
*** jtomasek has quit IRC16:22
*** stendulker has joined #tripleo16:24
*** gfidente has quit IRC16:37
*** paramite has quit IRC16:39
*** Goneri has quit IRC16:42
*** gfidente has joined #tripleo16:43
*** gfidente has quit IRC16:43
*** gfidente has joined #tripleo16:43
*** sthillma has quit IRC16:44
*** Goneri has joined #tripleo16:45
*** pleia2 has joined #tripleo16:48
openstackgerritRyan Brady proposed openstack/tripleo-common: Fixes create_plan behavior in PlanManager  https://review.openstack.org/25748116:50
*** shivrao has quit IRC16:50
*** cwolferh has joined #tripleo16:52
*** absubram has joined #tripleo16:52
*** jtomasek has joined #tripleo17:18
*** mbound_ has quit IRC17:20
*** openstackstatus has quit IRC17:20
*** rwsu has joined #tripleo17:20
*** openstackstatus has joined #tripleo17:23
*** ChanServ sets mode: +v openstackstatus17:23
*** devvesa has quit IRC17:31
*** jcoufal_ has quit IRC17:32
*** dsneddon_biab is now known as dsneddon17:40
*** olap has joined #tripleo17:49
*** penick has joined #tripleo17:51
*** penick has quit IRC17:55
*** pradk has quit IRC17:57
*** ukalifon has quit IRC17:59
*** yamahata has joined #tripleo18:02
*** derekh has quit IRC18:04
*** rhallisey has quit IRC18:06
*** stendulker has quit IRC18:07
*** rhallisey has joined #tripleo18:12
*** rhallisey_ has joined #tripleo18:13
*** rhallisey_ has quit IRC18:13
*** rhallisey_ has joined #tripleo18:13
*** pradk has joined #tripleo18:13
*** rhallisey_ has quit IRC18:15
*** nico_auvv has quit IRC18:15
*** trown|outttypeww is now known as trown18:20
*** shivrao has joined #tripleo18:22
*** mgould has quit IRC18:23
openstackgerritGiulio Fidente proposed openstack/python-tripleoclient: We do not need to pass a NeutronControlPlaneID  https://review.openstack.org/25752018:29
dprinceslagle: hi, so I spent some time manually testing this one. Found an issue and fixed it... https://review.openstack.org/#/c/234704/18:33
thrashtrown: dprince the conditional include never did work. It worked when the file wasn't there, but it didn't ever seem to work when it was there.18:34
thrashtrown: dprince so I'm thinking this: https://review.gerrithub.io/#/c/255380/18:34
thrashwdyt?18:34
trownthrash: that is a built in thing?18:35
thrashtrown: yep18:35
dprincethrash: I'm probably okay w/ that as a temporary fix.18:35
trownthrash: cool... learned something new for rpm packaging18:35
dprincethrash: perhaps we remove it quickly once things get past this blocker18:35
thrashdprince: yes. It should allow the CI to run upstream.18:35
dprinceslagle: see also my comment here https://review.openstack.org/#/c/222227/18:36
thrashdprince: that's the idea. Notice that the script is no longer in the spec. So, once the upstream passes, then we can remove that line and add in the script18:36
dprinceslagle: if you rebase that patch on the keystone v3 one... and remove the keystone stuff from your patch I think it'll all be working18:36
dprinceslagle: by my tests it was anyway18:36
*** masco has quit IRC18:37
slagledprince: ok, thx. i saw your review on 222227, and i'll update it accordingly. it's a better solution18:39
*** david-lyle_ has joined #tripleo18:42
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Manage keystone initialization directly in t-h-t manifests  https://review.openstack.org/24416218:45
*** david-lyle has quit IRC18:45
gfidentethrash, do you remember why we ever did https://review.openstack.org/257520 ?18:50
dprincetrown: I added a puppet-nova fix here as well https://etherpad.openstack.org/p/delorean_master_current_issues18:52
dprincetrown: just hit that one today...18:52
trowndprince: nice... I think that is a new one... fixing trunk/current long enough to promote something is hard18:53
trowndprince: we are finding issues at a much higher rate than resolving them18:54
*** tosky has quit IRC18:55
dprincetrown: yeah, I feel like we are in a leaky boat. And our buckets aren't big enough or something. We need a bilge pump18:55
dprincetrown: anyways, lets bring these issues to the table in the meeting tomorrow18:55
dprincetrown: anyone working on the overcloud swift issue yet?18:56
trowndprince: sounds good, I am working on filing bugs for the IPA and tripleoclient issues18:56
dprincetrown: I can probably take that...18:56
trowndprince: dont think so18:56
dprincetrown: okay, I will notify derek so he isn't working on the same18:56
dprinceshardy, slagle, jdob, bnemec, gfidente: one of you might send this one? https://review.openstack.org/#/c/255995/18:57
gfidentedprince, ack, thanks18:58
dprincegfidente: you might go ahead and hit this one too https://review.openstack.org/#/c/256631/19:02
dprincegfidente: a related cleanup of the cruft...19:02
openstackgerritJason Guiditta proposed openstack/tripleo-puppet-elements: Add monitoring dependencies  https://review.openstack.org/25205119:02
*** leanderthal|afk has quit IRC19:05
*** sparkyco_ has joined #tripleo19:06
openstackgerritJuan Antonio Osorio Robles proposed openstack/python-tripleoclient: Use renamed overcloud.yaml template  https://review.openstack.org/25335419:07
thrashgfidente: it was probably done because that's what the old instack script did.19:09
*** thrash is now known as thrash|bbl19:12
*** sthillma has joined #tripleo19:12
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Set swift replicas = min(device_count, replicas)  https://review.openstack.org/25754019:13
dprincetrown: ^^^19:13
trowndprince: sweet that was fast19:13
dprincetrown: yeah, lets see if CI likes it but I think it'll go19:14
trowndprince: cool, after I get the IPA bug filed I can verify too19:15
dprincetrown: I added all of our Swift guys as well19:16
dprincetrown: anything else you'd like me to look into?19:16
*** yamahata has quit IRC19:17
dprincetrown: I will keep watch on this etherpad for new reviews19:17
*** yamahata has joined #tripleo19:17
trowndprince: the tripleoclient one might be pretty major, but it is probabbly worth waiting until I file the bug19:17
dprincetrown: file it, and we'll see19:18
*** ukalifon1 has joined #tripleo19:20
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Align template defaults with the client  https://review.openstack.org/25754119:26
*** sparkyco_ has quit IRC19:28
openstackgerritGiulio Fidente proposed openstack/python-tripleoclient: We do not need to pass a NeutronControlPlaneID  https://review.openstack.org/25752019:29
*** dprince has quit IRC19:32
jaosorioranybody has some spare cycles to check this one out? https://review.openstack.org/#/c/254771/19:37
openstackgerritGiulio Fidente proposed openstack/python-tripleoclient: WIP: Forcibly clear parameters (now passed as parameter_defaults)  https://review.openstack.org/25667019:39
*** ukalifon1 has quit IRC19:39
jaosoriorAlso, if someone can check the following backports, would be nice https://review.openstack.org/#/q/status:open+project:openstack/tripleo-heat-templates+branch:stable/liberty+topic:tls_enablement,n,z19:39
*** sthillma has quit IRC19:41
*** sthillma has joined #tripleo19:42
*** ukalifon1 has joined #tripleo19:47
*** lucasagomes is now known as lucas-dinner19:51
*** ukalifon1 has quit IRC19:51
openstackgerritJohn Trowbridge proposed openstack/python-tripleoclient: neutron client has been migrated to openstack SDK  https://review.openstack.org/25755019:53
*** dprince has joined #tripleo19:54
*** electrofelix has quit IRC19:57
openstackgerritMerged openstack/instack-undercloud: Change undercloud replica count to 1  https://review.openstack.org/25599519:58
*** penick has joined #tripleo20:01
*** eil397 has joined #tripleo20:02
*** penick has quit IRC20:03
*** mbound_ has joined #tripleo20:04
*** whydidyoustealmy has quit IRC20:07
*** shakamunyi has quit IRC20:07
*** david-lyle_ has quit IRC20:10
*** eil397 has quit IRC20:11
*** eil397 has joined #tripleo20:12
*** penick has joined #tripleo20:12
*** david-lyle has joined #tripleo20:13
*** rhallisey_ has joined #tripleo20:39
*** rhallisey_ has quit IRC20:40
*** rhallisey_ has joined #tripleo20:40
*** rhallisey_ has quit IRC20:41
*** thrash|bbl is now known as thrash20:47
*** rhallisey has quit IRC20:52
*** penick has quit IRC20:55
*** penick has joined #tripleo20:58
*** david-lyle has quit IRC21:01
*** jayg is now known as jayg|g0n321:03
openstackgerritJohn Trowbridge proposed openstack/python-tripleoclient: neutron client has been migrated to openstack SDK  https://review.openstack.org/25755021:06
*** david-lyle has joined #tripleo21:07
openstackgerritBen Swartzlander proposed openstack/tripleo-heat-templates: Enable Manila integration  https://review.openstack.org/18813721:09
*** david-lyle has quit IRC21:12
openstackgerritBen Swartzlander proposed openstack/tripleo-heat-templates: Enable Manila integration  https://review.openstack.org/18813721:20
*** dprince has quit IRC21:28
*** lblanchard has quit IRC21:34
*** rhallisey has joined #tripleo21:34
*** absubram has quit IRC21:41
*** chlong has quit IRC21:51
*** pradk has quit IRC21:54
*** jtomasek has quit IRC21:55
openstackgerritMerged openstack/tripleo-heat-templates: Add output for host entries  https://review.openstack.org/24017122:05
*** julim has quit IRC22:08
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Fix typo in HostsEntry output description  https://review.openstack.org/25757822:12
jaosoriorbnemec: ^^22:13
jaosoriorthe follow up you requested22:13
bnemecjaosorior: Thanks, +2.22:14
*** trown is now known as trown|outttypeww22:14
jaosoriorbnemec: Hey, I uploaded a CR that adds makes HAProxy add X-Forwarded-Proto for requests coming for keystone (Still gotta make keystone be configured to receive it)22:17
jaosoriorbnemec: When you were testing TLS, do you remember what other services needed that?22:18
bnemecjaosorior: Heat for sure, maybe Nova too.22:18
jaosoriorsome of them can do with the configuration they have in the keystone endpoint list22:18
jaosoriorbnemec: This is the CR I'm talking about https://review.openstack.org/#/c/254771/ I'll upload another one soon making keystone actually use that22:19
jaosoriorbnemec: Alright, just wanted to make sure. I'll check those services out22:19
shardySimple cosmetic review that's passed CI if anyone has a moment:22:28
shardyhttps://review.openstack.org/#/c/255785/122:28
shardybnemec: hey, do you have a moment to discuss https://review.openstack.org/#/c/250059/ ?22:30
shardyIt was about creating all the per-role flavors by default, when folks might not always need them22:30
shardyContext is https://review.openstack.org/#/c/254667/ which potentially enables predictable placement by tagging specific nodes as e.g "controller-0"22:31
shardyif you do that, it's an alternative to the per-role flavors which would no longer be required22:32
bnemecshardy: Having not seen this before, I think I'm okay with it replacing the per-role flavors.  However, I think we'd need to update AHC to be aware of the new method of assigning profiles.22:34
bnemecshardy: Although it does have some drawbacks for users who don't necessarily want to assign every node.22:35
shardybnemec: Yeah, it's definitely not something everyone will want, but it's been a frequent RFE for some deployments where they desire control over placement22:36
bnemecFor example, if you have 100 compute nodes and one of them dies, using this method we'd need some way to make sure the new compute node gets assigned the right compute-X tag.22:36
shardybnemec: Yeah, this actually plays pretty well with the template RemovalPolicies, in that node indexes are never reused22:36
*** rasca has quit IRC22:37
shardySo if a node fails, it's permanently excluded and you have to choose a new unique ID22:37
bnemecshardy: Ah, interesting.22:37
shardybnemec: not quite sure how this would interface with AHC?  I was thinking more just support nodes.json tagging per-node on registration with ironic22:37
bnemecshardy: So my only goal here is to prevent people from having to type in a big long flavor-create command that they're liable to typo.  If this can replace that then +2 from me.22:38
shardybnemec: I guess it doesn't replace that, but it is an alternative22:38
bnemecshardy: I think you'd have to teach AHC to tag nodes as node:profile-X instead of just profile:whatever.22:38
*** david-lyle has joined #tripleo22:39
shardybnemec: ah, I was thinking we'd still allow that, but optionally also allow folks to feed in per-node mappings at node registration time22:39
shardyso perhaps we just land your patches as-is22:39
shardyI'm not sure how you'd define per-node criteria when matching, as I assume some at least will be the same22:39
shardyunless you match on DMI id's or something, which seems more work than just passing a tag via nodes.json22:40
bnemecshardy: Yeah, I'm also okay with them coexisting.  My thought was that it's better to have unused flavors than have people making mistakes creating only the flavors they need.22:40
shardybnemec: Ok, makes sense, I guess that's what you get with most out-of-the-box nova setups anyway22:41
shardybnemec: thanks, just wanted to discuss it before +Aing your patches related to this22:41
bnemecshardy: Yep, exactly.22:41
shardysounds like we can do both, at least for now22:41
bnemecshardy: Roger, appreciate the reviews.22:41
*** weshay has quit IRC22:47
openstackgerritMerged openstack/tripleo-heat-templates: Fix wrong keypair parameter description  https://review.openstack.org/25578522:48
openstackgerritSteven Hardy proposed openstack-infra/tripleo-ci: Don't pass duplicate projects to tripleo.sh --delorean-build  https://review.openstack.org/24981922:54
openstackgerritBen Nemec proposed openstack/tripleo-heat-templates: Fix yaml validation errors in multiple-nics  https://review.openstack.org/25268322:54
openstackgerritBen Nemec proposed openstack/tripleo-heat-templates: Add Management Network For System Administration.  https://review.openstack.org/22605722:54
* bnemec looks at his pending review list22:59
* bnemec despairs22:59
*** thrash is now known as thrash|g0ne23:04
*** chlong has joined #tripleo23:05
openstackgerritGiulio Fidente proposed openstack/python-tripleoclient: Forcibly clear parameters (now passed as parameter_defaults)  https://review.openstack.org/25667023:08
*** olap has quit IRC23:12
*** olap has joined #tripleo23:13
*** gfidente has quit IRC23:17
*** shardy has quit IRC23:21
openstackgerritJuan Antonio Osorio Robles proposed openstack/python-tripleoclient: Use renamed overcloud.yaml template  https://review.openstack.org/25335423:22
*** rlandy has quit IRC23:28
*** jcoufal has joined #tripleo23:36
*** sthillma has quit IRC23:44
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates: Allow to enable Monitoring agent on Overcloud Controller(s)  https://review.openstack.org/25478823:47
*** jaosorior has quit IRC23:57
*** dmacpher has quit IRC23:59

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