| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-04-05 | |||
| 17:43:23 | dansmith | *one | |
| 17:43:34 | clarkb | dansmith: tox lying is one of the things I don't like about it. Nox is a lot more explicit | |
| 17:43:54 | dansmith | clarkb: I know, I've never understood that behavior | |
| 17:44:19 | sean-k-mooney | well its because we removed the config that froce it to use the specifed version since it "fixed" | |
| 17:44:32 | sean-k-mooney | dansmith: were you geting a non default python version locally | |
| 17:44:33 | clarkb | re python3.11 we're successfully using the rc package on jammy for unittests in zuul then rely on functional testing with python:3.11-bullseye based images to sanity check it | |
| 17:44:38 | sean-k-mooney | i.e. something other then 3.11? | |
| 17:45:10 | dansmith | no | |
| 17:45:27 | sean-k-mooney | clarkb: good to know but we should get teh full released version in like 2 or 3 weeks in 22.04 right? | |
| 17:45:28 | dansmith | sean-k-mooney: run this: 'tox -esnarglepuss9000' | |
| 17:46:03 | sean-k-mooney | ok that should create a default env sicne it wont match any of our default ones | |
| 17:46:10 | sean-k-mooney | and run whatever we have as the default | |
| 17:46:13 | dansmith | tox -e'failopotomus2000' | |
| 17:46:18 | dansmith | exactly | |
| 17:46:22 | dansmith | as does functional-py311 | |
| 17:46:38 | sean-k-mooney | yes that more or less what i expect | |
| 17:46:45 | sean-k-mooney | but if you did tox -e functional | |
| 17:46:49 | sean-k-mooney | it would have used 3.11 | |
| 17:46:55 | sean-k-mooney | if that is your system default | |
| 17:47:08 | sean-k-mooney | i know its not intuitive | |
| 17:47:28 | sean-k-mooney | but that was the behvior i observed when we looked at using the generitive envs | |
| 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? | |