| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-07-14 | |||
| 11:42:11 | sean-k-mooney | oh we already have | |
| 11:42:38 | sean-k-mooney | spwan is now a call to spawn on with the default executor | |
| 11:43:24 | sean-k-mooney | but i bet the futurist executor might be using spwan_n somewhere along the way | |
| 11:44:32 | sean-k-mooney | we have a hacking check to prevent raw usage of spawn and spwan_n https://github.com/openstack/nova/commit/6179854ae77960d4ae883349626069023b407cb0 | |
| 11:45:22 | sean-k-mooney | well and eventlet in genreal so any usage is likely not in nova | |
| 11:46:01 | sean-k-mooney | gibi: yep https://github.com/openstack/futurist/blob/6f870896819197ca323b3dd8af8f3b0389aff69c/futurist/_futures.py#L718 | |
| 11:46:55 | sean-k-mooney | gibi: so the GreenThreadPoolExecotor uses spwan_n whne it spins up new workers | |
| 11:47:12 | gibi | I will be so happy when we finally delete eventlet :) | |
| 11:47:42 | sean-k-mooney | hehe so that litrally made me lol | |
| 11:47:48 | sean-k-mooney | but i agree | |
| 11:48:33 | sean-k-mooney | gibi: that can help you create a test case but i assume you already have one now | |
| 11:53:50 | opendevreview | sean mooney proposed openstack/nova stable/2025.2: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/997028 | |
| 11:54:30 | opendevreview | sean mooney proposed openstack/nova stable/2025.1: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/997029 | |
| 12:02:42 | gibi | sean-k-mooney: here is the repro and the fix https://review.opendev.org/c/openstack/oslo.concurrency/+/997200 | |
| 12:05:18 | sean-k-mooney | oh thats the fix :) | |
| 12:05:25 | sean-k-mooney | i was like wait they alrady have a bug for this | |
| 12:05:50 | sean-k-mooney | i.e. reading "# This is bug https://bugs.launchpad.net/oslo.concurrency/+bug/2160596" | |
| 12:06:59 | gibi | :) | |
| 12:07:10 | gibi | sean-k-mooney: thanks for the help root causing this issue | |
| 12:07:47 | sean-k-mooney | no worries i wrote the test that was beign broken so i was interested in the resolution | |
| 12:08:05 | sean-k-mooney | i tried really hard to make the FairLockGuard robust | |
| 12:08:22 | sean-k-mooney | so i was concerned i missed somthing | |
| 12:08:43 | gibi | sean-k-mooney: yeah all that effort is in vain if eventlet breaks you | |
| 12:17:59 | sean-k-mooney | minior nit in the commit messge of the second patch | |
| 12:18:07 | sean-k-mooney | but the logic in both looks good to me | |
| 12:18:50 | gibi | fixed | |
| 12:20:20 | sean-k-mooney | :) | |
| 12:59:39 | opendevreview | Joan Gilabert proposed openstack/nova master: Report OWNER_NOVA trait on resource providers https://review.opendev.org/c/openstack/nova/+/994299 | |
| 12:59:39 | opendevreview | Joan Gilabert proposed openstack/nova master: Add libvirt driver MDEV ARQ support for Cyborg https://review.opendev.org/c/openstack/nova/+/995840 | |
| 13:10:55 | opendevreview | Merged openstack/nova master: Reproducer for https://bugs.launchpad.net/nova/+bug/2160224 https://review.opendev.org/c/openstack/nova/+/996756 | |
| 13:49:48 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not try to return empty attachment_id from API https://review.opendev.org/c/openstack/nova/+/996757 | |
| 13:52:37 | gibi | gmaan: sean-k-mooney: ^^ this is now ready to land ;) | |
| 13:53:38 | gibi | sean-k-mooney: also if you have a sec we have this small fix https://review.opendev.org/c/openstack/nova/+/996757/1 | |
| 13:54:01 | sean-k-mooney | isnt that te same link | |
| 13:54:02 | gibi | then when these are landed I will propose a patch to make the functional-py313-threading patch as voting | |
| 13:54:31 | gibi | https://review.opendev.org/c/openstack/nova/+/996793 this is the correct link for the small fix | |
| 13:54:39 | sean-k-mooney | :) | |
| 13:54:45 | gibi | (copy paste is hard as you see) | |
| 13:54:46 | sean-k-mooney | sure ill look at both now | |
| 13:54:49 | gibi | thanks | |
| 13:56:08 | sean-k-mooney | +2w on the fix the cyborg irc call is starting nowish so ill see if i can review the attachmet id one before then | |
| 13:56:23 | gibi | OK | |
| 13:56:46 | sean-k-mooney | ach cool you addeed the return code checks | |
| 13:57:16 | sean-k-mooney | well you added them as new tests | |
| 13:57:30 | sean-k-mooney | but ya i think thats ok | |
| 14:04:53 | sean-k-mooney | gibi: +2 on that too i dont really have any feedback to add so if gmaan is happy to proceed then cool | |
| 14:05:28 | sean-k-mooney | irc meting just started but are you plannning to follow up with the other bug to make repsonce validation return a 500 | |
| 14:06:28 | gibi | I will take a look on that bug too yes | |
| 14:12:17 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make functional-py313-threading voting https://review.opendev.org/c/openstack/nova/+/997216 | |
| 14:13:29 | gibi | sambork: am I correct that this is still valid as you only fixed something in vncproxy especially? https://bugs.launchpad.net/nova/+bug/2151537 | |
| 14:16:27 | sambork | gibi I think so because my fix was only in proxies | |
| 14:17:23 | sambork | but I can take a look and apply similar fix if it's applies also to this issue | |
| 14:17:58 | gibi | sambork: so far I only know it is a pickling issue but not more | |
| 14:18:33 | gibi | we don't need to rush, if you have time you can take it I'm not actively working on it. I just did a quick scrub of eventlet-removal bugs as some was outdated and I noticed this | |
| 14:18:48 | sambork | ack! | |
| 15:16:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Stabilize test_create_images_negative_invalid_state https://review.opendev.org/c/openstack/nova/+/997222 | |
| 15:22:29 | opendevreview | Merged openstack/nova stable/2026.1: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/997027 | |
| 17:39:00 | opendevreview | Mike Lowe proposed openstack/nova stable/2025.1: libvirt: Fix false mem-encryption/UEFI conflict with cpu_mode=custom https://review.opendev.org/c/openstack/nova/+/997258 | |
| 17:42:54 | opendevreview | Mike Lowe proposed openstack/nova master: libvirt: Fix false mem-encryption/UEFI conflict with cpu_mode=custom https://review.opendev.org/c/openstack/nova/+/997259 | |
| 18:14:07 | gmaan | gibi: sean-k-mooney lgtm, +w | |
| 18:20:12 | sean-k-mooney | gmaan: you may also be intrested in https://review.opendev.org/c/openstack/nova/+/971153/2 and the fix https://review.opendev.org/c/openstack/nova/+/946223/6 | |
| 18:20:36 | sean-k-mooney | there is a window where invoking instnace actions can reulst in a 500 before the inital server create is complete | |
| 18:21:03 | sean-k-mooney | those repoduce and adress that and call out that precondtion more explcitly | |
| 18:21:26 | gmaan | sean-k-mooney: ack, will check sometime today otherwise tomorrow | |
| 18:21:32 | sean-k-mooney | the repoducer need a littel work so no rush | |
| 18:21:59 | sean-k-mooney | the fix is more or less correct but ya just good to add to yoru queue | |
| 18:23:45 | gmaan | sure | |
| 18:28:59 | opendevreview | Fabian Wiesel proposed openstack/nova master: vmware: optionally combine host+datastore inventory query https://review.opendev.org/c/openstack/nova/+/997283 | |
| 18:34:30 | opendevreview | sean mooney proposed openstack/os-vif master: typing: Adapt to oslo.versionedobjects 3.11 https://review.opendev.org/c/openstack/os-vif/+/997289 | |
| 19:51:57 | opendevreview | sean mooney proposed openstack/os-vif master: typing: Adapt to oslo.versionedobjects 3.11 https://review.opendev.org/c/openstack/os-vif/+/997289 | |
| 19:51:58 | opendevreview | sean mooney proposed openstack/os-vif master: typing: Define supported OVS VIF types https://review.opendev.org/c/openstack/os-vif/+/997296 | |
| 19:51:59 | opendevreview | sean mooney proposed openstack/os-vif master: ovs: Preserve versioned field compatibility https://review.opendev.org/c/openstack/os-vif/+/997297 | |
| #openstack-nova - 2026-07-15 | |||
| 06:58:34 | opendevreview | Rodolfo Alonso proposed openstack/os-vif master: ovs: Pass ``multi_queue`` inside ``ifr`` dict for tuntap devices https://review.opendev.org/c/openstack/os-vif/+/995933 | |
| 07:08:50 | opendevreview | Masahito Muroi proposed openstack/nova master: Add bug 2106085 reproducer test case https://review.opendev.org/c/openstack/nova/+/971153 | |
| 07:08:51 | opendevreview | Masahito Muroi proposed openstack/nova master: Add check_instance_state validation to the four action API https://review.opendev.org/c/openstack/nova/+/946223 | |
| 07:50:14 | artem_vasilyev | Hi, please make a review for this bugfix: https://review.opendev.org/c/openstack/nova/+/984048 It fixes race between resize and update_available_resource for dedicated CPUs, which can lead to CPUUnpinningInvalid errors | |
| 08:20:40 | opendevreview | Merged openstack/os-vif master: typing: Adapt to oslo.versionedobjects 3.11 https://review.opendev.org/c/openstack/os-vif/+/997289 | |
| 08:20:41 | opendevreview | Merged openstack/os-vif master: typing: Define supported OVS VIF types https://review.opendev.org/c/openstack/os-vif/+/997296 | |
| 08:37:02 | gibi | folks nova-alt-configuration is broken and blocking the gate https://zuul.opendev.org/t/openstack/builds?job_name=nova-alt-configurations&project=openstack/nova | |
| 08:37:55 | gibi | do we have a bug report and somebody already actively working on it? | |
| 08:44:33 | bauzas | gibi: sorry, no idea | |
| 08:45:06 | bauzas | but this is clear that it failed at some point on Monday | |
| 08:45:56 | gibi | yep | |
| 08:47:05 | frickler | ah, that's a known issue in OSC I think "public endpoint for block-storage service in RegionOne region not found" | |
| 08:47:25 | frickler | see https://review.opendev.org/c/openstack/openstacksdk/+/997117 | |
| 08:49:56 | gibi | frickler: thanks | |
| 08:50:01 | frickler | actually it looks like osc 10.2.1 fixes this, so just need to wait for the u-c bump to merge https://review.opendev.org/c/openstack/releases/+/997204 | |
| 08:51:17 | gibi | cool then I will wait for https://review.opendev.org/c/openstack/releases/+/997204 | |
| 08:51:49 | bauzas | thanks frickler | |
| 08:52:26 | gibi | yeah nova-alt-configuration disables cinder so the bug description matches | |
| 10:16:34 | opendevreview | Merged openstack/nova stable/2025.2: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/997028 | |
| 10:22:37 | opendevreview | Balazs Gibizer proposed openstack/placement master: Reproduce bug XXX https://review.opendev.org/c/openstack/placement/+/997362 | |
| 10:23:01 | gibi | sean-k-mooney: ^^ this is the repro for the next placement a_c query slowness we saw downstream | |
| 10:23:30 | sean-k-mooney | gibi: oh cool you had time to start triaging it | |
| 10:23:34 | gibi | yepp | |
| 10:23:48 | gibi | at least with the existing functional test it is easy to reproduce | |
| 10:23:54 | sean-k-mooney | i have a intuition of what we shoudl be doing at the sql level | |
| 10:24:01 | gibi | I will see if I have time this sprint to look for an optimization | |
| 10:24:04 | sean-k-mooney | but im not sure how to express that in placement | |
| 10:24:41 | sean-k-mooney | do you have time to chat about it on meet breifly | |