| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-07-14 | |||
| 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: Add libvirt driver MDEV ARQ support for Cyborg https://review.opendev.org/c/openstack/nova/+/995840 | |
| 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 | |
| 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 | |
| 10:25:08 | gibi | not right now | |
| 10:25:12 | gibi | I will ping you later | |
| 10:25:36 | sean-k-mooney | ok cool my idea is pretty simple and i tought something placement already idd but apprently not | |
| 10:26:40 | sean-k-mooney | effectivly we shoudl be flattenting both the provider tree and aagreating the counts of all free resouce classe in the tree and flatteing the request and droping any root provider where the recquest exceed any RC in the flatttened tree | |
| 10:27:05 | sean-k-mooney | that tells use that regardless of the toplogy that root provider cant supprot the request | |
| 10:27:26 | sean-k-mooney | if that flanned check passes then and only then do we process it in python | |