| opendevreview | Merged openstack/nova stable/2025.2: Add functional reproducer for bug 2125030 https://review.opendev.org/c/openstack/nova/+/979041 | 00:55 |
|---|---|---|
| opendevreview | Merged openstack/nova master: Support multiple allocations for vGPUs https://review.opendev.org/c/openstack/nova/+/845757 | 04:44 |
| 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? | 08:05 |
| opendevreview | Max proposed openstack/nova master: fix: orphaned attachments during migration delete https://review.opendev.org/c/openstack/nova/+/972045 | 09:16 |
| 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 |
| 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 |
| 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:29 |
| 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 |
| nicolairuckel | I ran the same command (tox -epy312-threading -vv --skip-missing-interpreters=false) locally without any errors. | 09:55 |
| 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:34 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Add functional tests for NUMA aware vswitches https://review.opendev.org/c/openstack/nova/+/979999 | 10:42 |
| nicolairuckel | gibi: That test doesn't seem to be excluded for 2025.2 though. | 10:45 |
| 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:02 |
| gibi | I will investigate | 11:09 |
| nicolairuckel | https://github.com/openstack/nova/blob/stable/2025.2/threading_unit_test_excludes.txt | 11:10 |
| elodilles | sorry, i did not see the discussion and rechecked it, but at least nova-tox-py312-threading has passed now o:) | 11:24 |
| 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 |
| gibi | I guess there was a fix on the exclude list that was not backported to 2025.2 | 11:25 |
| nicolairuckel | and 2025.1 if I see that correctly | 11:26 |
| gibi | yepp https://review.opendev.org/c/openstack/nova/+/961781 this is missing from stable. I will do the backports | 11:27 |
| nicolairuckel | thank you | 11:27 |
| gibi | thanks for notifying | 11:27 |
| gibi | nicolairuckel: I don't think the job should run on 2025.1 | 11:28 |
| gibi | but let me double check | 11:28 |
| nicolairuckel | It's very self-serving for me. :D | 11:28 |
| gibi | :) that is fine | 11:29 |
| gibi | yeah it does not exists on 2025.1 | 11:29 |
| 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:30 |
| gibi | elodilles: ^^ | 11:31 |
| elodilles | thanks, looking o/ | 11:31 |
| *** tobias-u1 is now known as tobias-urdin | 11:33 | |
| elodilles | gibi: +2+W'd it | 11:34 |
| elodilles | thanks for backporting it o/ | 11:34 |
| gibi | thanks for the quick review | 11:39 |
| 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 |
| gibi | r-taketn: yes that was agreed on the last nova meeting on Monday | 11:46 |
| r-taketn | Got it. Thanks for confirming. | 11:48 |
| 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:23 |
| opendevreview | Dominik proposed openstack/nova master: NUMA Topology with Resource Providers: Object changes https://review.opendev.org/c/openstack/nova/+/978209 | 13:28 |
| opendevreview | Dominik proposed openstack/nova master: NUMA Topology with Resource Providers: Scheduler changes https://review.opendev.org/c/openstack/nova/+/971176 | 13:30 |
| opendevreview | Dominik proposed openstack/nova master: Regression test for Placement allocations remaining during failed schedule https://review.opendev.org/c/openstack/nova/+/969251 | 13:37 |
| opendevreview | Dominik proposed openstack/nova master: Remove Placement allocations in the broken build cleanup https://review.opendev.org/c/openstack/nova/+/968446 | 13:38 |
| 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:39 |
| sean-k-mooney | i think the acual fix is reasonable | 13:40 |
| 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 |
| sean-k-mooney | but i have not loaded context on the code to really say definititvly | 13:40 |
| 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 | 13:55 |
| 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: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: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:47 |
| sean-k-mooney | i can review again im not against +2ing but im inclidned to hold +w until rc1 is tagged | 14:48 |
| Uggla | sean-k-mooney, dansmith, we merged the reproducer. Is it ok if we don't merge the fix before rc1 ? | 14:52 |
| sean-k-mooney | sure why would it not be | 14:52 |
| opendevreview | Max proposed openstack/nova master: fix: device_by_alias should respect config type https://review.opendev.org/c/openstack/nova/+/975651 | 14:52 |
| dansmith | of course | 14:52 |
| sean-k-mooney | we have had repoduces in the code base fo months before the acual fix | 14:52 |
| sean-k-mooney | its just a test and has no runtime impact on the shipped nova content | 14:53 |
| Uggla | ok so it will be after rc1. | 14:53 |
| sean-k-mooney | we can proably merge it early next week | 14:53 |
| sean-k-mooney | we just wotn cherry pick it until after the offical release is done | 14:53 |
| sean-k-mooney | note ill be on pto monday->wednesday inclusive | 14:54 |
| Uggla | sean-k-mooney, dansmith, bauzas, gibi, gmaan I think nothing on your radars that still needs to land before RC1 ? | 14:55 |
| Uggla | melwitt too ^ | 14:55 |
| dansmith | nothing on my radar, although the dev alias fixes from MaxLamprecht[m] are definitely on my short list after | 14:55 |
| 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:55 |
| Uggla | stephenfin too ^ | 14:56 |
| 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:56 |
| Uggla | sean-k-mooney, yep that's my feeling too. But just checking with all of you. | 14:57 |
| 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:58 |
| 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 |
| Uggla | sean-k-mooney ok | 14:59 |
| 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 |
| 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 |
| opendevreview | OpenStack Release Bot proposed openstack/placement master: Update master for stable/2026.1 https://review.opendev.org/c/openstack/placement/+/980096 | 15:00 |
| 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:01 |
| sean-k-mooney | Uggla: on an unrelated note did you ever start woring on manilla share hotplug or live migration supprot | 15:04 |
| sean-k-mooney | i know we wanted to do add supprot for memfd as well | 15:04 |
| sean-k-mooney | to improve the overall expeince | 15:04 |
| sean-k-mooney | is that still somethign your interested in? | 15:04 |
| 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 |
| 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 | 15:06 |
| stephenfin | though I will note server share support is still not done in SDK or OSC | 15:06 |
| stephenfin | that should really happen in H: it's been 3 or 4 cycles now | 15:06 |
| Uggla | stephenfin, i know :( | 15:06 |
| sean-k-mooney | Uggla: ya it would be nice ot have at least a daft spec on that topic before the ptg if you intened to progress it but cool good to know | 15:07 |
| sean-k-mooney | stephenfin: 2 it merged in 2025.1 | 15:07 |
| sean-k-mooney | but yes it should | 15:07 |
| stephenfin | 3 we won't get it done in G | 15:08 |
| stephenfin | :) | 15:08 |
| sean-k-mooney | i guess it depend on how you count be either way ya it shoudl be resolved if we expect to extend it in h | 15:08 |
| sean-k-mooney | i woudl assum that horizon also does not have supprot for it | 15:09 |
| Uggla | sean-k-mooney, stephenfin, my understanding is that it is an "internal" priority now, so that might help. | 15:09 |
| sean-k-mooney | coudl be good to add to a cross project session | 15:09 |
| opendevreview | Akira KAMIO proposed openstack/nova master: libvirt: support flavor disk/cdrom bus and SCSI model https://review.opendev.org/c/openstack/nova/+/980072 | 15:09 |
| stephenfin | that would be my assumption. And I don't think Gophercloud does either so right now no one can use it using our clients | 15:09 |
| stephenfin | except if they're using gtema's rust client, perhaps | 15:09 |
| gmaan | Uggla: stephenfin we need to get this in before rc1, still stuck in gate https://review.opendev.org/c/openstack/nova/+/978494 | 15:21 |
| sean-k-mooney | stephenfin: speaking of sdk/client supprot have you added suprpot for flavor seach by name | 15:21 |
| sean-k-mooney | i.e. openstack flavor list --name ... | 15:21 |
| stephenfin | oh, I thought that had merged already | 15:21 |
| sean-k-mooney | it may have i just never checked | 15:21 |
| sean-k-mooney | oh you mean ganshams link | 15:22 |
| sean-k-mooney | that woudl quallify for rc2 anwyay and befcause fo the oder we do the rpc version pinning we alwasy? have one | 15:22 |
| stephenfin | yeah. I haven't done the client changes yet | 15:23 |
| sean-k-mooney | ok i should go back to testing things o/ | 15:25 |
| stephenfin | sean-k-mooney: https://review.opendev.org/c/openstack/openstacksdk/+/980101 | 15:27 |
| sean-k-mooney | do we need changes (seperatly) for 103 | 15:28 |
| sean-k-mooney | to acoont for the rild cleanup | 15:29 |
| opendevreview | Merged openstack/placement master: Update master for stable/2026.1 https://review.opendev.org/c/openstack/placement/+/980096 | 15:29 |
| sean-k-mooney | maybe it woudl be good to have a list of these soemwhere | 15:29 |
| opendevreview | Merged openstack/placement stable/2026.1: Update .gitreview for stable/2026.1 https://review.opendev.org/c/openstack/placement/+/980094 | 15:30 |
| opendevreview | Merged openstack/placement stable/2026.1: Update TOX_CONSTRAINTS_FILE for stable/2026.1 https://review.opendev.org/c/openstack/placement/+/980095 | 15:32 |
| gibi | Uggla: nothing on my radar that needed in rc1 | 15:51 |
| gibi | but I see we are tracking a fix above | 15:51 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Add functional tests for NUMA aware vswitches https://review.opendev.org/c/openstack/nova/+/979999 | 16:07 |
| Uggla | gibi, yes gmaan's patch which does not want to land. :) | 16:34 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Add functional tests for NUMA aware vswitches https://review.opendev.org/c/openstack/nova/+/979999 | 17:48 |
| opendevreview | Merged openstack/nova stable/2025.2: Move cleanup of vTPM secret from driver to compute https://review.opendev.org/c/openstack/nova/+/979052 | 20:37 |
| opendevreview | melanie witt proposed openstack/nova-specs master: Amend vTPM live migration spec https://review.opendev.org/c/openstack/nova-specs/+/980165 | 21:28 |
| opendevreview | melanie witt proposed openstack/nova-specs master: Propose vTPM live migration: 'deployment' mode https://review.opendev.org/c/openstack/nova-specs/+/980166 | 21:28 |
| opendevreview | melanie witt proposed openstack/nova-specs master: Propose vTPM live migration: 'deployment' mode https://review.opendev.org/c/openstack/nova-specs/+/980166 | 21:31 |
| melwitt | Uggla: my lame attempt to split the spec ^ I kept the original blueprint url the same but updated the "title" to say 'host' mode. and I made a new blueprint for 'deployment' mode here https://blueprints.launchpad.net/nova/+spec/vtpm-live-migration-deployment-mode (which is linked in the new spec) | 21:34 |
| opendevreview | Merged openstack/nova master: Fix wrong strict assertion of quota class set id https://review.opendev.org/c/openstack/nova/+/978494 | 21:45 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!