| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-06-13 | |||
| 20:51:04 | artom_ | dansmith, we do stuff with placement allocations, that's for sure | |
| 20:51:20 | dansmith | artom_: for the base resources, but not numa | |
| 20:51:28 | mriedem | we don't do claims for live migration | |
| 20:51:31 | dansmith | hence my comment on the later one | |
| 20:51:41 | artom_ | dansmith, right, and NUMA isn't ready in placement, so we're using RT for now (right?) | |
| 20:51:55 | dansmith | artom_: RT for numa, and not for live migration | |
| 20:51:57 | mriedem | https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L74 | |
| 20:53:21 | jmlowe | uh oh, seems I broke horizon as well as nova cli | |
| 20:54:03 | artom | We definitely create a Migration, nova/conductor/manager.py L404, but I didn't find any MigrationContext | |
| 20:54:07 | mriedem | i want to make a 'damn it jim' joke | |
| 20:54:14 | mriedem | artom: the migratoin context is in the RT | |
| 20:54:17 | mriedem | which doesn't track live migratoins | |
| 20:54:17 | jroll | damn it matt | |
| 20:54:28 | artom | And the Migration only has the basic stuff in it, source, dest, etc | |
| 20:54:43 | mriedem | we also have the migration record for tracking allocations in placement during the move | |
| 20:54:48 | mriedem | so they aren't doubled on the instance | |
| 20:54:50 | artom | Also, MigrateData was specifically created to include virt-specific stuff, which this kinda is | |
| 20:55:20 | mriedem | migrate data is definitely for, and only used by, live migration | |
| 20:55:20 | dansmith | artom: it was yeah | |
| 20:55:35 | dansmith | artom: I think you're fine on this, I just wasn't thnking | |
| 20:55:43 | artom | dansmith, ooof, thanks :) | |
| 20:55:47 | mriedem | artom: we're all counting on you, good luck | |
| 20:56:04 | artom | mriedem, only if you review it ^_^ | |
| 20:57:01 | artom | mriedem, so wait, how do I go about claiming the NUMA resources on the dest? Does RT need to start tracking LM now? | |
| 20:57:24 | artom | Or can claims be done "independently", so to speak? | |
| 20:57:40 | mriedem | having never read that spec yet, i defer | |
| 20:58:05 | dansmith | artom: you don't want to go down that rabbit hole | |
| 20:58:06 | dansmith | take the other pill | |
| 20:58:16 | artom | mriedem, heh, fair about, I think hay and sylvain are on the hook for that, maybe stephen | |
| 20:58:20 | artom | *fair enough | |
| 20:58:22 | mriedem | i honestly don't know what the numa resource claim code looks like | |
| 20:58:23 | dansmith | artom: IMHO, get your stuff working without claims, like everything else does currently related to live, and then point at placement as the gap that will fix it | |
| 20:58:25 | mriedem | i intentionally avoid that mess | |
| 20:58:36 | artom | dansmith, it does seem like a massive rabbit hole | |
| 20:58:42 | mriedem | artom: stephenfin said he was waiting for you to do numa live migration for his numa networks stuff | |
| 20:59:03 | artom | Outright waiting? I understood it as he'll get his done, minus the working live migration stuff | |
| 20:59:09 | mriedem | no, i mean, | |
| 20:59:15 | dansmith | artom: waiting and also looking fairly impatient | |
| 20:59:21 | dansmith | artom: like a bus is late | |
| 20:59:22 | mriedem | numa networks just won't work with live migration, because numa doesn't work with live migration | |
| 20:59:26 | mriedem | s/waiting/punting/ | |
| 20:59:35 | artom | mriedem, great, more pressure | |
| 20:59:50 | artom | No wonder no one wanted to touch this for years | |
| 20:59:53 | artom | ;) | |
| 21:00:03 | mriedem | https://www.youtube.com/watch?v=SmHeP9Sve48 | |
| 21:00:10 | dansmith | I did hear him say that if only we had heartbeating of RPC calls, then other stuff would be a lot better and he could get back to work | |
| 21:00:16 | dansmith | dunno if that's true, but sounded really good | |
| 21:00:37 | mriedem | i did my part | |
| 21:00:55 | artom | ... is it worrying that I have no idea what "heartbeating of RPC calls" means? | |
| 21:01:11 | mriedem | EVERYONE has been talking about it | |
| 21:01:19 | mriedem | it's like the new containers | |
| 21:01:28 | dansmith | it slices | |
| 21:01:29 | dansmith | it dices | |
| 21:01:35 | mriedem | it...pings? | |
| 21:01:38 | dansmith | it does the kids' homework | |
| 21:01:41 | dansmith | heh | |
| 21:02:00 | artom | This is high school all over again :( | |
| 21:02:01 | mriedem | artom: https://review.openstack.org/#/c/566696/ | |
| 21:02:13 | dansmith | artom: mildly live-migration-related :P | |
| 21:02:21 | jmlowe | so, about that url http(s) problem, seems to be fixed with a trailing / | |
| 21:02:54 | artom | Ah, I see | |
| 21:07:17 | jmlowe | Is that worth filing a documentation bug? If you are using ssl then you need a trailing slash on your nova endpoint url | |
| 21:08:38 | mgagne | you do?! | |
| 21:14:54 | dansmith | mriedem: did that tempest test get reverted? | |
| 21:15:06 | dansmith | just noticing a lot of nova things in check have failing tests | |
| 21:15:12 | mriedem | dansmith: the patch is up | |
| 21:15:12 | dansmith | which is probably not helping load | |
| 21:15:21 | dansmith | right, but unmerged? | |
| 21:15:24 | mriedem | http://status.openstack.org/elastic-recheck/#1776684 | |
| 21:15:26 | melwitt | fwiw, I found this old thread about trailing slash, not sure if it's the same thing http://lists.openstack.org/pipermail/openstack-dev/2016-March/090143.html | |
| 21:15:27 | dansmith | it's now 2h15m to get running | |
| 21:15:30 | mriedem | correct https://review.openstack.org/#/c/575132 | |
| 21:15:53 | dansmith | wonder if we could poke someone to slam that in? | |
| 21:16:18 | mriedem | i pinged mtreinish and gmann when i posted it in -qa | |
| 21:16:23 | mriedem | but gmann won't be around for awhile | |
| 21:16:27 | mriedem | ken'ichi is qa core | |
| 21:16:33 | dansmith | okay | |
| 21:16:38 | mriedem | but is asking for a config option, which i don't really think should hold up the revert | |
| 21:16:47 | dansmith | maybe melwitt can take that torch, I'm gonna disappear soon | |
| 21:20:04 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Ignore UserWarning for scope checks during test runs https://review.openstack.org/575237 | |
| 21:20:05 | mriedem | this is for cdent ^ | |
| 21:20:56 | mriedem | dansmith: you could also comment on or +1 the revert | |
| 21:21:04 | dansmith | I thought I did | |
| 21:21:26 | mriedem | call sdague out of the bullpen | |
| 21:23:43 | melwitt | which torch? this tempest revert? | |
| 21:23:51 | dansmith | melwitt: yeah | |
| 21:23:56 | melwitt | ack | |
| 22:01:15 | tonyb | Should we wait on the | |
| 22:01:59 | tonyb | Revert "Add server name check for multiple creations" to merge or just balcklist the test in the short term to unblock the queens gate? | |
| 22:19:07 | openstackgerrit | Matt Riedemann proposed openstack/nova master: placement: Allocation.consumer field https://review.openstack.org/565405 | |
| 22:19:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407 | |
| 22:19:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | |
| 22:19:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix nits from change Id609789ef6b4a4c745550cde80dd49cabe03869a https://review.openstack.org/574324 | |
| 22:20:36 | tonyb | dansmith, mriedem I'll nag gmann in a bit when he's online | |
| 22:21:50 | mriedem | thanks | |
| 22:22:52 | melwitt | tonyb, mriedem: that's an idea ... we do have a special test rc file for the cells v1 job IIRC. and it's affecting master too right, not just queens? | |
| 22:23:30 | tonyb | melwitt: Yeah it's probably all branches you're right | |
| 22:23:52 | melwitt | https://github.com/openstack/nova/blob/master/devstack/tempest-dsvm-cells-rc | |
| 22:24:12 | tonyb | melwitt: so that's less appealing as we'd need to merge and revert is on (potentailly) 4 branches :( | |
| 22:24:31 | melwitt | yeah | |