| *** absubram has quit IRC | 00:17 | |
| openstackgerrit | Reedip proposed openstack/os-cloud-config: Remove support for remove py33/py26 https://review.openstack.org/264481 | 00:21 |
|---|---|---|
| openstackgerrit | Reedip proposed openstack/os-cloud-config: Remove support for py33/py26 https://review.openstack.org/264481 | 00:22 |
| openstackgerrit | James Slagle proposed openstack/instack-undercloud: Make 98-undercloud-setup rerunnable https://review.openstack.org/254946 | 00:44 |
| openstackgerrit | James Slagle proposed openstack/instack-undercloud: Set nova ironic admin_url correctly https://review.openstack.org/264492 | 00:44 |
| openstackgerrit | James Slagle proposed openstack/instack-undercloud: Manage keystone related configuration via Puppet https://review.openstack.org/264493 | 00:44 |
| *** dmacpher has joined #tripleo | 00:48 | |
| *** eil397 has quit IRC | 01:03 | |
| *** absubram has joined #tripleo | 01:42 | |
| *** absubram_ has joined #tripleo | 01:43 | |
| *** absubram has quit IRC | 01:47 | |
| *** absubram_ is now known as absubram | 01:47 | |
| *** julim has joined #tripleo | 02:00 | |
| *** sthillma has quit IRC | 02:09 | |
| *** sthillma has joined #tripleo | 02:14 | |
| *** trozet has joined #tripleo | 02:18 | |
| *** absubram has quit IRC | 02:28 | |
| *** sthillma has quit IRC | 02:59 | |
| *** links has joined #tripleo | 03:10 | |
| *** links has quit IRC | 03:10 | |
| *** links has joined #tripleo | 03:12 | |
| *** apetrich_ has joined #tripleo | 03:16 | |
| *** apetrich has quit IRC | 03:18 | |
| *** trozet has quit IRC | 03:33 | |
| *** yamahata has quit IRC | 03:36 | |
| *** thrash is now known as thrash|g0ne | 03:42 | |
| *** sthillma has joined #tripleo | 03:45 | |
| *** tzumainn has quit IRC | 03:47 | |
| *** sthillma_ has joined #tripleo | 03:48 | |
| *** sthillma has quit IRC | 03:50 | |
| *** sthillma_ is now known as sthillma | 03:50 | |
| *** rhallisey has quit IRC | 03:51 | |
| *** coolsvap|away is now known as coolsvap | 04:21 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-tripleo: Update to pull in rspec-puppet-facts https://review.openstack.org/264561 | 04:28 |
| *** jaosorior has joined #tripleo | 04:45 | |
| *** dmacpher has quit IRC | 04:58 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/python-tripleoclient: Fix chicken and eggs problem with tripleo-heat-templates https://review.openstack.org/264571 | 05:03 |
| *** absubram has joined #tripleo | 05:06 | |
| *** dmacpher has joined #tripleo | 05:14 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/python-tripleoclient: Move more functions to utils.py https://review.openstack.org/264574 | 05:15 |
| *** masco has joined #tripleo | 05:41 | |
| *** absubram has quit IRC | 06:05 | |
| *** sthillma has quit IRC | 06:35 | |
| *** sthillma has joined #tripleo | 06:38 | |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/os-net-config: Use assertTrue/False instead of assertEqual(T/F) https://review.openstack.org/264607 | 06:45 |
| *** coolsvap is now known as coolsvap|away | 06:56 | |
| *** oshvartz has joined #tripleo | 07:09 | |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/python-tripleoclient: Use assertTrue/False instead of assertEqual(T/F) https://review.openstack.org/264634 | 07:20 |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/tuskar: Use assertTrue/False instead of assertEqual(T/F) https://review.openstack.org/264639 | 07:25 |
| *** jprovazn has joined #tripleo | 07:26 | |
| *** ukalifon1 has joined #tripleo | 07:36 | |
| *** coolsvap|away is now known as coolsvap | 07:41 | |
| *** jtomasek has joined #tripleo | 07:53 | |
| *** dmacpher has quit IRC | 07:56 | |
| *** liverpooler has joined #tripleo | 07:58 | |
| *** stendulker has joined #tripleo | 08:16 | |
| *** bvandenh has joined #tripleo | 08:17 | |
| *** aufi has joined #tripleo | 08:23 | |
| *** sthillma has quit IRC | 08:28 | |
| *** jtomasek has quit IRC | 08:29 | |
| *** jtomasek has joined #tripleo | 08:30 | |
| *** devvesa has joined #tripleo | 08:33 | |
| *** gfidente has joined #tripleo | 08:35 | |
| *** gfidente has quit IRC | 08:35 | |
| *** gfidente has joined #tripleo | 08:35 | |
| *** olap_ has joined #tripleo | 08:37 | |
| *** ifarkas has joined #tripleo | 08:40 | |
| *** mcornea has joined #tripleo | 08:45 | |
| *** jcoufal has joined #tripleo | 08:57 | |
| *** athomas has joined #tripleo | 09:01 | |
| *** jaosorior has quit IRC | 09:17 | |
| *** bvandenh has quit IRC | 09:21 | |
| *** nico_auv has joined #tripleo | 09:23 | |
| openstackgerrit | Jaume Devesa proposed openstack/tripleo-heat-templates: Changes for configuring Nuage https://review.openstack.org/260399 | 09:25 |
| *** jaosorior has joined #tripleo | 09:32 | |
| *** bvandenh has joined #tripleo | 09:33 | |
| *** derekh has joined #tripleo | 09:35 | |
| *** shardy has joined #tripleo | 09:35 | |
| *** jistr has joined #tripleo | 09:41 | |
| *** bvandenh has quit IRC | 09:41 | |
| * jistr forgot to start irc :) | 09:41 | |
| *** yamahata has joined #tripleo | 09:41 | |
| jistr | gfidente: i'll rebase the upgrade patch chain and do a few small tweaks, sounds good? | 09:50 |
| gfidente | jistr, ack go ahead, i'm on call with marios talking about the tht stable/libery branch | 09:51 |
| jistr | gfidente: alright. If you guys think i might be of some use on that, i can join you, otherwise i'm happy doing the upgrades amendments/rebase. | 09:52 |
| *** yamahata has quit IRC | 09:53 | |
| *** dtantsur|afk is now known as dtantsur | 10:03 | |
| *** mkovacik has joined #tripleo | 10:04 | |
| devvesa | Hi shardy | 10:05 |
| shardy | devvesa: Hi! | 10:06 |
| shardy | thanks for rebasing the stable/liberty patches | 10:06 |
| devvesa | Asking to you because I think you have all the documentation in your mind :) | 10:06 |
| devvesa | shardy: Thanks for review them! | 10:06 |
| *** paramite has joined #tripleo | 10:07 | |
| devvesa | I have my virtual tripleo env : http://docs.openstack.org/developer/tripleo-docs/environments/environments.html#virtual-environment | 10:07 |
| devvesa | Is there any doc to follow to add more VMs? | 10:07 |
| shardy | devvesa: I normally just set NODE_COUNT before running instack-virt-setup, but do you need to add VMs to an existing environment? | 10:09 |
| devvesa | shardy: exactly | 10:10 |
| devvesa | Well, I don't need it. I can build another environment, but I guess it will take more time | 10:10 |
| shardy | devvesa: Yeah, sorry we don't have that documented atm, would be a good idea tho | 10:15 |
| *** electrofelix has joined #tripleo | 10:15 | |
| shardy | it'll be a case of looking at instack-virt-setup and creating the VMs with virsh in a similar way | 10:15 |
| shardy | then update instackenv.json and register the new nodes | 10:15 |
| shardy | tbh I normally just delete everything and re-run instack-virt-setup to get a clean environment | 10:15 |
| devvesa | I've done this... (update the instackenv.json) but the new node in ironic is in maintenance mode and it fails | 10:16 |
| devvesa | I mean: | 10:16 |
| devvesa | I run `tripleo.sh --register-nodes` and it fails | 10:17 |
| devvesa | and then the new node becomes on maintenance | 10:18 |
| devvesa | Well, nevermind. I'll create a new clean env :) With everything merged in master it will be easier than before | 10:18 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: TEST: this will help testing major upgrades https://review.openstack.org/260456 | 10:18 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Split pacemaker common check_service function out of _restart.sh https://review.openstack.org/260443 | 10:18 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Add resources for major upgrade in Pacemaker scenario https://review.openstack.org/253276 | 10:18 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Use timeout to check for services status https://review.openstack.org/259573 | 10:18 |
| shardy | https://github.com/openstack/instack-undercloud/blob/master/scripts/instack-virt-setup#L113 | 10:19 |
| shardy | devvesa: I think you'll need to either re-run the tripleo devtest_testenv.sh, or figure out what it's doing with the nodes | 10:19 |
| jistr | gfidente: i only changed this one https://review.openstack.org/#/c/259573/2/extraconfig/tasks/pacemaker_resource_restart.sh, the rest is rebases | 10:19 |
| shardy | I'm not sure on the exact details without looking at the code | 10:20 |
| jistr | gfidente: lemme know if something seems bad about the changes | 10:20 |
| jistr | gfidente: i may go ahead and define an echo_error function in pacemaker_functions to DRY the "| tee /dev/fd2" parts | 10:22 |
| jistr | originally shardy suggested we echo errors to stderr because that is processed separately by heat i think, and could be used to find the errors more easily than going through the full stdout output | 10:23 |
| shardy | devvesa: you can find it at /usr/libexec/openstack-tripleo/devtest_testenv.sh | 10:23 |
| shardy | there's scope for simplifying things I think, as this stuff is derived from the old TripleO devtest setup | 10:23 |
| * jistr testing the upgrade locally now | 10:25 | |
| *** jcoufal has quit IRC | 10:28 | |
| *** jaosorior has quit IRC | 10:34 | |
| *** jaosorior has joined #tripleo | 10:35 | |
| *** krickwix has joined #tripleo | 10:39 | |
| *** tremble has joined #tripleo | 10:40 | |
| *** tremble has left #tripleo | 10:40 | |
| *** mgould has joined #tripleo | 10:53 | |
| *** paramite is now known as paramite|afk | 10:58 | |
| *** coolsvap is now known as coolsvap|away | 11:11 | |
| *** regebro has joined #tripleo | 11:16 | |
| *** stendulker has quit IRC | 11:18 | |
| *** lucas-dinner is now known as lucasagomes | 11:20 | |
| cmyster | shardy: ping | 11:23 |
| shardy | cmyster: hi! | 11:23 |
| cmyster | yo | 11:24 |
| cmyster | pm? | 11:24 |
| *** akrivoka has joined #tripleo | 11:24 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Enable Dell Storage Center iscsi Backends in Cinder https://review.openstack.org/244923 | 11:25 |
| openstackgerrit | Merged openstack/tripleo-common: Update the container flag https://review.openstack.org/264373 | 11:28 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Enable configuration of Neutron QoS https://review.openstack.org/247056 | 11:40 |
| openstackgerrit | Merged openstack/python-tripleoclient: Fix db error when running python34 unit tests https://review.openstack.org/264031 | 11:48 |
| openstackgerrit | Merged openstack/instack-undercloud: Add ip addresses to horizon allowed_hosts https://review.openstack.org/234939 | 11:50 |
| shardy | Simple instack-undercloud fix passing CI if someone has a moment: | 11:53 |
| shardy | https://review.openstack.org/#/c/224852/ | 11:53 |
| shardy | checks for sufficient ram when installing the undercloud | 11:53 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Add TimeZone parameter for all node types https://review.openstack.org/249886 | 11:56 |
| *** paramite|afk is now known as paramite | 12:01 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/diskimage-builder: deploy-ironic: Fix syntax error when checking for root device hints https://review.openstack.org/264736 | 12:07 |
| *** morazi has quit IRC | 12:08 | |
| *** jprovazn has quit IRC | 12:18 | |
| openstackgerrit | Merged openstack/instack-undercloud: Add check for sufficient memory to undercloud install https://review.openstack.org/224852 | 12:20 |
| *** thrash|g0ne is now known as thrash | 12:20 | |
| *** jcoufal has joined #tripleo | 12:27 | |
| jistr | hmm i'm still hitting the problem with stacks in UPDATE_IN_PROGRESS despite all resources are *_COMPLETE.. kinda blocks me from testing the upgrades on upstream environment. Is this perhaps fixed in a newer Heat release than what i have? | 12:29 |
| jistr | [stack@instack ~]$ rpm -q openstack-heat-engine | 12:29 |
| jistr | openstack-heat-engine-6.0.0-dev248.el7.centos.noarch | 12:29 |
| jistr | i'm searching for more info on this but if someone knows something related off the bat, i'd be grateful for any input | 12:30 |
| jistr | http://fpaste.org/308152/14521692/raw/ | 12:30 |
| slagle | jistr: there was a heat issue with that, if you restart heat-engine it should move the top level stack to failed | 12:42 |
| slagle | jistr: it's possible the delorean in use is too old to have that fix | 12:42 |
| jistr | slagle: yeah i recall doing that previously (before the holidays break), which got me to UPDATE_FAILED, but when i retried the stack update, it got stuck in exactly the same place... | 12:43 |
| jistr | slagle: i'll try to update to heat from delorean-current | 12:43 |
| jistr | thanks :) | 12:44 |
| slagle | i think the cause of it is an unhandled heat exception somewhere | 12:44 |
| *** jprovazn has joined #tripleo | 12:46 | |
| openstackgerrit | Merged openstack/instack-undercloud: Set nova ironic admin_url correctly https://review.openstack.org/264492 | 12:49 |
| *** gfidente is now known as gfidente|pto | 12:58 | |
| *** dprince has joined #tripleo | 13:07 | |
| *** morazi has joined #tripleo | 13:07 | |
| *** links has quit IRC | 13:07 | |
| shardy | jistr: I think the fix may be https://review.openstack.org/#/q/Ic948c2fe5baf23c9c4ced33060f672ca9c278a19,n,z ? | 13:10 |
| *** weshay_xchat has joined #tripleo | 13:10 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Enable configuration of Neutron QoS https://review.openstack.org/264757 | 13:10 |
| *** tzumainn has joined #tripleo | 13:13 | |
| jistr | shardy: yeah that could be it. I see the fix landed on Nov 18, while http://trunk.rdoproject.org/centos7/current-tripleo/ has heat from Nov 16, so the heat update could fix my issues indeed. Just finished overcloud deploy, now starting an upgrade to see if i can get further. Thanks :) | 13:14 |
| shardy | jistr: ouch, that's a pretty old current-tripleo pin :( | 13:15 |
| jistr | yeah there were some issues with bumping that IIRC, not sure if they're resolved now | 13:15 |
| *** jayg|g0n3 is now known as jayg | 13:16 | |
| shardy | jistr: kk, FWIW I've been running with a local delorean built heat from heat master lately and it seems to work OK | 13:16 |
| shardy | I hit one issue when restarting heat-engine in the middle of an update yesterday, so I wouldn't reccomend doing that unless you have no other option | 13:17 |
| shardy | ref https://bugs.launchpad.net/heat/+bug/1531547 | 13:17 |
| openstack | Launchpad bug 1531547 in heat "Interrupted stack-update leaves undeletable stack" [High,In progress] - Assigned to Steven Hardy (shardy) | 13:17 |
| jistr | shardy: ack, thanks for the info | 13:17 |
| *** Madkiss has quit IRC | 13:17 | |
| *** Madkiss has joined #tripleo | 13:17 | |
| *** Madkiss has joined #tripleo | 13:17 | |
| jistr | got further now (to a properly failed upgrade :) ) thanks shardy & slagle | 13:24 |
| *** rhallisey has joined #tripleo | 13:25 | |
| *** trown|outttypeww is now known as trown | 13:35 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/diskimage-builder: Install IPA in a virtual enviroment https://review.openstack.org/235843 | 13:36 |
| *** masco has quit IRC | 13:36 | |
| dtantsur | lucasagomes, btw I've finished https://review.openstack.org/#/c/263309/ could you take a look? | 13:38 |
| lucasagomes | dtantsur, ++ | 13:38 |
| lucasagomes | will do now | 13:38 |
| dtantsur | +504, -554 | 13:38 |
| dtantsur | meh, not an easy patch to review :( | 13:38 |
| *** rhallisey has quit IRC | 13:40 | |
| lucasagomes | yeah it's hard to map what exactly have changed, because it was almost completely refactored (and that's a good thing, cause os-cloud-config is not flexible the way it is right now in tree) | 13:43 |
| *** rhallisey has joined #tripleo | 13:43 | |
| *** rhallisey has quit IRC | 13:45 | |
| openstackgerrit | Emilien Macchi proposed openstack/instack-undercloud: Deploy Logging on the undercloud with Puppet https://review.openstack.org/258653 | 13:45 |
| *** rhallisey has joined #tripleo | 13:46 | |
| dtantsur | lucasagomes, yeah, it's essentially a full rewrite | 13:50 |
| slagle | lucasagomes: does this look at familiar at all? http://fpaste.org/308175/14521746/ | 13:51 |
| slagle | lucasagomes: seeing that error on our stable/liberty CI jobs | 13:51 |
| lucasagomes | slagle, haven't seem that error before | 13:52 |
| lucasagomes | odd, lemme see if I can find something about it | 13:52 |
| lucasagomes | slagle, seems the nova code in liberty is depending upon some newer code from oslo.serialization? | 13:53 |
| slagle | lucasagomes: this looks suspect: https://review.openstack.org/#/c/246792/ | 13:54 |
| slagle | merged yesterday | 13:54 |
| slagle | or today rather | 13:54 |
| slagle | yea, that's got to be it | 13:55 |
| lucasagomes | slagle, yeah I bet it may be it | 13:55 |
| lucasagomes | does oslo.serialization have a stable branch too? If so, we need to make sure https://github.com/openstack/oslo.serialization/commit/92f2111b26d21a30e3dda62a942aa7f28ae41537 was backported to it | 13:56 |
| lucasagomes | tho it would be odd because it's not fixing anything | 13:56 |
| *** absubram has joined #tripleo | 13:57 | |
| *** rhallisey has quit IRC | 13:57 | |
| *** absubram_ has joined #tripleo | 13:58 | |
| lucasagomes | dtantsur, reviewed, left a question at L246 of nodes.py | 13:59 |
| dtantsur | lucasagomes, ironicclient does retry on service unavailable: https://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/http.py#L166-L167 | 14:01 |
| lucasagomes | slagle, dump_as_bytes is not part of the liberty version of oslo.serialization https://github.com/openstack/oslo.serialization/blob/stable/liberty/oslo_serialization/jsonutils.py | 14:01 |
| *** absubram has quit IRC | 14:01 | |
| *** absubram_ is now known as absubram | 14:01 | |
| lucasagomes | dtantsur, oh | 14:04 |
| lucasagomes | dtantsur, ack, I thought it was only conflict | 14:04 |
| *** rhallisey has joined #tripleo | 14:04 | |
| dtantsur | lucasagomes, we implemented the same thing that nova used to retry | 14:05 |
| lucasagomes | dtantsur, gotcha! Missed that bit | 14:05 |
| lucasagomes | changed my vote | 14:05 |
| *** trozet has joined #tripleo | 14:06 | |
| *** trozet_ has joined #tripleo | 14:08 | |
| *** trozet_ has quit IRC | 14:09 | |
| lucasagomes | slagle, should re revert that? Since the method is not available in the liberty version of oslo.serialization? | 14:09 |
| *** trozet_ has joined #tripleo | 14:09 | |
| dtantsur | I think revert or quick fix is required | 14:09 |
| EmilienM | shardy, dprince: does heat support versionless for auth_uri & identity_uri ? look my last comment on https://review.openstack.org/261326 for full context | 14:15 |
| openstackgerrit | Merged openstack/puppet-tripleo: Upgrade all packages after puppet managed ones https://review.openstack.org/261041 | 14:15 |
| openstackgerrit | Merged openstack/puppet-tripleo: Haproxy has non-working Horizon session persistence. https://review.openstack.org/258483 | 14:16 |
| slagle | lucasagomes: sorry, was on a call | 14:17 |
| shardy | EmilienM: Yes, I recently proposed some changes to devstack which switch all auth_uri's except the ec2authtoken one to versionless | 14:17 |
| slagle | lucasagomes: the needed patch isn't in oslo.serialization stable/liberty? | 14:17 |
| EmilienM | shardy: in liberty? | 14:17 |
| shardy | EmilienM: No, master/mitaka | 14:17 |
| EmilienM | shardy: ah | 14:17 |
| EmilienM | puppet ci is on liberty atm | 14:17 |
| EmilienM | it might explain why | 14:17 |
| shardy | The same config should work for liberty, but I've not tested it yet | 14:17 |
| EmilienM | shardy: please comment on the patch if we need to set the version | 14:18 |
| lucasagomes | slagle, yeah it's not in stable/liberty | 14:18 |
| slagle | yea i see | 14:18 |
| lucasagomes | here's the jsonutils.py module for stable/liberty: https://github.com/openstack/oslo.serialization/blob/stable/liberty/oslo_serialization/jsonutils.py | 14:18 |
| *** bvandenh has joined #tripleo | 14:19 | |
| slagle | yea was just looking :) | 14:19 |
| shardy | EmilienM: so, master puppet-heat CI tests with liberty heat? | 14:19 |
| EmilienM | shardy: right. | 14:19 |
| EmilienM | shardy: tl;dr: we wait for trusty packaging to bump our CI to mitaka | 14:20 |
| EmilienM | shardy: should happen very very soon | 14:20 |
| shardy | EmilienM: Ah, OK | 14:20 |
| shardy | AFAIK the config should be the same, I was just confused :) | 14:20 |
| EmilienM | shardy: so you mean we could use versionless? See logs, it does not look like it works | 14:23 |
| shardy | http://paste.openstack.org/show/483229/ | 14:23 |
| shardy | EmilienM: that's how we configure things on recent devstack | 14:24 |
| shardy | it's all versionless except ec2authtoken | 14:24 |
| shardy | EmilienM: I'm trying to see the difference between that and the puppet config | 14:24 |
| EmilienM | shardy: thanks, all heat config is in logs, you can easily find it | 14:25 |
| *** lblanchard has joined #tripleo | 14:26 | |
| slagle | lucasagomes: i submitted a nova revert, https://review.openstack.org/#/c/264793/ | 14:28 |
| lucasagomes | thanks! | 14:28 |
| shardy | EmilienM: I think you're missing the domain configuration, which is required with versionless, because it defaults to the v3 auth plugin | 14:29 |
| shardy | I'll comment on the review | 14:29 |
| EmilienM | shardy: thx a lot | 14:29 |
| *** trozet_ has quit IRC | 14:30 | |
| *** egafford has joined #tripleo | 14:30 | |
| *** trozet_ has joined #tripleo | 14:32 | |
| *** lucasagomes is now known as lucas-hungry | 14:32 | |
| *** trozet has quit IRC | 14:35 | |
| *** trozet_ is now known as trozet | 14:35 | |
| *** trozet_ has joined #tripleo | 14:39 | |
| *** rpothier has joined #tripleo | 14:44 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Making nova parameters configurable for nuage-metadata-agent https://review.openstack.org/263773 | 14:48 |
| slagle | our stable/liberty CI is failing due to, https://bugs.launchpad.net/tripleo/+bug/1531881 | 14:49 |
| openstack | Launchpad bug 1531881 in tripleo "AttributeError: 'module' object has no attribute 'dump_as_bytes'" [Critical,New] - Assigned to James Slagle (james-slagle) | 14:49 |
| slagle | fyi | 14:49 |
| *** jcoufal_ has joined #tripleo | 14:50 | |
| EmilienM | slagle, dprince: monitoring patch pass now CI and addressed all your reviews: https://review.openstack.org/#/c/253709/ | 14:51 |
| EmilienM | there is also its dep on puppet-elements: https://review.openstack.org/#/c/252051/ | 14:52 |
| EmilienM | if we could land it soon or so, so we can iterate and get some feedback | 14:52 |
| dprince | EmilienM: cool, I will review again shortly | 14:52 |
| EmilienM | thanks! | 14:52 |
| *** liverpooler has quit IRC | 14:53 | |
| *** jcoufal has quit IRC | 14:53 | |
| *** dprince has quit IRC | 14:58 | |
| *** dprince has joined #tripleo | 14:59 | |
| *** pradk has joined #tripleo | 15:01 | |
| rhallisey | dprince, morning, what was the reason why we arn't building the heat-docker-agents container with the openstack packages install? | 15:03 |
| rhallisey | I don't remember | 15:03 |
| dprince | rhallisey: we were trying to avoid installing packages, because it means any package update would potentially require a new heat-docker-agents container as well | 15:05 |
| dprince | rhallisey: sorry, installing is fine, pre-installing was what we tried to avoid | 15:05 |
| rhallisey | dprince, would that be ok though? swapping out the container | 15:05 |
| dprince | rhallisey: it is okay, as an iterim solution | 15:05 |
| rhallisey | since that's how a general upgrade would work | 15:05 |
| *** oshvartz has quit IRC | 15:06 | |
| rhallisey | dprince, well what would be the long term solution though, or the ideal one | 15:06 |
| rhallisey | I'd rather swap out a container for an update then doing a yum update | 15:06 |
| *** mcornea has quit IRC | 15:08 | |
| *** egafford has quit IRC | 15:09 | |
| *** bvandenh has quit IRC | 15:09 | |
| *** bvandenh has joined #tripleo | 15:09 | |
| *** absubram has quit IRC | 15:12 | |
| *** yamahata has joined #tripleo | 15:18 | |
| *** jtomasek has quit IRC | 15:20 | |
| *** masco has joined #tripleo | 15:21 | |
| *** jtomasek has joined #tripleo | 15:25 | |
| *** nico_auv has quit IRC | 15:30 | |
| *** dprince has quit IRC | 15:32 | |
| *** dprince_ has joined #tripleo | 15:32 | |
| *** lucas-hungry is now known as lucasagomes | 15:34 | |
| *** slagle has quit IRC | 15:39 | |
| *** olap_ has quit IRC | 15:43 | |
| *** electrofelix has quit IRC | 15:50 | |
| *** dprince_ has quit IRC | 15:51 | |
| *** dprince has joined #tripleo | 15:51 | |
| *** electrofelix has joined #tripleo | 15:51 | |
| *** rook has quit IRC | 15:51 | |
| *** rook has joined #tripleo | 15:52 | |
| *** egafford has joined #tripleo | 16:02 | |
| *** absubram has joined #tripleo | 16:17 | |
| *** nico_auv has joined #tripleo | 16:18 | |
| * marios just release tripleo-heat-templates 0.8.7 (stable/liberty) and 1.0.0 (master) http://tarballs.openstack.org/tripleo-heat-templates/ | 16:25 | |
| *** weshay_xchat has quit IRC | 16:32 | |
| *** aufi has quit IRC | 16:33 | |
| *** ifarkas has quit IRC | 16:33 | |
| *** weshay_xchat has joined #tripleo | 16:34 | |
| *** gfidente|pto has quit IRC | 16:36 | |
| lucasagomes | hi all, someone can take a look at this patch https://review.openstack.org/#/c/239561 please ? It already has a 2+2 (and the failure there seems transient) | 16:39 |
| lucasagomes | this is a dependency for another patch fixing the ironic-agent element when installing things from source | 16:39 |
| *** masco has quit IRC | 16:39 | |
| *** sthillma has joined #tripleo | 16:52 | |
| *** ukalifon1 has quit IRC | 17:03 | |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Enable Equallogic Backends in Cinder https://review.openstack.org/264871 | 17:08 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Enable Dell Storage Center iscsi Backends in Cinder https://review.openstack.org/264872 | 17:08 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Wire Neutron ML2 plugin and OVS agent settings as arrays https://review.openstack.org/260414 | 17:08 |
| *** trown is now known as trown|lunch | 17:13 | |
| *** krickwix has quit IRC | 17:15 | |
| *** dtantsur is now known as dtantsur|afk | 17:15 | |
| *** slagle has joined #tripleo | 17:17 | |
| *** jprovazn has quit IRC | 17:22 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Remove deleted Cinder rows https://review.openstack.org/209019 | 17:23 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Use new heat-docker-agents images https://review.openstack.org/253515 | 17:30 |
| shardy | Hey folks, this https://review.openstack.org/#/c/250498/ is passing, adds coverage of overcloud deletes to CI runs | 17:32 |
| shardy | needs one more +2/A | 17:32 |
| shardy | Also https://review.openstack.org/#/c/249819/ is another toci_instack patch | 17:33 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Don't pass duplicate projects to tripleo.sh --delorean-build https://review.openstack.org/249819 | 17:36 |
| jistr | i guess the overcloud delete adds a few minutes to the CI run time but probably is worth it. Maybe if we're really confident about overcloud deletes in the future, they can be removed again from CI. | 17:36 |
| shardy | jistr: it doesn't add much IME | 17:37 |
| shardy | jistr: and there's a whole class of bug folks get bitten by related to DELETE_FAILED stacks which we never hit in CI | 17:37 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Add CI coverage of deleting the deployed overcloud https://review.openstack.org/250498 | 17:38 |
| shardy | jistr: In a followup patch I'll add some instrumentation to quantify it, but I'd be surprised if it's even one minute | 17:38 |
| rhallisey | jistr, https://review.openstack.org/#/c/259441/ this backport has merged upstream | 17:38 |
| rhallisey | just an fyi | 17:38 |
| rhallisey | as of like 1 min ago | 17:38 |
| shardy | locally my overcloud deletes take probably 20-30 seconds | 17:38 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Use new heat-docker-agents images https://review.openstack.org/259441 | 17:40 |
| jistr | rhallisey: cool, thanks for letting me know | 17:40 |
| jistr | rhallisey: ^ in stable/liberty too now | 17:40 |
| rhallisey | jistr, your clever :) | 17:41 |
| rhallisey | jistr, thanks | 17:41 |
| *** jcoufal_ has quit IRC | 17:44 | |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: TEST: this will help testing major upgrades https://review.openstack.org/260456 | 17:44 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-heat-templates: Add resources for major upgrade in Pacemaker scenario https://review.openstack.org/253276 | 17:44 |
| *** jistr has quit IRC | 17:45 | |
| *** mkovacik has quit IRC | 17:46 | |
| *** athomas has quit IRC | 17:51 | |
| *** gchamoul has quit IRC | 17:52 | |
| *** bvandenh has quit IRC | 18:01 | |
| *** akrivoka has quit IRC | 18:06 | |
| openstackgerrit | Merged openstack/tripleo-specs: New Spec: Deploy Puppet modules via Swift https://review.openstack.org/245309 | 18:06 |
| *** mgould has quit IRC | 18:08 | |
| *** penick has quit IRC | 18:08 | |
| *** trown|lunch is now known as trown | 18:08 | |
| *** eil397 has joined #tripleo | 18:09 | |
| *** paramite is now known as paramite|afk | 18:11 | |
| *** derekh has quit IRC | 18:11 | |
| *** jayg has quit IRC | 18:13 | |
| *** bvandenh has joined #tripleo | 18:13 | |
| *** nico_auv has quit IRC | 18:13 | |
| *** jpeeler has quit IRC | 18:13 | |
| *** lblanchard1 has joined #tripleo | 18:13 | |
| *** masco has joined #tripleo | 18:14 | |
| *** rbrady has quit IRC | 18:14 | |
| *** trozet has quit IRC | 18:15 | |
| *** mburned has quit IRC | 18:15 | |
| *** jayg has joined #tripleo | 18:15 | |
| *** lblanchard has quit IRC | 18:15 | |
| *** trozet_ has quit IRC | 18:15 | |
| *** trown has quit IRC | 18:15 | |
| *** trown|brb has joined #tripleo | 18:16 | |
| *** weshay_xchat has quit IRC | 18:16 | |
| *** rhallisey has quit IRC | 18:16 | |
| *** masco has quit IRC | 18:19 | |
| *** rook has quit IRC | 18:20 | |
| *** rook-desktio has joined #tripleo | 18:21 | |
| *** trown|brb is now known as trown | 18:21 | |
| *** cwolferh has quit IRC | 18:21 | |
| *** cwolferh has joined #tripleo | 18:21 | |
| greghaynes | bnemec: we should pin that fedora mirror in dib | 18:25 |
| greghaynes | I think I tried it a while back but realized it didnt 'just work' | 18:25 |
| *** cwolferh has quit IRC | 18:26 | |
| *** trozet has joined #tripleo | 18:28 | |
| *** rhallisey has joined #tripleo | 18:28 | |
| *** weshay_xchat has joined #tripleo | 18:30 | |
| *** dsneddon has quit IRC | 18:33 | |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-common: Build image files from definitions in yaml https://review.openstack.org/235569 | 18:34 |
| *** dsneddon has joined #tripleo | 18:34 | |
| *** jaosorior has quit IRC | 18:34 | |
| *** jpeeler has joined #tripleo | 18:34 | |
| *** jaosorior has joined #tripleo | 18:35 | |
| *** trozet_ has joined #tripleo | 18:35 | |
| *** eggmaste` has joined #tripleo | 18:36 | |
| *** dsneddon has quit IRC | 18:36 | |
| *** dsneddon has joined #tripleo | 18:36 | |
| bnemec | greghaynes: It looks like DIB_DISTRIBUTION_MIRROR should work: https://github.com/openstack/diskimage-builder/blob/master/elements/fedora/pre-install.d/01-set-fedora-mirror | 18:36 |
| *** eggmaster has quit IRC | 18:37 | |
| bnemec | We use it in toci and it helped with our mirror failures immensely. | 18:37 |
| greghaynes | hrm | 18:37 |
| *** dsneddon has quit IRC | 18:37 | |
| *** dsneddon has joined #tripleo | 18:37 | |
| *** dsneddon has quit IRC | 18:37 | |
| *** dsneddon has joined #tripleo | 18:38 | |
| greghaynes | http://git.openstack.org/cgit/openstack/diskimage-builder/tree/elements/fedora/test-elements/build-succeeds/environment.d/10-fedora-pin-mirror.bash | 18:38 |
| greghaynes | that should be pinning it | 18:38 |
| greghaynes | but it doesn't appear to be working | 18:38 |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-common: Use the new yaml syntax in CI https://review.openstack.org/254886 | 18:39 |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-common: Build image files from definitions in yaml https://review.openstack.org/235569 | 18:39 |
| *** paramite|afk is now known as paramite | 18:42 | |
| *** trozet_ has quit IRC | 18:43 | |
| bnemec | greghaynes: Oh, the failure in http://logs.openstack.org/61/239561/6/check/gate-dib-dsvm-functests-devstack-centos7/0b750bd/console.html actually happens before the pin script gets run. | 18:43 |
| bnemec | The other failure on that patch was downloading the base cloud image, so again before the pinning would happen. | 18:43 |
| *** eggmaste` has quit IRC | 18:44 | |
| *** eggmaste` has joined #tripleo | 18:44 | |
| greghaynes | ah, right, and IIRC there isnt fedora pinning for that yet | 18:44 |
| bnemec | You would have to export DIB_CLOUD_IMAGES: https://github.com/openstack/diskimage-builder/blob/master/elements/fedora/root.d/10-fedora-cloud-image#L27 | 18:46 |
| bnemec | I'm not sure whether you could do that in an environment file or not. | 18:46 |
| *** ayoung has quit IRC | 18:46 | |
| *** rbrady has joined #tripleo | 18:46 | |
| *** ayoung has joined #tripleo | 18:46 | |
| greghaynes | We should be able to | 18:46 |
| *** pradk has quit IRC | 18:47 | |
| *** gchamoul has joined #tripleo | 18:47 | |
| *** trozet has quit IRC | 18:48 | |
| *** rbrady has quit IRC | 18:50 | |
| *** electrofelix has quit IRC | 18:58 | |
| *** trozet has joined #tripleo | 18:59 | |
| *** mburned has joined #tripleo | 19:04 | |
| *** eggmaste` has quit IRC | 19:07 | |
| *** yamahata has quit IRC | 19:07 | |
| *** pradk has joined #tripleo | 19:14 | |
| openstackgerrit | Dan Prince proposed openstack/instack-undercloud: Enable swift's bulk middleware https://review.openstack.org/264931 | 19:14 |
| *** cwolferh has joined #tripleo | 19:17 | |
| openstackgerrit | Ryan Hallisey proposed openstack/tripleo-common: Deploy Docker compute role with tripleo.sh https://review.openstack.org/264933 | 19:17 |
| openstackgerrit | Ryan Hallisey proposed openstack/tripleo-common: Update the container flag https://review.openstack.org/264934 | 19:18 |
| *** paramite is now known as paramite|afk | 19:19 | |
| rhallisey | shardy dprince, ^ those 2 should fix the gate in the stable/liberty branch | 19:19 |
| dprince | rhallisey: cool | 19:20 |
| rhallisey | lots of backports O.o | 19:21 |
| *** penick has joined #tripleo | 19:21 | |
| slagle | c | 19:22 |
| slagle | ww | 19:22 |
| shardy | rhallisey: nice, lgtm, just waiting on CI | 19:23 |
| rhallisey | only the second one should pass | 19:24 |
| rhallisey | for the container job that is | 19:24 |
| *** yamahata has joined #tripleo | 19:28 | |
| shardy | Easy +A for someone, passing CI: https://review.openstack.org/#/c/263669/ | 19:28 |
| shardy | and https://review.openstack.org/#/c/260144 | 19:29 |
| egafford | Seeing consistent "No valid host was found. There are not enough hosts available" in my virt env, consistently on my third overcloud test cycle (heat stack-delete, tripleo.sh --overcloud-deploy). Is there a known issue that would reasonably cause this, or should I spelunk? Not asking for research, just "Oh yeah, you have to juggle the quantum reverberators, everyone knows that." if such advice exists. | 19:33 |
| EmilienM | bnemec: I think https://review.openstack.org/#/c/244162 is critical to have it soon, lot of work depends on it I think | 19:34 |
| *** marios_ has joined #tripleo | 19:34 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-common: tripleo.sh support updating and checking deployed overcloud https://review.openstack.org/261077 | 19:34 |
| shardy | egafford: I'd check the nova hypervisor-stats after each delete, with particular attention on count and running_vms, along with related checks on the ironic node state via ironic node-list | 19:36 |
| shardy | egafford: beyond that, not sure as a few things can cause that error | 19:37 |
| *** eggmaster has joined #tripleo | 19:37 | |
| egafford | shardy: Yeah, that's pretty much "something is wrong on the internet." | 19:37 |
| egafford | It's just the regularity with which it pops up that makes me think it might be a known resource leak. | 19:37 |
| rhallisey | shardy, can the name field in a structured deployment be an environment variable? | 19:37 |
| rhallisey | like https://paste.fedoraproject.org/308327/19546314/ | 19:38 |
| shardy | egafford: Yeah, that's why you might glean some clues e.g if the stats are decrementing after each create/delete cycle | 19:38 |
| *** pradk has quit IRC | 19:38 | |
| *** eil397 has quit IRC | 19:38 | |
| *** Goneri has quit IRC | 19:38 | |
| *** kbyrne has quit IRC | 19:38 | |
| *** marios has quit IRC | 19:38 | |
| egafford | Yeah, not seeing any leaks in the hypervisor-stats. | 19:38 |
| shardy | rhallisey: you mean "name: {get_resource: DeploymentVersion}" ? | 19:39 |
| rhallisey | shadower, yes | 19:39 |
| rhallisey | is that legal? | 19:39 |
| rhallisey | shardy, ^ | 19:39 |
| *** pradk has joined #tripleo | 19:40 | |
| rhallisey | it's a string | 19:40 |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates: Enable the ML2 port security extension driver by default https://review.openstack.org/264940 | 19:40 |
| shardy | rhallisey: Yes, you can reference another resource called DeploymentVersion - the get_resource will set the name to the ID of that resource | 19:40 |
| shardy | I'm not sure exactly how that relates to environment variables tho | 19:40 |
| * shardy going to grab dinner, bbiab | 19:41 | |
| *** shardy is now known as shardy_afk | 19:41 | |
| *** kbyrne has joined #tripleo | 19:42 | |
| *** Goneri has joined #tripleo | 19:43 | |
| bnemec | EmilienM: It's merge conflicted and failing CI according to Jirka's last comment. What are you looking for from me? | 19:47 |
| EmilienM | bnemec: you're the patch owner, I was wondering if we make progress on it, otherwise I'm happy to help | 19:48 |
| *** lucasagomes is now known as lucas-dinner | 19:49 | |
| bnemec | EmilienM: Oh, right. I only proposed that because I proposed the original revert, and knew we would need to get this back in. I was under the impression you and dprince were driving it now. | 19:53 |
| EmilienM | bnemec: ok, will continue that way then | 19:53 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Manage keystone initialization directly in t-h-t manifests https://review.openstack.org/244162 | 19:55 |
| EmilienM | for the monitoring, it would be great to land it (undercloud) https://review.openstack.org/#/c/253709/ and https://review.openstack.org/#/c/252051/ - patches already reviewed, CI passing, no blocker. | 19:55 |
| EmilienM | thanks ^ | 19:55 |
| slagle | EmilienM: ci isn't passing on those. probably not related to your patches, but just sayin' | 20:02 |
| EmilienM | slagle: it pass non-ha | 20:02 |
| EmilienM | slagle: to me it's enough, since I'm patching undercloud. | 20:02 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: endpoint_map: bump nova api from 2.0 to 2.1 https://review.openstack.org/263366 | 20:02 |
| slagle | looks like they were both killed due to timeouts | 20:03 |
| slagle | maybe we're not using a cache somewhere | 20:03 |
| bnemec | slagle: I'm wondering if heat-engine is getting OOM'd on some of those timeouts. It looks like Heat just randomly stops at some point in the deployment process. | 20:07 |
| bnemec | And we do specifically recommend a bigger undercloud than I think we're using in CI. | 20:07 |
| beagles | rhallisey, did I read right that you are working on something that fixes consistent CI/gate failures on stable/liberty? | 20:08 |
| slagle | bnemec: could be. either that or it's the need 2 vcpu's thing | 20:09 |
| slagle | one of the failures is "ERROR: Timed out waiting for a reply to message ID 3ddc049ccc1243559cb6ac13546998f3" | 20:09 |
| rhallisey | beagles, sorry that was misleading. It's a fix for the container gate job | 20:09 |
| slagle | which the heat folks have said could be due to only 1 vcpu | 20:09 |
| beagles | rhallisey, ah okay | 20:09 |
| bnemec | slagle: Yeah, that also. We need to talk to derek about getting the undercloud vms sized up. | 20:09 |
| *** dhellmann has quit IRC | 20:11 | |
| *** ayoung has quit IRC | 20:19 | |
| beagles | shardy: is there anyone looking into the stable/liberty CI failures? | 20:21 |
| beagles | oh he's not here .. slagle, same question: ^^^ | 20:22 |
| *** openstackgerrit has quit IRC | 20:23 | |
| *** openstackgerrit has joined #tripleo | 20:24 | |
| *** gchamoul has quit IRC | 20:30 | |
| *** gchamoul has joined #tripleo | 20:32 | |
| slagle | beagles: yes, i submitted a revert to nova this morning | 20:32 |
| slagle | beagles: https://review.openstack.org/#/c/264793/ | 20:33 |
| *** dprince has quit IRC | 20:33 | |
| *** Marga_ has joined #tripleo | 20:35 | |
| *** Marga_ has quit IRC | 20:35 | |
| *** Marga_ has joined #tripleo | 20:35 | |
| *** ayoung has joined #tripleo | 20:39 | |
| *** CaptTofu___ is now known as CaptTofu | 20:51 | |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Add Management Network For System Administration. https://review.openstack.org/264963 | 20:58 |
| *** lblanchard1 has quit IRC | 21:00 | |
| openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates: Add Management Network For System Administration. https://review.openstack.org/264963 | 21:01 |
| *** jayg is now known as jayg|g0n3 | 21:02 | |
| *** rbrady has joined #tripleo | 21:04 | |
| openstackgerrit | Ethan Gafford proposed openstack/tripleo-heat-templates: Sahara Integration https://review.openstack.org/220863 | 21:05 |
| openstackgerrit | Ben Nemec proposed openstack/tripleo-heat-templates: Configure keystone public_endpoint https://review.openstack.org/264970 | 21:14 |
| openstackgerrit | Ben Nemec proposed openstack/python-tripleoclient: Ignore ssl warnings from requests https://review.openstack.org/264971 | 21:14 |
| bnemec | jaosorior: ^Needed to fix a couple of issues with ssl overcloud | 21:14 |
| bnemec | I'd be surprised if there aren't other issues, but since the overcloud is non-functional right now I can't test further. :-( | 21:15 |
| *** cwolferh has quit IRC | 21:18 | |
| *** penick has quit IRC | 21:20 | |
| shardy_afk | dsneddon: Hey I'm working up a quick example for generating a mapping file, do you need something that works on OSP7? | 21:30 |
| *** shardy_afk is now known as shardy | 21:30 | |
| shardy | or can I use new-for-liberty syntax? | 21:30 |
| dsneddon | shardy, Oh, boy, I didn't realize it had changed. I need to document both. | 21:30 |
| shardy | dsneddon: It hasn't changed, I can just use a slightly cleaner syntax post-liberty | 21:30 |
| shardy | I'll create an example which works on both | 21:31 |
| shardy | https://review.openstack.org/#/c/215116/5 | 21:31 |
| shardy | it's basically similar to that | 21:31 |
| shardy | I'll tweak the example and post it in a few mins | 21:31 |
| dsneddon | shardy, Great, thanks for your help | 21:31 |
| *** penick has joined #tripleo | 21:34 | |
| *** paramite|afk is now known as paramite | 21:43 | |
| *** cwolferh has joined #tripleo | 21:45 | |
| *** paramite is now known as paramite|afk | 21:48 | |
| *** eggmaster has quit IRC | 21:50 | |
| *** jaosorior has quit IRC | 21:50 | |
| *** slagle has quit IRC | 21:50 | |
| *** dtantsur|afk has quit IRC | 21:50 | |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-common: Use the new yaml syntax in CI https://review.openstack.org/254886 | 21:51 |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-common: Build image files from definitions in yaml https://review.openstack.org/235569 | 21:51 |
| *** eggmaster has joined #tripleo | 21:51 | |
| *** jaosorior has joined #tripleo | 21:51 | |
| *** slagle has joined #tripleo | 21:51 | |
| *** dtantsur|afk has joined #tripleo | 21:51 | |
| *** ayoung has quit IRC | 21:58 | |
| *** rpothier has quit IRC | 22:03 | |
| *** trown is now known as trown|outttypeww | 22:04 | |
| *** rhallisey has quit IRC | 22:06 | |
| *** Marga_ has quit IRC | 22:10 | |
| *** penick has quit IRC | 22:15 | |
| *** pradk has quit IRC | 22:24 | |
| *** absubram has quit IRC | 22:26 | |
| *** penick has joined #tripleo | 22:32 | |
| *** sthillma_ has joined #tripleo | 22:44 | |
| *** sthillma has quit IRC | 22:44 | |
| *** sthillma_ is now known as sthillma | 22:44 | |
| *** thrash is now known as thrash|g0ne | 22:45 | |
| *** jtomasek has quit IRC | 22:45 | |
| *** mkovacik has joined #tripleo | 23:05 | |
| *** mkovacik has quit IRC | 23:12 | |
| openstackgerrit | Ben Nemec proposed openstack/tripleo-docs: Document deploying the overcloud with ssl https://review.openstack.org/265006 | 23:14 |
| *** mkovacik has joined #tripleo | 23:30 | |
| openstackgerrit | Ethan Gafford proposed openstack/tripleo-heat-templates: Sahara Integration https://review.openstack.org/220863 | 23:35 |
| *** chem has quit IRC | 23:37 | |
| openstackgerrit | Steven Hardy proposed openstack/tripleo-heat-templates: Add example of creating per-node os-net-config mappings https://review.openstack.org/265014 | 23:38 |
| shardy | dsneddon: Hey, I'm going to have to call it a night, but there's a WIP patch | 23:38 |
| dsneddon | shardy, Much appreciated. G'night. | 23:39 |
| shardy | it's partially tested, but I'm having some issues getting the NodeConfigDataLookup parameter data into a format that survives the str_replace and ends up parsable by json.loads | 23:39 |
| shardy | dsneddon: so if you feel like fixing that and pushing another revision, feel free, otherwise I'll take another look tomorrow | 23:40 |
| shardy | also, I think there's a big gotcha with using a firstboot script | 23:40 |
| shardy | in that if you ever change NodeConfigDataLookup (for example to add new nodes), it will replace all the existing nodes, because the user-data changes | 23:40 |
| dsneddon | shardy, OK, I'm doing deployment after deployment testing IPv6, so I'll throw this in and see what pops out the other side. | 23:40 |
| shardy | I'm still thinking about the best way around that | 23:40 |
| shardy | dsneddon: I think you'll find it generates an empty mapping file, because the json is malformed when it hits the shell script | 23:41 |
| dsneddon | shardy, So under Liberty you would have used map_merge? | 23:41 |
| shardy | check out /var/log/cloud-init-output.log and you'll see the script which fails (you can then re-run it) | 23:42 |
| shardy | dsneddon: No, I can use a json parameter type, which is then transparently encoded as json via the str_replace | 23:42 |
| shardy | which is way, way easier than trying to hand-roll the json with all the escapes etc as a string | 23:43 |
| dsneddon | shardy, Is json parameter type really that new? I've gotten so used to it, I take it for granted. | 23:45 |
| shardy | http://docs.openstack.org/developer/heat/template_guide/hot_spec.html#str-replace | 23:45 |
| shardy | dsneddon: No, it's not the json parameter type, it's that str_replace didn't accept json parameters until the latest verion of HOT (for liberty) | 23:45 |
| shardy | before liberty, we'd throw a validation error saying the params for str_replace had to be a string | 23:45 |
| dsneddon | shardy, Right. Thanks for this, I'll play with it. I still think it's worth modifying os-net-config so we can give it one giant array of mac addresses and nic names. | 23:46 |
| shardy | since HOT version 2015-10-15 (liberty) we accept list/json parameters and json encode them to create strings | 23:46 |
| shardy | dsneddon: Yeah I agree it'd be worth modifying os-net-config, but we still have the problem of how to update the mapping before the network is up, which implies user-data | 23:47 |
| shardy | which means we have to figure out how to not rebuild the world if the mapping ever changes ;) | 23:47 |
| shardy | I'll give that further thought after some sleep, I have a few ideas | 23:47 |
| shardy | the challenges is finding one which may be backportable | 23:47 |
| dsneddon | shardy, To be continued... | 23:48 |
| shardy | dsneddon: yup, good luck, night! | 23:48 |
| *** ayoung has joined #tripleo | 23:48 | |
| *** shardy has quit IRC | 23:48 | |
| *** weshay_xchat has quit IRC | 23:49 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Adding ManagementIpSubnet to linux bridge net conf https://review.openstack.org/263669 | 23:58 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: updating enable_ceph conditions for controller https://review.openstack.org/263397 | 23:59 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Include the branch name in row descriptions https://review.openstack.org/259505 | 23:59 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Tweak the report colors a little https://review.openstack.org/259506 | 23:59 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Remove parse logs https://review.openstack.org/259507 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!