| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-03-10 | |||
| 15:04:10 | opendevreview | Merged openstack/nova master: Add regression test for bug #2088066 https://review.opendev.org/c/openstack/nova/+/965927 | |
| 15:11:39 | opendevreview | Nicolai Ruckel proposed openstack/nova stable/2025.1: Add functional reproducer for bug 2125030 https://review.opendev.org/c/openstack/nova/+/979864 | |
| 15:11:40 | opendevreview | Nicolai Ruckel proposed openstack/nova stable/2025.1: Move cleanup of vTPM secret from driver to compute https://review.opendev.org/c/openstack/nova/+/979865 | |
| 15:11:41 | opendevreview | Nicolai Ruckel proposed openstack/nova stable/2025.1: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/979866 | |
| 15:25:32 | dansmith | Uggla: your -1 is still on here from Feb: https://review.opendev.org/c/openstack/nova/+/972045/2 | |
| 15:25:52 | dansmith | I guess because you -1'd this patch for the thing on the one below, and since this one didn't change it didn't reset | |
| 15:44:06 | bauzas | gibi: melwitt: just a note that I updated https://review.opendev.org/c/openstack/nova/+/845757 by adding the bug report in the reno file | |
| 15:44:18 | bauzas | (and thanks for your reviews <3 ) | |
| 15:53:04 | MaxLamprecht[m] | dansmith: thx for review of the attachment stuff. During debugging I found 2 more things that are really small and broken: https://review.opendev.org/c/openstack/nova/+/973103 and https://review.opendev.org/c/openstack/nova/+/975651 | |
| 16:01:47 | Uggla | dansmith yep unfortunately you are right. It has not change since that time. | |
| 16:04:26 | dansmith | MaxLamprecht[m]: I'm about to get on a couple _hours_ of calls but will look closer at the second in in a bit | |
| 16:05:56 | gibi | bauzas: +2 | |
| 16:06:05 | bauzas | <3 | |
| 16:11:03 | MaxLamprecht[m] | dansmith: thx :) | |
| 16:12:16 | gibi | I just caugth up on the nova meeting logs from yesterday (slow me). I see that the SEV refactor is on hold until branch is open. That is OK for me. | |
| 18:20:32 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 18:20:32 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron subnets https://review.opendev.org/c/openstack/nova/+/962190 | |
| 18:20:33 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron extensions https://review.opendev.org/c/openstack/nova/+/962270 | |
| 18:20:33 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron floating IPs https://review.opendev.org/c/openstack/nova/+/962604 | |
| 19:25:15 | melwitt | bauzas: cool thanks I will check the update | |
| #openstack-nova - 2026-03-11 | |||
| 00:55:45 | opendevreview | Merged openstack/nova stable/2025.2: Add functional reproducer for bug 2125030 https://review.opendev.org/c/openstack/nova/+/979041 | |
| 04:44:06 | opendevreview | Merged openstack/nova master: Support multiple allocations for vGPUs https://review.opendev.org/c/openstack/nova/+/845757 | |
| 08:05:33 | nicolairuckel | melwitt: Should I just abandon this https://review.opendev.org/c/openstack/nova/+/979866 and create a new one with the correct cherry-picks? | |
| 09:16:46 | opendevreview | Max proposed openstack/nova master: fix: orphaned attachments during migration delete https://review.opendev.org/c/openstack/nova/+/972045 | |
| 09:29:35 | opendevreview | Nicolai Ruckel proposed openstack/nova stable/2025.1: Add functional reproducer for bug 2125030 https://review.opendev.org/c/openstack/nova/+/979864 | |
| 09:29:36 | opendevreview | Nicolai Ruckel proposed openstack/nova stable/2025.1: Move cleanup of vTPM secret from driver to compute https://review.opendev.org/c/openstack/nova/+/979865 | |
| 09:29:37 | opendevreview | Nicolai Ruckel proposed openstack/nova stable/2025.1: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/979866 | |
| 09:55:06 | nicolairuckel | The gate fails with an sqlalchemy error for me: https://b49a1a6bd7022e637e55-03d448e06513a4e4bb78797852449f92.ssl.cf1.rackcdn.com/openstack/cf4f116365064c10b0f3e76c6db33568/testr_results.html I vaguely remember people here talking about some sqlalchemy stuff. Was that related to that? | |
| 09:55:56 | nicolairuckel | I ran the same command (tox -epy312-threading -vv --skip-missing-interpreters=false) locally without any errors. | |
| 10:34:04 | gibi | nicolairuckel: am I blind or that job should not even run that test case due to the known issue https://github.com/openstack/nova/blob/master/threading_unit_test_excludes.txt#L38 | |
| 10:42:14 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add functional tests for NUMA aware vswitches https://review.opendev.org/c/openstack/nova/+/979999 | |
| 10:45:08 | nicolairuckel | gibi: That test doesn't seem to be excluded for 2025.2 though. | |
| 11:02:28 | opendevreview | Thibaut Démaret proposed openstack/nova master: libvirt: add disk rotation_rate support for local disks https://review.opendev.org/c/openstack/nova/+/979693 | |
| 11:09:36 | gibi | I will investigate | |
| 11:10:59 | nicolairuckel | https://github.com/openstack/nova/blob/stable/2025.2/threading_unit_test_excludes.txt | |
| 11:24:42 | elodilles | sorry, i did not see the discussion and rechecked it, but at least nova-tox-py312-threading has passed now o:) | |
| 11:25:36 | gibi | OK so https://review.opendev.org/c/openstack/nova/+/953475 is in 2025.2 so the job expected to run that branch | |
| 11:25:50 | gibi | I guess there was a fix on the exclude list that was not backported to 2025.2 | |
| 11:26:40 | nicolairuckel | and 2025.1 if I see that correctly | |
| 11:27:34 | gibi | yepp https://review.opendev.org/c/openstack/nova/+/961781 this is missing from stable. I will do the backports | |
| 11:27:41 | nicolairuckel | thank you | |
| 11:27:50 | gibi | thanks for notifying | |
| 11:28:25 | gibi | nicolairuckel: I don't think the job should run on 2025.1 | |
| 11:28:30 | gibi | but let me double check | |
| 11:28:33 | nicolairuckel | It's very self-serving for me. :D | |
| 11:29:42 | gibi | :) that is fine | |
| 11:29:51 | gibi | yeah it does not exists on 2025.1 | |
| 11:30:54 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.2: [nova-tox-py312-threading]Ignore failing tests https://review.opendev.org/c/openstack/nova/+/980013 | |
| 11:31:03 | gibi | elodilles: ^^ | |
| 11:31:54 | elodilles | thanks, looking o/ | |
| 11:34:18 | elodilles | gibi: +2+W'd it | |
| 11:34:31 | elodilles | thanks for backporting it o/ | |
| 11:39:11 | gibi | thanks for the quick review | |
| 11:46:08 | r-taketn | Hello, Uggla, gibi, bauzas. Regarding sev refactor series, my understanding from this week's irclogs is that reviews will restart after stable/2026.01(RC1) branch is cut. Is that correct? | |
| 11:46:58 | gibi | r-taketn: yes that was agreed on the last nova meeting on Monday | |
| 11:48:42 | r-taketn | Got it. Thanks for confirming. | |
| 13:23:57 | DominikDanelski[m] | sean-k-mooney: If you were interested, I addressed your comments and finally dealt with the flaky tests in https://review.opendev.org/c/openstack/nova/+/969251 and https://review.opendev.org/c/openstack/nova/+/968446 so both now pass. | |
| 13:28:16 | opendevreview | Dominik proposed openstack/nova master: NUMA Topology with Resource Providers: Object changes https://review.opendev.org/c/openstack/nova/+/978209 | |
| 13:30:39 | opendevreview | Dominik proposed openstack/nova master: NUMA Topology with Resource Providers: Scheduler changes https://review.opendev.org/c/openstack/nova/+/971176 | |
| 13:37:18 | opendevreview | Dominik proposed openstack/nova master: Regression test for Placement allocations remaining during failed schedule https://review.opendev.org/c/openstack/nova/+/969251 | |
| 13:38:27 | opendevreview | Dominik proposed openstack/nova master: Remove Placement allocations in the broken build cleanup https://review.opendev.org/c/openstack/nova/+/968446 | |
| 13:39:44 | sean-k-mooney | DominikDanelski[m]: ill need to find some dedicated time to reveiw the actual fix but we can likely proceed with the repoducer | |
| 13:40:06 | sean-k-mooney | i think the acual fix is reasonable | |
| 13:40:21 | nicolairuckel | I noticed something else that seems odd to me. The CI fails for one of my backports to 2025.1 (https://review.opendev.org/c/openstack/nova/+/979866) because of the check-cherry-pick job. Since the cherry-picked commit wasn't merged to 2025.2 yet that makes sense to me. But shouldn't this also fail then? https://review.opendev.org/c/openstack/nova/+/979865. | |
| 13:40:25 | sean-k-mooney | but i have not loaded context on the code to really say definititvly | |
| 13:55:37 | opendevreview | Akira KAMIO proposed openstack/nova master: libvirt: honor flavor disk bus and SCSI model for BFV https://review.opendev.org/c/openstack/nova/+/980072 | |
| 14:05:15 | opendevreview | Thibaut Démaret proposed openstack/nova master: libvirt: add disk rotation_rate support for local disks https://review.opendev.org/c/openstack/nova/+/979693 | |
| 14:46:05 | Uggla | gibi or melwitt can you have a look at https://review.opendev.org/c/openstack/nova/+/934984/9 there is a +2 from dansmith and sean-k-mooney previously. So I think it can be merged. Unless dansmith has an objection ? | |
| 14:47:46 | dansmith | no direct objection other than that rc1 is tomorrow and we really need to be quieting down.. you decide if it's better to do that now vs backport on friday.. like, if we start seeing failures after rc1 is cut... | |
| 14:48:58 | sean-k-mooney | i can review again im not against +2ing but im inclidned to hold +w until rc1 is tagged | |
| 14:52:06 | Uggla | sean-k-mooney, dansmith, we merged the reproducer. Is it ok if we don't merge the fix before rc1 ? | |
| 14:52:23 | sean-k-mooney | sure why would it not be | |
| 14:52:26 | opendevreview | Max proposed openstack/nova master: fix: device_by_alias should respect config type https://review.opendev.org/c/openstack/nova/+/975651 | |
| 14:52:29 | dansmith | of course | |
| 14:52:35 | sean-k-mooney | we have had repoduces in the code base fo months before the acual fix | |
| 14:53:06 | sean-k-mooney | its just a test and has no runtime impact on the shipped nova content | |
| 14:53:26 | Uggla | ok so it will be after rc1. | |
| 14:53:42 | sean-k-mooney | we can proably merge it early next week | |
| 14:53:55 | sean-k-mooney | we just wotn cherry pick it until after the offical release is done | |
| 14:54:20 | sean-k-mooney | note ill be on pto monday->wednesday inclusive | |
| 14:55:09 | Uggla | sean-k-mooney, dansmith, bauzas, gibi, gmaan I think nothing on your radars that still needs to land before RC1 ? | |
| 14:55:37 | Uggla | melwitt too ^ | |
| 14:55:50 | dansmith | nothing on my radar, although the dev alias fixes from MaxLamprecht[m] are definitely on my short list after | |
| 14:55:54 | sean-k-mooney | not form me. i have some bug fixes i would like review on but not before rc1 or the final rc | |
| 14:56:12 | Uggla | stephenfin too ^ | |
| 14:56:48 | sean-k-mooney | Uggla: i think most of us are in a wait and see holding patten until the rc is done and trying to minimise change | |
| 14:57:18 | Uggla | sean-k-mooney, yep that's my feeling too. But just checking with all of you. | |
| 14:58:40 | sean-k-mooney | Uggla: https://review.opendev.org/q/topic:%22bug/2140631%22 and https://review.opendev.org/c/openstack/nova/+/975500 are the changes i personally want to make progress on next but they can wait a week or two | |
| 14:59:23 | sean-k-mooney | the second id very low risk but we can backprot that to enable the ci testing later. neutorn alrady have it enabeld on there side | |
| 14:59:34 | Uggla | sean-k-mooney ok | |
| 15:00:10 | opendevreview | OpenStack Release Bot proposed openstack/placement stable/2026.1: Update .gitreview for stable/2026.1 https://review.opendev.org/c/openstack/placement/+/980094 | |
| 15:00:11 | opendevreview | OpenStack Release Bot proposed openstack/placement stable/2026.1: Update TOX_CONSTRAINTS_FILE for stable/2026.1 https://review.opendev.org/c/openstack/placement/+/980095 | |
| 15:00:13 | opendevreview | OpenStack Release Bot proposed openstack/placement master: Update master for stable/2026.1 https://review.opendev.org/c/openstack/placement/+/980096 | |
| 15:01:45 | sean-k-mooney | on a releated note ^ they have actully started creating the rc/stable branch today so the point is sort of mute | |
| 15:04:01 | sean-k-mooney | Uggla: on an unrelated note did you ever start woring on manilla share hotplug or live migration supprot | |
| 15:04:20 | sean-k-mooney | i know we wanted to do add supprot for memfd as well | |
| 15:04:32 | sean-k-mooney | to improve the overall expeince | |
| 15:04:41 | sean-k-mooney | is that still somethign your interested in? | |
| 15:06:11 | Uggla | sean-k-mooney, this is on the radar for H. I know I'll have some time to prepare the spec for share hotplug and live migration support. So we could discuss at the PTG. | |
| 15:06:14 | stephenfin | Uggla: nothing from me either. I have a few bug fixes starting at https://review.opendev.org/c/openstack/nova/+/974687/ but they're all latent issues/not regressions | |