| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-07-25 | |||
| 12:16:22 | aspiers | ah, got removed again | |
| 12:16:44 | sean-k-mooney | ya because it was in merge conflict | |
| 12:16:52 | aspiers | not when I added it | |
| 12:17:17 | sean-k-mooney | sure but it went into merge conflict when you were on vaction | |
| 12:17:21 | aspiers | yes | |
| 12:17:30 | aspiers | at least that bit wasn't my fault ;-) | |
| 12:17:45 | sean-k-mooney | :) damb cores merging code | |
| 12:17:54 | sean-k-mooney | its almost as if its part of there jobs | |
| 12:18:17 | aspiers | I know, disgraceful behaviour | |
| 12:21:45 | sean-k-mooney | ok, shower, lunch and then back before a meeting at 2. that should be totally doable... | |
| 12:21:58 | aspiers | good luck | |
| 12:24:01 | mriedem | brinzhang: https://review.opendev.org/#/c/612949/10 | |
| 12:24:06 | mriedem | alex_xu: ^ | |
| 12:24:36 | mriedem | i think what i've described as the main proposed change (in my comments) makes sense, but the spec overall is a bit confusing since it seems to mix in lots of different things | |
| 12:24:53 | mriedem | if the spec can be cleaned up in time for the spec freeze i'll probably be +2 on it | |
| 12:25:49 | mriedem | i do appreciate that the entire dev team from inspur has +1ed the spec without leaving any comments though :) | |
| 12:26:19 | aspiers | mriedem: since I'm back from vacation and have got the SEV series back to (AFAICS) 100% health, I've readded to the runway | |
| 12:26:23 | mriedem | within like 5 minutes of each other :) | |
| 12:26:45 | mriedem | aspiers: ok | |
| 12:27:02 | openstackgerrit | Daniel Speichert proposed openstack/nova-specs master: Directly download and upload images to RBD https://review.opendev.org/658903 | |
| 12:55:18 | aspiers | sean-k-mooney: almost got the PS ready | |
| 12:56:09 | efried | aspiers: also, it's generally discouraged to put a series in a runway slot if the devs are going to be on vacation or otherwise unavailable during the window to address comments (and merge conflicts). | |
| 12:57:03 | efried | see last bullet under "Requirements for being eligible..." | |
| 12:57:07 | aspiers | efried: ack. IIRC it was far back in the queue so didn't seem to have much chance of getting addressed while I was away | |
| 12:57:11 | aspiers | but maybe I miscalculated | |
| 12:57:49 | aspiers | also, I was expecting a colleague to be available while I was away | |
| 12:58:06 | aspiers | but I think he got busy with other commitments | |
| 12:58:09 | efried | aspiers: What should have happened is whoever actually moved it from the queue to a slot should have asked about dev availability. | |
| 12:58:18 | aspiers | life happens ... | |
| 12:58:19 | efried | anyway, nbd | |
| 12:58:41 | aspiers | I'm here now anyway :) | |
| 12:58:46 | aspiers | and it's all looking pretty good | |
| 12:59:11 | efried | it's not like nuclear missile launches will or will not happen based on how tightly we run our runway queue. | |
| 12:59:42 | aspiers | they won't? :-o | |
| 12:59:53 | aspiers | damn, I'm in the wrong business | |
| 13:01:08 | cdent | efried: speak for yourself efried, I run my small republic's military on the comings and goings of openstack etherpads | |
| 13:01:23 | cdent | and we have da bomb | |
| 13:01:25 | aspiers | I was only in OpenStack because I thought I was helping hurry the apocalypse along | |
| 13:01:32 | cdent | _exactly_ | |
| 13:01:54 | aspiers | cdent: is it hotter than hell where you are too? | |
| 13:02:08 | aspiers | maybe the apocalypse has already started | |
| 13:02:22 | cdent | aspiers: no. it's hotter than normal, but since I'm in corwall by the sea, it's a balmy and breezy 24 | |
| 13:02:34 | efried | It's unseasonably cool here in central texas. | |
| 13:02:35 | cdent | so apparently the apocolypse is upcountry, which make sense | |
| 13:02:38 | aspiers | bah | |
| 13:02:53 | aspiers | I guess London deserves the wrath of satan more than the rest of the country | |
| 13:02:56 | cdent | yes | |
| 13:03:08 | aspiers | s/guess/know/ | |
| 13:23:32 | openstackgerrit | Adam Spiers proposed openstack/nova master: libvirt: harden Host.get_domain_capabilities() https://review.opendev.org/670189 | |
| 13:24:39 | efried | stephenfin: I'm not sure what cores are able to review --^ but I thought you might be one of them? | |
| 13:24:43 | efried | I know it's not me :( | |
| 13:25:22 | aspiers | sean-k-mooney, kashyap: new version up ^^^ | |
| 13:29:28 | openstackgerrit | Adam Spiers proposed openstack/nova master: libvirt: harden Host.get_domain_capabilities() https://review.opendev.org/670189 | |
| 13:29:29 | aspiers | kashyap: ah, just saw your nits - addressed | |
| 13:30:42 | mriedem | efried: fyi i'll be running my kid to a class thing during the meeting so won't be around | |
| 13:31:38 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Revert "[libvirt] Filter hypervisor_type by virt_type" https://review.opendev.org/672723 | |
| 13:35:37 | kashyap | aspiers: Thanks :-) | |
| 13:35:45 | efried | meeting, right, meeting. | |
| 13:42:43 | stephenfin | efried: It's on my list. Got to get to the combined VCPU/PCPU instances spec first though | |
| 13:43:39 | openstackgerrit | Merged openstack/nova master: Remove super old unnecessary TODO from API start() method https://review.opendev.org/672330 | |
| 13:43:47 | openstackgerrit | Merged openstack/nova master: Completely remove fake_libvirt_utils. https://review.opendev.org/643897 | |
| 13:43:55 | openstackgerrit | Merged openstack/nova master: Remove usused umask argument to virt.libvirt.utils.write_to_file https://review.opendev.org/645086 | |
| 13:53:44 | efried | thanks stephenfin, specs are much more important rn. | |
| 13:58:51 | sean-k-mooney | efried: cob today is spec freeze right | |
| 13:59:11 | efried | sean-k-mooney: Yes. http://lists.openstack.org/pipermail/openstack-discuss/2019-July/008019.html | |
| 14:00:06 | sean-k-mooney | yep read that yesterday but its been a busy few days | |
| 14:00:20 | dansmith | sean-k-mooney: it's been a few days since yesterday? | |
| 14:00:49 | efried | nova meeting now in -meeting | |
| 14:01:06 | openstackgerrit | Merged openstack/nova master: Revert "[libvirt] Filter hypervisor_type by virt_type" https://review.opendev.org/672559 | |
| 14:01:13 | sean-k-mooney | no just ingeneral i was traviling form cashel to shannon after spending a few days at my mothers since her car broke down | |
| 14:01:14 | openstackgerrit | Merged openstack/nova master: Consts for need_healing https://review.opendev.org/672284 | |
| 14:01:22 | openstackgerrit | Merged openstack/nova master: Fix cleaning up console tokens https://review.opendev.org/637716 | |
| 14:01:29 | openstackgerrit | Merged openstack/nova master: Disambiguate logs in delete_allocation_for_instance https://review.opendev.org/671869 | |
| 14:01:37 | openstackgerrit | Merged openstack/nova master: Remove old TODO about forced_host policy check https://review.opendev.org/669474 | |
| 14:01:43 | dansmith | sean-k-mooney: heh, it was just a funny mind-o highlighting the business (i.e. two days felt like three) | |
| 14:01:44 | openstackgerrit | Merged openstack/nova master: Avoid logging traceback when detach device not found https://review.opendev.org/671640 | |
| 14:01:50 | dansmith | er, busy-ness | |
| 14:02:12 | sean-k-mooney | ha ya | |
| 14:05:00 | sean-k-mooney | speaking of specs i agree that https://review.opendev.org/#/c/608696/ and https://review.opendev.org/#/c/602201/ are the closest of the remaing set | |
| 14:35:09 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Avoid crashing while getting libvirt capabilities with unknown arch names https://review.opendev.org/672746 | |
| 14:35:09 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Revert "[libvirt] Filter hypervisor_type by virt_type" https://review.opendev.org/672747 | |
| 14:40:46 | artom | Did something change with placement in functional tests recently? I swear my NUMA LM func test was getting to updating the XML last night, but this morning it's failing on placement not giving any allocations | |
| 14:42:51 | cdent | artom you have an example of the query that's being made, that might clarify things | |
| 14:43:19 | cdent | just today some cache adjustments were merged. when did things last work? | |
| 14:44:11 | artom | Last night (EDT) | |
| 14:44:13 | artom | 27.0.0.1 "GET /placement/allocation_candidates?limit=1000&required=%21COMPUTE_STATUS_DISABLED&resources=DISK_GB%3A20%2CMEMORY_MB%3A2048%2CVCPU%3A3" status: 200 len: 53 microversion: 1.31 | |
| 14:44:27 | artom | Got no allocation candidates from the Placement API. This could be due to insufficient resources or a temporary occurrence as compute nodes start up. | |
| 14:45:53 | artom | (Which I guess makes even less sense - if it was the same as before, what changed?) | |
| 14:47:52 | cdent | artom: got a patch up so I can play with it myself? | |
| 14:48:47 | cdent | when did compute status disabled support merge in nova? | |
| 14:48:53 | artom | cdent, https://review.opendev.org/#/c/672595, but I had to rebase it locally, so hang on | |
| 14:48:58 | cdent | aye aye | |
| 14:49:01 | mriedem | cdent: last week i think | |
| 14:49:04 | mriedem | or 2 weeks ago | |
| 14:51:42 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: [WIP-until-series-is-ready] Introduce live_migration_claim() https://review.opendev.org/635669 | |
| 14:51:43 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: New objects for NUMA live migration https://review.opendev.org/634827 | |
| 14:51:43 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: LM: add support for sending NUMAMigrateData to the source https://review.opendev.org/634828 | |
| 14:51:44 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: LM: add support for updating NUMA-related XML on the source https://review.opendev.org/635229 | |
| 14:51:44 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: RPC changes to prepare for NUMA live migration https://review.opendev.org/634605 | |
| 14:51:45 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: NUMA live migration support https://review.opendev.org/634606 | |