*** dmacpher has quit IRC | 00:21 | |
*** dmacpher has joined #tripleo | 00:22 | |
*** jcoufal has quit IRC | 00:41 | |
*** chlong has joined #tripleo | 00:48 | |
*** bvandenh has joined #tripleo | 02:02 | |
*** bvandenh has quit IRC | 02:21 | |
*** tserong has joined #tripleo | 03:11 | |
*** tserong has joined #tripleo | 03:11 | |
*** mgagne has quit IRC | 04:36 | |
*** larstobi has quit IRC | 04:36 | |
*** dsneddon_biab has quit IRC | 04:37 | |
*** mgagne has joined #tripleo | 04:37 | |
*** mgagne is now known as Guest73233 | 04:37 | |
*** larstobi has joined #tripleo | 04:38 | |
*** dsneddon_biab has joined #tripleo | 04:38 | |
*** masco has joined #tripleo | 05:11 | |
*** devvesa has joined #tripleo | 05:28 | |
*** jrist has quit IRC | 05:36 | |
*** jrist has joined #tripleo | 05:38 | |
*** jrist has joined #tripleo | 05:38 | |
*** yuanying has joined #tripleo | 06:11 | |
*** shivrao has joined #tripleo | 06:13 | |
*** shivrao_ has joined #tripleo | 06:16 | |
*** shivrao has quit IRC | 06:18 | |
*** shivrao_ has quit IRC | 06:19 | |
*** shivrao has joined #tripleo | 06:19 | |
*** shivrao has quit IRC | 06:26 | |
*** shivrao has joined #tripleo | 06:39 | |
*** shivrao has quit IRC | 06:43 | |
*** shivrao has joined #tripleo | 06:45 | |
*** olap has joined #tripleo | 06:52 | |
*** rcernin has joined #tripleo | 07:01 | |
*** devvesa has quit IRC | 07:04 | |
*** openstackstatus has joined #tripleo | 07:07 | |
*** ChanServ sets mode: +v openstackstatus | 07:07 | |
*** openstackgerrit has joined #tripleo | 07:10 | |
*** rasca has joined #tripleo | 07:10 | |
*** rasca has quit IRC | 07:11 | |
openstackgerrit | Ian Wienand proposed openstack/diskimage-builder: yum-minimal: Don't create eth0 & eth1 https://review.openstack.org/257172 | 07:16 |
---|---|---|
openstackgerrit | Ian Wienand proposed openstack/diskimage-builder: yum-minimal: leave behind dummy /etc/resolv.conf https://review.openstack.org/257173 | 07:16 |
openstackgerrit | sonu proposed openstack/python-tripleoclient: Removes MANIFEST.in as it is not needed explicitely by PBR https://review.openstack.org/257178 | 07:25 |
*** ukalifon has joined #tripleo | 07:45 | |
openstackgerrit | Steve Martinelli proposed openstack/tuskar: use keystonemiddleware instead of keystoneclient.middleware https://review.openstack.org/257186 | 07:45 |
*** chlong has quit IRC | 07:46 | |
*** jprovazn has joined #tripleo | 08:00 | |
openstackgerrit | Steve Martinelli proposed openstack/tuskar: use keystonemiddleware instead of keystoneclient.middleware https://review.openstack.org/257186 | 08:00 |
*** jcoufal has joined #tripleo | 08:01 | |
*** olap has quit IRC | 08:06 | |
*** gchamoul- is now known as gchamoul | 08:18 | |
*** rasca has joined #tripleo | 08:21 | |
*** liverpooler has joined #tripleo | 08:21 | |
*** pino|work has joined #tripleo | 08:24 | |
*** devvesa has joined #tripleo | 08:24 | |
*** shardy has joined #tripleo | 08:25 | |
*** aufi has joined #tripleo | 08:25 | |
openstackgerrit | Merged openstack/python-tripleoclient: Set NeutronMetadataProxySharedSecret https://review.openstack.org/254857 | 08:25 |
*** paramite has joined #tripleo | 08:35 | |
*** shivrao has quit IRC | 08:37 | |
devvesa | Hi Anyone with more experience than me on Puppet can tell me why the gates fail now on this patch: | 08:41 |
devvesa | https://review.openstack.org/#/c/251319/ | 08:41 |
openstackgerrit | Marek Aufart proposed openstack/tripleo-common: Add environment capabilities https://review.openstack.org/232534 | 08:41 |
*** paramite has quit IRC | 08:43 | |
*** ifarkas has joined #tripleo | 08:45 | |
*** shardy has quit IRC | 08:52 | |
*** gfidente has joined #tripleo | 08:53 | |
*** gfidente has quit IRC | 08:53 | |
*** gfidente has joined #tripleo | 08:53 | |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: WIP: Allow predictable IPs for Controllers on the ctlplane https://review.openstack.org/256003 | 08:54 |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: [WIP] Add the Plan deploy endpoint to start a Heat deploy https://review.openstack.org/256384 | 08:55 |
gfidente | marios, christmas gift? https://review.openstack.org/#/c/222497/ | 08:58 |
marios | gfidente: bah! humbug! | 09:00 |
gfidente | I had to rebase it, but it's same you voted on before! | 09:01 |
gfidente | and I fixed what you asked about oac | 09:01 |
marios | gfidente: 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 |
gfidente | auch, haven't got there yet I think myself | 09:02 |
*** mcornea has joined #tripleo | 09:06 | |
*** jistr has joined #tripleo | 09:12 | |
*** nico_auvv has joined #tripleo | 09:13 | |
*** mbound_ has joined #tripleo | 09:14 | |
*** shardy has joined #tripleo | 09:19 | |
*** devvesa has quit IRC | 09:23 | |
*** devvesa has joined #tripleo | 09:23 | |
*** akrivoka has joined #tripleo | 09:24 | |
*** jrist has quit IRC | 09:26 | |
*** leanderthal has joined #tripleo | 09:26 | |
*** jrist has joined #tripleo | 09:30 | |
*** jrist has joined #tripleo | 09:30 | |
*** morazi has joined #tripleo | 09:30 | |
*** yamahata has quit IRC | 09:32 | |
*** athomas has joined #tripleo | 09:37 | |
*** leanderthal has quit IRC | 09:42 | |
*** jrist has quit IRC | 09:46 | |
*** jrist has joined #tripleo | 09:47 | |
*** jrist has joined #tripleo | 09:47 | |
*** olap has joined #tripleo | 09:50 | |
*** bandini has quit IRC | 09:56 | |
*** bandini has joined #tripleo | 09:56 | |
*** mkovacik has joined #tripleo | 09:56 | |
*** jrist has quit IRC | 09:57 | |
*** jrist has joined #tripleo | 09:59 | |
*** jrist has joined #tripleo | 09:59 | |
*** dtantsur|afk is now known as dtantsur | 10:00 | |
*** leanderthal has joined #tripleo | 10:01 | |
nico_auvv | hello, 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/1 | 10:05 |
*** jtomasek has joined #tripleo | 10:06 | |
*** ifarkas has quit IRC | 10:06 | |
*** paramite has joined #tripleo | 10:16 | |
*** electrofelix has joined #tripleo | 10:19 | |
*** leanderthal has quit IRC | 10:21 | |
*** hewbrocca has joined #tripleo | 10:21 | |
*** rcernin has quit IRC | 10:25 | |
devvesa | Hi jisrt: Do you have a second to look at this review https://review.openstack.org/#/c/251319/? | 10:30 |
devvesa | I have no idea why the puppet gates fail | 10:30 |
*** pleia2 has quit IRC | 10:32 | |
*** nico_auvv has quit IRC | 10:41 | |
*** nico_auvv has joined #tripleo | 10:41 | |
openstackgerrit | Jaume Devesa proposed openstack/tripleo-heat-templates: MidoNet heat templates https://review.openstack.org/238998 | 10:43 |
*** rleander has joined #tripleo | 10:43 | |
*** tosky has joined #tripleo | 10:45 | |
shardy | nico_auvv: done, thanks! | 10:47 |
openstackgerrit | Merged openstack/tripleo-puppet-elements: Install puppet-timezone via puppet-modules element https://review.openstack.org/251920 | 10:48 |
*** rleander has quit IRC | 10:49 | |
hewbrocca | devvesa: ITYM jistr ^^^ | 10:49 |
devvesa | hewbrocca: Yes, I do | 10:51 |
devvesa | :) | 10:52 |
*** mgould has joined #tripleo | 10:56 | |
nico_auvv | thx shardy ! | 10:56 |
*** leanderthal has joined #tripleo | 10:57 | |
* jistr multitasking :) | 10:58 | |
jistr | hi devvesa | 10:58 |
devvesa | Hi jistr | 10:59 |
jistr | the 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 |
jistr | these jobs are maintained by puppet folks i think | 10:59 |
jistr | so 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 |
devvesa | So that can be caused by a change on the job definition in project-config | 11:00 |
jistr | yes, or a new version of some rubygem | 11:00 |
devvesa | Ok. Thanks! I'll ask them | 11:00 |
*** rleander has joined #tripleo | 11:04 | |
*** rasca has quit IRC | 11:11 | |
*** ifarkas has joined #tripleo | 11:35 | |
*** chlong has joined #tripleo | 11:39 | |
*** leanderthal has quit IRC | 11:43 | |
*** derekh has joined #tripleo | 11:47 | |
openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Adding psych on Gemfile explicitly https://review.openstack.org/257294 | 11:53 |
*** slagle has quit IRC | 11:53 | |
devvesa | jistr: I found this: https://github.com/bundler/bundler/issues/2068 | 11:54 |
devvesa | So I pushed the review ^^ | 11:54 |
devvesa | The bug is quite TL;DR :D But I think I got the idea | 11:54 |
*** slagle has joined #tripleo | 11:54 | |
openstackgerrit | Merged openstack/python-tuskarclient: Update README.md for repo project https://review.openstack.org/214427 | 11:54 |
devvesa | It is old, but let the jenkins run if it solves the issue | 11:55 |
devvesa | jistr, shardy. The patch fixes the issue | 12:03 |
devvesa | Can you review it, please? It is quick | 12:03 |
*** hewbrocca has quit IRC | 12:04 | |
openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency https://review.openstack.org/251319 | 12:07 |
jistr | thanks devvesa, i +2'd the Gemfile change | 12:23 |
*** rasca has joined #tripleo | 12:29 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-tripleoclient: Updated from global requirements https://review.openstack.org/254463 | 12:35 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-tuskarclient: Updated from global requirements https://review.openstack.org/232904 | 12:35 |
*** dprince has joined #tripleo | 12:38 | |
*** trown|outttypeww is now known as trown | 12:38 | |
openstackgerrit | Merged openstack/tripleo-common: Set a default timeout for 240 minutes for Heat Updates https://review.openstack.org/254954 | 12:39 |
*** weshay_xchat has joined #tripleo | 12:48 | |
*** weshay_xchat is now known as weshay | 12:48 | |
*** thrash|wknd is now known as thrash | 12:50 | |
*** mburned has joined #tripleo | 12:56 | |
dtantsur | bnemec, dprince, hi, any plans to backport https://review.openstack.org/#/c/247088/ to liberty? | 13:00 |
dprince | dtantsur: I'd be fine with that | 13:01 |
dtantsur | if nobody is working on it already, I'll do it | 13:01 |
*** jdob has joined #tripleo | 13:03 | |
*** rebrego has joined #tripleo | 13:05 | |
*** regebro is now known as Guest93423 | 13:05 | |
*** rebrego is now known as regebro | 13:05 | |
trown | dtantsur: I am not sure we would want to change liberty defaults... | 13:05 |
dtantsur | trown, well, we're using the deprecated ramdisk | 13:06 |
dtantsur | and as downstream, we're going to switch to IPA anyway | 13:06 |
trown | dtantsur: btw something is broken with trunk IPA... working on gathering more info | 13:06 |
trown | dtantsur: 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' pin | 13:08 |
dtantsur | not sure I understood what is broken.. | 13:08 |
trown | dtantsur: for tripleoci, I would say broken is not being able to land anything because CI cant pass | 13:09 |
dtantsur | yeah, but I don't understand how IPA is related right now.. is something broken with it? | 13:10 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-tripleoclient: Deprecate bash ramdisks https://review.openstack.org/257338 | 13:10 |
dtantsur | I've put a patch ^^ do whatever you feel like with it :) | 13:10 |
trown | dtantsur: something with it, DIB, or tripleoclient... still not havent totally narrowed down what | 13:10 |
dtantsur | trown, how does it manifest? | 13:10 |
trown | dtantsur: boot loop on deploy | 13:10 |
trown | dtantsur: ramdisk boots... then just shuts off over and over | 13:11 |
dtantsur | trown, at what stage does it gets shut off? | 13:11 |
trown | dtantsur: images here have the issue | 13:11 |
trown | https://ci.centos.org/artifacts/rdo/images/mitaka/delorean/testing/ | 13:11 |
dprince | EmilienM: morning, hey I did some manual testing on this and posted some feedback https://review.openstack.org/#/c/234704/11 | 13:12 |
trown | dtantsur: 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 info | 13:13 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-tripleoclient: Allow assigning profiles based on XXX_profile capabilities https://review.openstack.org/250405 | 13:14 |
dtantsur | quick question folks, does 'overcloud deploy' have something like --dry-run mode to only run validations? | 13:15 |
*** jayg|g0n3 is now known as jayg | 13:16 | |
EmilienM | dprince: ok I will | 13:16 |
dprince | EmilienM: 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#n70 | 13:17 |
dprince | EmilienM: I actually think this fix would let us re-land the overcloud patch too. | 13:17 |
dprince | EmilienM: I'm looking into that... | 13:17 |
EmilienM | dprince: in meeting now. | 13:17 |
openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Enable setting Neutron QoS feature https://review.openstack.org/247056 | 13:20 |
trown | derekh: that swift replica issue also happens in THT... | 13:20 |
dprince | trown: I'd like to send a mail to the list about the replicas thing | 13:20 |
dprince | trown: https://review.openstack.org/#/c/256631/ | 13:20 |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Add the missing plan name to the PlanAlreadyExistsError https://review.openstack.org/257343 | 13:20 |
dprince | trown: for the undercloud we can simply set to 1 (for now) | 13:20 |
EmilienM | dprince: I'll look in a few | 13:21 |
dprince | trown: for the overcloud we are now used for SAIO | 13:21 |
trown | dprince: SAIO? | 13:21 |
*** liverpooler has quit IRC | 13:21 | |
dprince | trown: Single All In One | 13:21 |
d0ugal | dtantsur: I don't think so, that would be a good idea tho' | 13:21 |
dprince | trown: like if you deploy an overcloud with just a single swift storage node | 13:22 |
dtantsur | ok, I'll try to do it | 13:22 |
dprince | trown: the new code wants replicas to be <= devices. And we hardcode it to 3 | 13:22 |
*** tzumainn has joined #tripleo | 13:23 | |
trown | dprince: right... derekh has a similar patch for undercloud btw... https://review.openstack.org/255995 | 13:23 |
trown | dprince: though yours has a bug... so I think you win :) | 13:23 |
dprince | trown: oh, I see | 13:24 |
dprince | trown: yeah, I missed the patch w/ no bug | 13:24 |
openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Pacemaker maintenance mode for the duration of Puppet run on update https://review.openstack.org/245093 | 13:24 |
trown | dprince: we see the same thing in a default `overcloud deploy --templates` as well... I imagine we would not see it on HA though | 13:25 |
dprince | trown: right, and before we make a change there I'd like to ask Clay | 13:25 |
dprince | trown: on openstack-dev how to best handle this | 13:25 |
dprince | trown: because it seems to me this is a breaking sort of change | 13:26 |
*** rhallisey has joined #tripleo | 13:26 | |
trown | dprince: fair enough... trunk is broken until that is resolved though there are a couple other issues blocking trunk CI too | 13:26 |
dprince | trown: I'm wondering if in the end we might not have to make any changes for this... because it is a swift bug | 13:27 |
trown | dprince: 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 |
trown | and now the deprecated was removed | 13:28 |
dprince | trown: if that is the case what would you set replicas to for our overcloud (now) | 13:29 |
trown | (# of controllers)? | 13:29 |
dprince | trown: I think we'd go for a max of 3 perhaps | 13:30 |
dprince | trown: so #controllers, until it hits 3 and then leave it at that | 13:30 |
trown | so min(3, # of controllers) | 13:31 |
dprince | basically | 13:31 |
trown | dprince: cool, it would be good to confirm expected swift behavior on ML | 13:31 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-tripleoclient: Add --dry-run argument to overcloud deploy command https://review.openstack.org/257348 | 13:31 |
dtantsur | d0ugal, ^^ | 13:31 |
trown | my perception is just based on conversation with derekh last week | 13:32 |
dprince | trown: yeah, I am just a little fuzzy on what exactly happened. That patch was rather large | 13:32 |
d0ugal | dtantsur: thanks | 13:34 |
dtantsur | d0ugal, btw is there a particular reason a lot of errors in this command are logged with tracebacks? does not look too user friendly IMO | 13:37 |
d0ugal | dtantsur: Not that I remember. | 13:37 |
d0ugal | dtantsur: but I don't think any of the logging output is displayed by default, is it? | 13:37 |
dtantsur | d0ugal, log.exception is displayed, at least for me | 13:38 |
d0ugal | hmm, actually, that makes sense that it would be | 13:38 |
d0ugal | We should probably improve that | 13:38 |
d0ugal | cc: regebro ^ | 13:38 |
dtantsur | d0ugal, 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 #tripleo | 13:42 | |
d0ugal | dtantsur: Yeah, that's just lazy on our part I think. | 13:43 |
d0ugal | .error() would make more sense | 13:44 |
dtantsur | ++ | 13:44 |
*** jaosorior has joined #tripleo | 13:47 | |
regebro | dtantsur, d0ugal I like that it failes with tracebacks. | 13:47 |
dtantsur | regebro, it should not be shown to users | 13:47 |
d0ugal | regebro: Why? I can't see when an attributeerror would be useful | 13:48 |
dtantsur | with --debug - yes | 13:48 |
dtantsur | but not by default | 13:48 |
regebro | It should definitely not show a traceback unless it's an actual failure, ie, the command stops. | 13:48 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-tripleoclient: Add --dry-run argument to overcloud deploy command https://review.openstack.org/257348 | 13:49 |
dtantsur | regebro, right, and here we have a user failure, not our failure | 13:49 |
regebro | dtantsur: 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 |
regebro | I 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 IRC | 13:53 | |
*** mcornea has joined #tripleo | 13:54 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-tripleoclient: Allow assigning profiles based on XXX_profile capabilities https://review.openstack.org/250405 | 13:55 |
derekh | dprince: trown sorry was at lunch | 13:57 |
derekh | <dprince> trown: yeah, I am just a little fuzzy on what exactly happened. That patch was rather large | 13:57 |
derekh | dprince: RE ^, swift used to warn about having less devices then replica's, it now errors instead | 13:57 |
dprince | derekh: okay, so maybe we have no choice here | 13:58 |
derekh | trown: you mentioned another bug on current did you? | 14:02 |
openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Adding psych on Gemfile explicitly https://review.openstack.org/257294 | 14:02 |
*** fabbione has quit IRC | 14:03 | |
dprince | derekh: anyways, sorry I missed your patch. +2 although note that instack had a bunch of useless tripleo::ringbuilder stuff in it | 14:03 |
derekh | dprince: 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 patch | 14:05 |
dprince | derekh: don't care who's patch lands, but it might be worth updating your patch to address the bug though | 14:05 |
*** fabbione has joined #tripleo | 14:05 | |
derekh | dprince: either way, doesn't make much difference, will +2 yours now | 14:05 |
dprince | derekh: then I'll rebase mine to just remove the cruft | 14:05 |
derekh | dprince: ok, will do that so | 14:05 |
*** pblaho has joined #tripleo | 14:06 | |
trown | derekh: ya there are two more... but I need to actually file bugs for them... probably wont get to that until late afternoon today | 14:07 |
derekh | trown: ok | 14:07 |
trown | tripleoclient deploy command does not work with latest openstackclient, and there is some bug in the IPA disk we build | 14:07 |
openstackgerrit | Derek Higgins proposed openstack/instack-undercloud: Change undercloud replica count to 1 https://review.openstack.org/255995 | 14:07 |
derekh | dprince: ^ | 14:08 |
derekh | Also 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/2 | 14:09 |
dprince | trown: 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 patches | 14:09 |
*** paramite is now known as paramite|afk | 14:11 | |
trown | dprince: cool will do | 14:12 |
openstackgerrit | Dan Prince proposed openstack/instack-undercloud: Remove tripleo::ringbuilder cruft https://review.openstack.org/256631 | 14:12 |
dprince | derekh: ^^ | 14:12 |
*** paramite|afk is now known as paramite | 14:12 | |
trown | dprince: I have an etherpad... it is pretty sparse but I will be filling in bugs/patches as I file them | 14:13 |
trown | dprince: https://etherpad.openstack.org/p/delorean_master_current_issues | 14:13 |
openstackgerrit | Lennart Regebro proposed openstack/python-tripleoclient: Make the undercloud install test discoverable https://review.openstack.org/256410 | 14:14 |
jaosorior | trown: Do you have a launchpad id for item number 2 in https://etherpad.openstack.org/p/delorean_master_current_issues ? | 14:14 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-tripleoclient: Allow assigning profiles based on XXX_profile capabilities https://review.openstack.org/250405 | 14:14 |
trown | jaosorior: I do not, on my list for today to file it | 14:15 |
openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency https://review.openstack.org/251319 | 14:15 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/python-tripleoclient: Add dynamic-login element to the AGENT_IMAGE_ELEMENT list https://review.openstack.org/257369 | 14:15 |
trown | jaosorior: it is because of the switch to the openstack sdk | 14:15 |
*** rlandy has joined #tripleo | 14:16 | |
jaosorior | trown: On which API of the openstack sdk? (Sorry, lacking context here :/ ) | 14:16 |
trown | jaosorior, 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 |
jaosorior | trown: Seems legit | 14:18 |
*** jcoufal_ has joined #tripleo | 14:19 | |
*** jcoufal has quit IRC | 14:22 | |
*** rbrady has joined #tripleo | 14:22 | |
*** regebro has quit IRC | 14:26 | |
openstackgerrit | Merged openstack/puppet-tripleo: Adding psych on Gemfile explicitly https://review.openstack.org/257294 | 14:26 |
*** lblanchard has joined #tripleo | 14:26 | |
shardy | Hey folks, couple of easy +A's if anyone has a moment: | 14:27 |
shardy | https://review.openstack.org/#/c/255407/ | 14:27 |
shardy | https://review.openstack.org/#/c/254667/ | 14:27 |
jaosorior | shardy: Wouldn't https://review.openstack.org/#/c/255407/ need https://review.openstack.org/#/c/253354/ ?? | 14:28 |
jaosorior | shardy: Ah, nevermind, I now saw that it was the one without the -without-mergepy suffix | 14:29 |
* jaosorior is extremely jetlagged | 14:29 | |
shardy | jaosorior: Yeah, it's just removing the old now-unused registry | 14:30 |
jaosorior | shardy: Yeah, still need coffee haha. Greetings from Mexico by the way. | 14:30 |
shardy | jaosorior: there was some discussion re backwards compatibility and tripleoclient recently | 14:30 |
*** NobodyCa1 is now known as NobodyCam | 14:30 | |
shardy | it 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 overclouds | 14:30 |
*** regebro has joined #tripleo | 14:31 | |
jaosorior | shardy: well, I could modify that CR to open the overcloud.yaml and then if that fails, fall back to the without-mergepy naming | 14:31 |
*** rpothier has joined #tripleo | 14:31 | |
shardy | jaosorior: Yeah, if it's not too much work that'd be cool | 14:31 |
jaosorior | shardy: No problem | 14:32 |
*** lucasagomes is now known as lucas-hungry | 14:34 | |
*** trown is now known as trown|outttypeww | 14:40 | |
openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Enable setting Neutron QoS feature https://review.openstack.org/247056 | 14:46 |
*** masco has quit IRC | 14:51 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-tripleoclient: Allow assigning profiles based on XXX_profile capabilities https://review.openstack.org/250405 | 14:53 |
openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Adding psych on Gemfile explicitly https://review.openstack.org/257420 | 14:56 |
*** akrivoka has quit IRC | 14:58 | |
*** jprovazn has quit IRC | 14:59 | |
*** akrivoka has joined #tripleo | 15:00 | |
*** eggmaster has joined #tripleo | 15:05 | |
*** pradk has joined #tripleo | 15:08 | |
openstackgerrit | Merged openstack/tripleo-heat-templates: Remove deprecated overcloud-resource-registry.yaml https://review.openstack.org/255407 | 15:14 |
openstackgerrit | Merged openstack/tripleo-heat-templates: Enable per-role SchedulerHints https://review.openstack.org/254667 | 15:20 |
openstackgerrit | Merged openstack/tripleo-heat-templates: Add pcmk constraints against haproxy-clone only if applicable https://review.openstack.org/255301 | 15:22 |
dprince | trown|outttypeww: https://bugs.launchpad.net/tripleo/+bug/1525957 | 15:33 |
openstack | Launchpad bug 1525957 in tripleo "nova neutronclient fails due to deprecated auth options" [Critical,New] | 15:33 |
dprince | trown|outttypeww: added to the etherpad | 15:34 |
*** lucas-hungry is now known as lucasagomes | 15:37 | |
*** rpothier has left #tripleo | 15:48 | |
*** mcornea has quit IRC | 15:50 | |
*** masco has joined #tripleo | 15:51 | |
*** rleander is now known as leanderthal|afk | 15:55 | |
*** athomas has quit IRC | 15:56 | |
*** 21WAAGP61 has quit IRC | 15:57 | |
*** aufi has quit IRC | 15:57 | |
*** andreaf_ has joined #tripleo | 15:57 | |
*** mkovacik has quit IRC | 15:57 | |
*** dtantsur is now known as dtantsur|afk | 15:59 | |
*** sthillma has joined #tripleo | 16:00 | |
*** akrivoka has quit IRC | 16:00 | |
openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Manage keystone related configuration via Puppet https://review.openstack.org/234704 | 16:01 |
openstackgerrit | Ben Swartzlander proposed openstack/tripleo-heat-templates: Enable Manila integration https://review.openstack.org/188137 | 16:02 |
*** jistr has quit IRC | 16:03 | |
openstackgerrit | Ben Swartzlander proposed openstack/tripleo-heat-templates: Add NetApp integration to Manila https://review.openstack.org/188138 | 16:03 |
*** olap has quit IRC | 16:04 | |
openstackgerrit | Ben Swartzlander proposed openstack/tripleo-heat-templates: Add Gluster integration to Manila https://review.openstack.org/230936 | 16:04 |
*** Goneri has quit IRC | 16:06 | |
*** morazi has quit IRC | 16:06 | |
*** ifarkas has quit IRC | 16:06 | |
*** Goneri has joined #tripleo | 16:09 | |
*** shivrao has joined #tripleo | 16:14 | |
*** Goneri has quit IRC | 16:16 | |
*** shivrao_ has joined #tripleo | 16:17 | |
*** shivrao has quit IRC | 16:18 | |
*** shivrao_ is now known as shivrao | 16:18 | |
*** Goneri has joined #tripleo | 16:19 | |
*** andreaf has quit IRC | 16:21 | |
*** andreaf_ is now known as andreaf | 16:21 | |
*** jtomasek has quit IRC | 16:22 | |
*** stendulker has joined #tripleo | 16:24 | |
*** gfidente has quit IRC | 16:37 | |
*** paramite has quit IRC | 16:39 | |
*** Goneri has quit IRC | 16:42 | |
*** gfidente has joined #tripleo | 16:43 | |
*** gfidente has quit IRC | 16:43 | |
*** gfidente has joined #tripleo | 16:43 | |
*** sthillma has quit IRC | 16:44 | |
*** Goneri has joined #tripleo | 16:45 | |
*** pleia2 has joined #tripleo | 16:48 | |
openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Fixes create_plan behavior in PlanManager https://review.openstack.org/257481 | 16:50 |
*** shivrao has quit IRC | 16:50 | |
*** cwolferh has joined #tripleo | 16:52 | |
*** absubram has joined #tripleo | 16:52 | |
*** jtomasek has joined #tripleo | 17:18 | |
*** mbound_ has quit IRC | 17:20 | |
*** openstackstatus has quit IRC | 17:20 | |
*** rwsu has joined #tripleo | 17:20 | |
*** openstackstatus has joined #tripleo | 17:23 | |
*** ChanServ sets mode: +v openstackstatus | 17:23 | |
*** devvesa has quit IRC | 17:31 | |
*** jcoufal_ has quit IRC | 17:32 | |
*** dsneddon_biab is now known as dsneddon | 17:40 | |
*** olap has joined #tripleo | 17:49 | |
*** penick has joined #tripleo | 17:51 | |
*** penick has quit IRC | 17:55 | |
*** pradk has quit IRC | 17:57 | |
*** ukalifon has quit IRC | 17:59 | |
*** yamahata has joined #tripleo | 18:02 | |
*** derekh has quit IRC | 18:04 | |
*** rhallisey has quit IRC | 18:06 | |
*** stendulker has quit IRC | 18:07 | |
*** rhallisey has joined #tripleo | 18:12 | |
*** rhallisey_ has joined #tripleo | 18:13 | |
*** rhallisey_ has quit IRC | 18:13 | |
*** rhallisey_ has joined #tripleo | 18:13 | |
*** pradk has joined #tripleo | 18:13 | |
*** rhallisey_ has quit IRC | 18:15 | |
*** nico_auvv has quit IRC | 18:15 | |
*** trown|outttypeww is now known as trown | 18:20 | |
*** shivrao has joined #tripleo | 18:22 | |
*** mgould has quit IRC | 18:23 | |
openstackgerrit | Giulio Fidente proposed openstack/python-tripleoclient: We do not need to pass a NeutronControlPlaneID https://review.openstack.org/257520 | 18:29 |
dprince | slagle: hi, so I spent some time manually testing this one. Found an issue and fixed it... https://review.openstack.org/#/c/234704/ | 18:33 |
thrash | trown: 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 |
thrash | trown: dprince so I'm thinking this: https://review.gerrithub.io/#/c/255380/ | 18:34 |
thrash | wdyt? | 18:34 |
trown | thrash: that is a built in thing? | 18:35 |
thrash | trown: yep | 18:35 |
dprince | thrash: I'm probably okay w/ that as a temporary fix. | 18:35 |
trown | thrash: cool... learned something new for rpm packaging | 18:35 |
dprince | thrash: perhaps we remove it quickly once things get past this blocker | 18:35 |
thrash | dprince: yes. It should allow the CI to run upstream. | 18:35 |
dprince | slagle: see also my comment here https://review.openstack.org/#/c/222227/ | 18:36 |
thrash | dprince: 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 script | 18:36 |
dprince | slagle: if you rebase that patch on the keystone v3 one... and remove the keystone stuff from your patch I think it'll all be working | 18:36 |
dprince | slagle: by my tests it was anyway | 18:36 |
*** masco has quit IRC | 18:37 | |
slagle | dprince: ok, thx. i saw your review on 222227, and i'll update it accordingly. it's a better solution | 18:39 |
*** david-lyle_ has joined #tripleo | 18:42 | |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Manage keystone initialization directly in t-h-t manifests https://review.openstack.org/244162 | 18:45 |
*** david-lyle has quit IRC | 18:45 | |
gfidente | thrash, do you remember why we ever did https://review.openstack.org/257520 ? | 18:50 |
dprince | trown: I added a puppet-nova fix here as well https://etherpad.openstack.org/p/delorean_master_current_issues | 18:52 |
dprince | trown: just hit that one today... | 18:52 |
trown | dprince: nice... I think that is a new one... fixing trunk/current long enough to promote something is hard | 18:53 |
trown | dprince: we are finding issues at a much higher rate than resolving them | 18:54 |
*** tosky has quit IRC | 18:55 | |
dprince | trown: yeah, I feel like we are in a leaky boat. And our buckets aren't big enough or something. We need a bilge pump | 18:55 |
dprince | trown: anyways, lets bring these issues to the table in the meeting tomorrow | 18:55 |
dprince | trown: anyone working on the overcloud swift issue yet? | 18:56 |
trown | dprince: sounds good, I am working on filing bugs for the IPA and tripleoclient issues | 18:56 |
dprince | trown: I can probably take that... | 18:56 |
trown | dprince: dont think so | 18:56 |
dprince | trown: okay, I will notify derek so he isn't working on the same | 18:56 |
dprince | shardy, slagle, jdob, bnemec, gfidente: one of you might send this one? https://review.openstack.org/#/c/255995/ | 18:57 |
gfidente | dprince, ack, thanks | 18:58 |
dprince | gfidente: you might go ahead and hit this one too https://review.openstack.org/#/c/256631/ | 19:02 |
dprince | gfidente: a related cleanup of the cruft... | 19:02 |
openstackgerrit | Jason Guiditta proposed openstack/tripleo-puppet-elements: Add monitoring dependencies https://review.openstack.org/252051 | 19:02 |
*** leanderthal|afk has quit IRC | 19:05 | |
*** sparkyco_ has joined #tripleo | 19:06 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/python-tripleoclient: Use renamed overcloud.yaml template https://review.openstack.org/253354 | 19:07 |
thrash | gfidente: it was probably done because that's what the old instack script did. | 19:09 |
*** thrash is now known as thrash|bbl | 19:12 | |
*** sthillma has joined #tripleo | 19:12 | |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Set swift replicas = min(device_count, replicas) https://review.openstack.org/257540 | 19:13 |
dprince | trown: ^^^ | 19:13 |
trown | dprince: sweet that was fast | 19:13 |
dprince | trown: yeah, lets see if CI likes it but I think it'll go | 19:14 |
trown | dprince: cool, after I get the IPA bug filed I can verify too | 19:15 |
dprince | trown: I added all of our Swift guys as well | 19:16 |
dprince | trown: anything else you'd like me to look into? | 19:16 |
*** yamahata has quit IRC | 19:17 | |
dprince | trown: I will keep watch on this etherpad for new reviews | 19:17 |
*** yamahata has joined #tripleo | 19:17 | |
trown | dprince: the tripleoclient one might be pretty major, but it is probabbly worth waiting until I file the bug | 19:17 |
dprince | trown: file it, and we'll see | 19:18 |
*** ukalifon1 has joined #tripleo | 19:20 | |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Align template defaults with the client https://review.openstack.org/257541 | 19:26 |
*** sparkyco_ has quit IRC | 19:28 | |
openstackgerrit | Giulio Fidente proposed openstack/python-tripleoclient: We do not need to pass a NeutronControlPlaneID https://review.openstack.org/257520 | 19:29 |
*** dprince has quit IRC | 19:32 | |
jaosorior | anybody has some spare cycles to check this one out? https://review.openstack.org/#/c/254771/ | 19:37 |
openstackgerrit | Giulio Fidente proposed openstack/python-tripleoclient: WIP: Forcibly clear parameters (now passed as parameter_defaults) https://review.openstack.org/256670 | 19:39 |
*** ukalifon1 has quit IRC | 19:39 | |
jaosorior | Also, 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,z | 19:39 |
*** sthillma has quit IRC | 19:41 | |
*** sthillma has joined #tripleo | 19:42 | |
*** ukalifon1 has joined #tripleo | 19:47 | |
*** lucasagomes is now known as lucas-dinner | 19:51 | |
*** ukalifon1 has quit IRC | 19:51 | |
openstackgerrit | John Trowbridge proposed openstack/python-tripleoclient: neutron client has been migrated to openstack SDK https://review.openstack.org/257550 | 19:53 |
*** dprince has joined #tripleo | 19:54 | |
*** electrofelix has quit IRC | 19:57 | |
openstackgerrit | Merged openstack/instack-undercloud: Change undercloud replica count to 1 https://review.openstack.org/255995 | 19:58 |
*** penick has joined #tripleo | 20:01 | |
*** eil397 has joined #tripleo | 20:02 | |
*** penick has quit IRC | 20:03 | |
*** mbound_ has joined #tripleo | 20:04 | |
*** whydidyoustealmy has quit IRC | 20:07 | |
*** shakamunyi has quit IRC | 20:07 | |
*** david-lyle_ has quit IRC | 20:10 | |
*** eil397 has quit IRC | 20:11 | |
*** eil397 has joined #tripleo | 20:12 | |
*** penick has joined #tripleo | 20:12 | |
*** david-lyle has joined #tripleo | 20:13 | |
*** rhallisey_ has joined #tripleo | 20:39 | |
*** rhallisey_ has quit IRC | 20:40 | |
*** rhallisey_ has joined #tripleo | 20:40 | |
*** rhallisey_ has quit IRC | 20:41 | |
*** thrash|bbl is now known as thrash | 20:47 | |
*** rhallisey has quit IRC | 20:52 | |
*** penick has quit IRC | 20:55 | |
*** penick has joined #tripleo | 20:58 | |
*** david-lyle has quit IRC | 21:01 | |
*** jayg is now known as jayg|g0n3 | 21:03 | |
openstackgerrit | John Trowbridge proposed openstack/python-tripleoclient: neutron client has been migrated to openstack SDK https://review.openstack.org/257550 | 21:06 |
*** david-lyle has joined #tripleo | 21:07 | |
openstackgerrit | Ben Swartzlander proposed openstack/tripleo-heat-templates: Enable Manila integration https://review.openstack.org/188137 | 21:09 |
*** david-lyle has quit IRC | 21:12 | |
openstackgerrit | Ben Swartzlander proposed openstack/tripleo-heat-templates: Enable Manila integration https://review.openstack.org/188137 | 21:20 |
*** dprince has quit IRC | 21:28 | |
*** lblanchard has quit IRC | 21:34 | |
*** rhallisey has joined #tripleo | 21:34 | |
*** absubram has quit IRC | 21:41 | |
*** chlong has quit IRC | 21:51 | |
*** pradk has quit IRC | 21:54 | |
*** jtomasek has quit IRC | 21:55 | |
openstackgerrit | Merged openstack/tripleo-heat-templates: Add output for host entries https://review.openstack.org/240171 | 22:05 |
*** julim has quit IRC | 22:08 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Fix typo in HostsEntry output description https://review.openstack.org/257578 | 22:12 |
jaosorior | bnemec: ^^ | 22:13 |
jaosorior | the follow up you requested | 22:13 |
bnemec | jaosorior: Thanks, +2. | 22:14 |
*** trown is now known as trown|outttypeww | 22:14 | |
jaosorior | bnemec: 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 |
jaosorior | bnemec: When you were testing TLS, do you remember what other services needed that? | 22:18 |
bnemec | jaosorior: Heat for sure, maybe Nova too. | 22:18 |
jaosorior | some of them can do with the configuration they have in the keystone endpoint list | 22:18 |
jaosorior | bnemec: This is the CR I'm talking about https://review.openstack.org/#/c/254771/ I'll upload another one soon making keystone actually use that | 22:19 |
jaosorior | bnemec: Alright, just wanted to make sure. I'll check those services out | 22:19 |
shardy | Simple cosmetic review that's passed CI if anyone has a moment: | 22:28 |
shardy | https://review.openstack.org/#/c/255785/1 | 22:28 |
shardy | bnemec: hey, do you have a moment to discuss https://review.openstack.org/#/c/250059/ ? | 22:30 |
shardy | It was about creating all the per-role flavors by default, when folks might not always need them | 22:30 |
shardy | Context is https://review.openstack.org/#/c/254667/ which potentially enables predictable placement by tagging specific nodes as e.g "controller-0" | 22:31 |
shardy | if you do that, it's an alternative to the per-role flavors which would no longer be required | 22:32 |
bnemec | shardy: 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 |
bnemec | shardy: Although it does have some drawbacks for users who don't necessarily want to assign every node. | 22:35 |
shardy | bnemec: Yeah, it's definitely not something everyone will want, but it's been a frequent RFE for some deployments where they desire control over placement | 22:36 |
bnemec | For 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 |
shardy | bnemec: Yeah, this actually plays pretty well with the template RemovalPolicies, in that node indexes are never reused | 22:36 |
*** rasca has quit IRC | 22:37 | |
shardy | So if a node fails, it's permanently excluded and you have to choose a new unique ID | 22:37 |
bnemec | shardy: Ah, interesting. | 22:37 |
shardy | bnemec: not quite sure how this would interface with AHC? I was thinking more just support nodes.json tagging per-node on registration with ironic | 22:37 |
bnemec | shardy: 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 |
shardy | bnemec: I guess it doesn't replace that, but it is an alternative | 22:38 |
bnemec | shardy: 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 #tripleo | 22:39 | |
shardy | bnemec: ah, I was thinking we'd still allow that, but optionally also allow folks to feed in per-node mappings at node registration time | 22:39 |
shardy | so perhaps we just land your patches as-is | 22:39 |
shardy | I'm not sure how you'd define per-node criteria when matching, as I assume some at least will be the same | 22:39 |
shardy | unless you match on DMI id's or something, which seems more work than just passing a tag via nodes.json | 22:40 |
bnemec | shardy: 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 |
shardy | bnemec: Ok, makes sense, I guess that's what you get with most out-of-the-box nova setups anyway | 22:41 |
shardy | bnemec: thanks, just wanted to discuss it before +Aing your patches related to this | 22:41 |
bnemec | shardy: Yep, exactly. | 22:41 |
shardy | sounds like we can do both, at least for now | 22:41 |
bnemec | shardy: Roger, appreciate the reviews. | 22:41 |
*** weshay has quit IRC | 22:47 | |
openstackgerrit | Merged openstack/tripleo-heat-templates: Fix wrong keypair parameter description https://review.openstack.org/255785 | 22:48 |
openstackgerrit | Steven Hardy proposed openstack-infra/tripleo-ci: Don't pass duplicate projects to tripleo.sh --delorean-build https://review.openstack.org/249819 | 22:54 |
openstackgerrit | Ben Nemec proposed openstack/tripleo-heat-templates: Fix yaml validation errors in multiple-nics https://review.openstack.org/252683 | 22:54 |
openstackgerrit | Ben Nemec proposed openstack/tripleo-heat-templates: Add Management Network For System Administration. https://review.openstack.org/226057 | 22:54 |
* bnemec looks at his pending review list | 22:59 | |
* bnemec despairs | 22:59 | |
*** thrash is now known as thrash|g0ne | 23:04 | |
*** chlong has joined #tripleo | 23:05 | |
openstackgerrit | Giulio Fidente proposed openstack/python-tripleoclient: Forcibly clear parameters (now passed as parameter_defaults) https://review.openstack.org/256670 | 23:08 |
*** olap has quit IRC | 23:12 | |
*** olap has joined #tripleo | 23:13 | |
*** gfidente has quit IRC | 23:17 | |
*** shardy has quit IRC | 23:21 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/python-tripleoclient: Use renamed overcloud.yaml template https://review.openstack.org/253354 | 23:22 |
*** rlandy has quit IRC | 23:28 | |
*** jcoufal has joined #tripleo | 23:36 | |
*** sthillma has quit IRC | 23:44 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Allow to enable Monitoring agent on Overcloud Controller(s) https://review.openstack.org/254788 | 23:47 |
*** jaosorior has quit IRC | 23:57 | |
*** dmacpher has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!