| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-04-05 | |||
| 17:47:31 | dansmith | yeah, you understand that I understand, right? I just wanted it to be updated so I can use my aliases, which all specify the version | |
| 17:47:48 | sean-k-mooney | yep | |
| 17:47:56 | sean-k-mooney | no issue with the patch | |
| 17:48:07 | sean-k-mooney | i just avoid using the versioned ones ot not have that problem | |
| 17:48:35 | clarkb | sean-k-mooney: I don't know if they will update the full version in jammy | |
| 17:48:37 | clarkb | they might | |
| 17:49:16 | sean-k-mooney | the yave in the past (not the default) but ya not sure | |
| 17:50:04 | sean-k-mooney | i dont really mind using debian instead for the unit/functional tests if that is what makes sense | |
| 17:50:25 | sean-k-mooney | is 3.11 in the PTI runtimes this release | |
| 17:50:47 | sean-k-mooney | looks like no | |
| 17:51:24 | sean-k-mooney | https://github.com/openstack/governance/blob/master/reference/runtimes/2023.2.rst but it would still be nice to have some coverage | |
| 17:55:10 | clarkb | its also just faster ime. On tumbleweed locally it is 10% quicker on my machine to run zuul's unittests under python3.11 + x86_64-v3 than python3.10 without x86_64-v3 (can't be sure how much of that is python version vs cpu stuff though) | |
| 17:56:10 | sean-k-mooney | tumbleweed is compliing for x86_64-v3 | |
| 17:56:18 | sean-k-mooney | i think rhel 9 just moved to -v1 | |
| 17:57:20 | sean-k-mooney | oh actully v2 https://developers.redhat.com/blog/2021/01/05/building-red-hat-enterprise-linux-9-for-the-x86-64-v2-microarchitecture-level | |
| 17:58:19 | sean-k-mooney | v3 gets avx2 which is nice but i woudl be surpised if that impacted unit tests | |
| 17:58:58 | sean-k-mooney | perhaps it impoved the hash funciton for dictonaries but either way 10% is nice | |
| 18:03:16 | clarkb | sean-k-mooney: they are doing overlay packages that do ldpreload magic or something | |
| 18:03:32 | clarkb | basically old cpu support remains then zypper detects if you've got a newer cpu and it will automaticall install the overlay packages too | |
| 18:03:53 | sean-k-mooney | ah ok so they are building for v3 and also for v2 or older | |
| 18:04:07 | sean-k-mooney | and using ldpreload to load the correct lib based on cpu support | |
| 18:04:36 | sean-k-mooney | ya i think rhel considerd that and tought it was too much work :) | |
| 19:15:06 | opendevreview | Merged openstack/nova master: Allow running functional-py311 https://review.opendev.org/c/openstack/nova/+/879559 | |
| 19:39:52 | opendevreview | Dan Smith proposed openstack/nova master: Add compute_id columns to instances, migrations https://review.opendev.org/c/openstack/nova/+/879499 | |
| 19:39:53 | opendevreview | Dan Smith proposed openstack/nova master: Add compute_id to Instance object https://review.opendev.org/c/openstack/nova/+/879500 | |
| 19:39:53 | opendevreview | Dan Smith proposed openstack/nova master: Add dest_compute_id to Migration object https://review.opendev.org/c/openstack/nova/+/879682 | |
| 21:29:40 | opendevreview | Dan Smith proposed openstack/nova master: DNM testing grenade object versions https://review.opendev.org/c/openstack/nova/+/879687 | |
| 21:30:52 | opendevreview | Dan Smith proposed openstack/nova master: DNM testing grenade object versions https://review.opendev.org/c/openstack/nova/+/879687 | |
| 21:36:16 | opendevreview | Dan Smith proposed openstack/nova master: DNM testing grenade object versions https://review.opendev.org/c/openstack/nova/+/879687 | |
| 21:37:07 | opendevreview | Dan Smith proposed openstack/nova master: DNM testing grenade object versions https://review.opendev.org/c/openstack/nova/+/879687 | |
| 23:37:09 | opendevreview | melanie witt proposed openstack/nova master: Support rescue with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/873675 | |
| 23:37:10 | opendevreview | melanie witt proposed openstack/nova master: Add reset_encryption_fields() and save_all() to BlockDeviceMappingList https://review.opendev.org/c/openstack/nova/+/870938 | |
| 23:37:10 | opendevreview | melanie witt proposed openstack/nova master: Update driver BDMs with ephemeral encryption image properties https://review.opendev.org/c/openstack/nova/+/870939 | |
| 23:37:11 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273 | |
| #openstack-nova - 2023-04-06 | |||
| 08:29:01 | opendevreview | Konrad Gube proposed openstack/nova-specs master: Re-propose using extend volume completion action for 2023.2 https://review.opendev.org/c/openstack/nova-specs/+/877233 | |
| 10:43:26 | opendevreview | Merged openstack/nova stable/xena: db: Resolve additional SAWarning warnings https://review.opendev.org/c/openstack/nova/+/874066 | |
| 12:44:04 | opendevreview | Merged openstack/nova stable/wallaby: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/821978 | |
| 13:19:36 | opendevreview | ribaudr proposed openstack/nova stable/yoga: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/879740 | |
| 13:38:24 | opendevreview | ribaudr proposed openstack/nova stable/yoga: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/879740 | |
| 13:42:56 | opendevreview | ribaudr proposed openstack/nova stable/yoga: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/879701 | |
| 13:45:49 | opendevreview | ribaudr proposed openstack/nova stable/ussuri: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/879702 | |
| 13:51:12 | opendevreview | ribaudr proposed openstack/nova stable/xena: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/879703 | |
| 13:52:18 | opendevreview | ribaudr proposed openstack/nova stable/wallaby: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/879704 | |
| 13:52:54 | opendevreview | ribaudr proposed openstack/nova stable/victoria: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/879705 | |
| 13:54:01 | opendevreview | ribaudr proposed openstack/nova stable/train: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/879706 | |
| 13:58:57 | opendevreview | liang jiechao proposed openstack/nova-specs master: Generic vdpa spec https://review.opendev.org/c/openstack/nova-specs/+/879338 | |
| 14:07:00 | stephenfin | bauzas: Think you could hit this up https://review.opendev.org/c/openstack/nova/+/872429/ ? | |
| 14:22:29 | opendevreview | Dan Smith proposed openstack/nova-specs master: Add compute-object-ids spec for 2023.2 https://review.opendev.org/c/openstack/nova-specs/+/877291 | |
| 14:22:44 | dansmith | sean-k-mooney: added a bunch more meat to this ^ around migration handling | |
| 14:23:08 | dansmith | as I'm working through the actual implementation I'm finding that we throw node names around more than I was really thinking, because of the migrations | |
| 14:46:12 | dansmith | sean-k-mooney: also, see my comment here: | |
| 14:46:13 | dansmith | https://review.opendev.org/c/openstack/nova/+/878238 | |
| 14:46:36 | dansmith | I don't want to recheck that and make it merge if you disagree, but.. I'm pretty sure that's not related since that same failure has been happening a lot | |
| 16:26:30 | opendevreview | Dan Smith proposed openstack/nova-specs master: Add compute-object-ids spec for 2023.2 https://review.opendev.org/c/openstack/nova-specs/+/877291 | |
| 16:49:52 | opendevreview | Merged openstack/nova stable/yoga: Reproduce live migration rollback w/o multi port bindings error https://review.opendev.org/c/openstack/nova/+/839921 | |
| 17:13:16 | sean-k-mooney | dansmith: ill be aroudn for a litle bit but before i finsih for the long weekend i wante to checkback with you for the laze load patch | |
| 17:13:20 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/878238 | |
| 17:13:46 | sean-k-mooney | you mentioned you see the same failure in other runs | |
| 17:14:33 | sean-k-mooney | given that its proably unrelated just looks suspcious given what you were updateing | |
| 17:14:46 | sean-k-mooney | i cna read +2 and recheck it | |
| 17:15:01 | sean-k-mooney | but i was wondering if you had filed a gate bz or anything for this | |
| 17:15:10 | sean-k-mooney | *gate bug | |
| 17:15:26 | dansmith | no, I haven't noticed it before this, I just went looking to see if it was really new here | |
| 17:15:41 | dansmith | I'm not sure why we'd see it for scheduler and not conductor, which already added the singleton infra | |
| 17:15:44 | sean-k-mooney | so i think i have seen this before but not consitently | |
| 17:16:07 | sean-k-mooney | well the base class is also alredy resting the singelton | |
| 17:16:17 | sean-k-mooney | all you changed is doing it via the property | |
| 17:16:28 | sean-k-mooney | which is even more strange | |
| 17:16:48 | sean-k-mooney | like i dont know why calling the singelton later would chagne the behavior | |
| 17:16:52 | dansmith | ...right | |
| 17:19:44 | sean-k-mooney | so the hits you found seam to be for 878763, 875621, 879500 and now 878238 | |
| 17:30:00 | sean-k-mooney | dansmith: so other then the fact that two of these were submited by you id dont see anything quicly that woudl make me think this is specific to your change | |
| 17:30:19 | sean-k-mooney | so i think this is an exting issue so we can proceed with the lazy loading | |
| 17:31:02 | sean-k-mooney | the first hit in the lst 30 days was Mar 28, 2023 @ 12:58:01.786 | |
| 17:31:11 | sean-k-mooney | so maybe we could see what merged just before that | |
| 17:31:49 | dansmith | but we're also not hitting them very often, so I'm not sure looking at temporal merges is likely to be very fruitful | |
| 17:32:02 | opendevreview | Dan Smith proposed openstack/nova master: Add compute_id to Instance object https://review.opendev.org/c/openstack/nova/+/879500 | |
| 17:32:34 | sean-k-mooney | fair i guess we can just keep an eye out and see if this starts happening more often | |
| 17:32:34 | dansmith | sean-k-mooney: based on this I'm assuming you didn't see my ping about it above, which means you likely didn't see my ping before that about the spec? | |
| 17:33:11 | sean-k-mooney | correct but scrolling up i see them | |
| 17:33:22 | dansmith | no rush, so don't worry about it today | |
| 17:33:37 | dansmith | just fyi for tomorrow or next week | |
| 17:35:09 | sean-k-mooney | ya proably next week | |
| 17:35:24 | sean-k-mooney | skiming it i did not think about migration being affect by this durign the ptg | |
| 17:35:39 | sean-k-mooney | but ya it would make sesne if we are addign the srive_id to the instace | |
| 17:35:39 | dansmith | me either, but we update host and node from migration a LOT | |
| 17:36:04 | sean-k-mooney | before the ptg i was not expectign to update the instnace in this cycle | |
| 17:36:16 | dansmith | my patch actually makes us do that less, updating it to be "this node here" instead of "whatever the migration said" in several places | |
| 17:36:23 | sean-k-mooney | i assumed it was just going ot be the compute but there is value in doing both | |
| 17:36:39 | sean-k-mooney | ack | |
| 17:37:18 | sean-k-mooney | ya so jira has eaten all my brin power so im not digesting the spec reading it | |
| 17:37:34 | sean-k-mooney | so ya ill take a look on tueday and just assume your right :) | |
| 17:38:01 | sean-k-mooney | i did see you had started the work on it | |
| 17:38:04 | dansmith | yep, np | |
| 17:38:21 | dansmith | as noted in the ptg, | |
| 17:38:36 | dansmith | I think we should do more developing the code and spec in parallel, after initial sign-off on the idea | |
| 17:38:45 | dansmith | so I'm trying to keep my spec updated with the wrinkles I'm finding | |
| 17:38:59 | dansmith | and just hope that bauzas will approve me before the spec lands :) | |
| 17:42:20 | sean-k-mooney | ill reply on artom patch | |
| 17:42:57 | sean-k-mooney | but if i recall corectly its because this is invoked in the perodic task asn we are trying to avoid loading all of the instnace numa toplogy blob form the data base | |