| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-01-26 | |||
| 16:53:15 | tkajinam | I got the impression that we may start with rejecting empty thing while leaving the others as these could be "valid partial match", I guess | |
| 16:53:42 | gibi | tkajinam: what happens that these fields are basically act as filters | |
| 16:53:53 | gibi | so the empty filter list means all devices | |
| 16:54:32 | gibi | so for me it is logical how nova behaves today | |
| 16:54:41 | tkajinam | hmm ok | |
| 16:54:50 | gibi | removing just the empty filter list case feels strange to me | |
| 16:55:06 | gibi | especially if we keep {address=""} doing the same thing, meaning all devcie | |
| 16:55:09 | gibi | device | |
| 16:56:50 | gibi | {managed=false} also means all devices today | |
| 16:57:15 | gibi | as managed is not a filter config but a result enhancing config here | |
| 16:57:37 | gibi | it means all matching devices get the managed flag set to false | |
| 16:57:44 | Uggla | yep, that's plenty of these exceptions I guess. | |
| 16:57:50 | gibi | yeah it is complex | |
| 16:58:01 | gibi | we can say 0 partial match but then we loose valid use case I feel | |
| 16:58:11 | Uggla | ack | |
| 16:58:27 | gibi | like { "vendor_id":"8086", "product_id":"10ca", "address": "0000:07:"} | |
| 16:58:33 | Uggla | can we improve the doc somewhere ? | |
| 16:58:43 | gibi | I have VFs under 07 and I want to match all | |
| 16:59:29 | gibi | Uggla: we can improve the doc be descrbing the current behavior but then the todays behavior becomes the expected behavior and Sean disagrees | |
| 16:59:37 | gibi | a/be/by/ | |
| 17:00:13 | Uggla | if not I agree to close it at non fix, explaining that it is not "ideal" but hard to improve. | |
| 17:01:00 | tkajinam | maybe we could document the current behavior first and then discuss how it can be improved separately | |
| 17:01:28 | gibi | As other cores I silent I would say lets move on. It is not that important | |
| 17:01:50 | gibi | s/other cores I silent/ other cores are silent/ | |
| 17:01:59 | Uggla | ok, we can discuss this later as well in async mode. | |
| 17:02:05 | gibi | sure | |
| 17:02:14 | gibi | my question is in the last comment of the bug | |
| 17:02:31 | Uggla | because we are on the top of the hour, time to close. | |
| 17:02:43 | Uggla | Thanks for joining this meeting. Have a nice day/evening and see you next week. | |
| 17:02:48 | gibi | thanks Uggla | |
| 17:02:51 | Uggla | #endmeeting | |
| 17:02:51 | opendevmeet | Meeting ended Mon Jan 26 17:02:51 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 17:02:51 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2026/nova.2026-01-26-16.01.html | |
| 17:02:51 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2026/nova.2026-01-26-16.01.txt | |
| 17:02:51 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2026/nova.2026-01-26-16.01.log.html | |
| 17:03:15 | elodilles | thanks o/ | |
| 17:15:50 | opendevreview | Merged openstack/nova stable/2024.2: libvirt: Skip unsupported firmware types https://review.opendev.org/c/openstack/nova/+/974409 | |
| 17:20:44 | opendevreview | Nisha Brahmankar proposed openstack/nova master: Fix exception handling in nova cache https://review.opendev.org/c/openstack/nova/+/974695 | |
| 19:12:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Libvirt event handling without eventlet https://review.opendev.org/c/openstack/nova/+/965949 | |
| 19:12:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use an executor to delay STOPPED events https://review.opendev.org/c/openstack/nova/+/974445 | |
| 19:13:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 19:13:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Test with oslo.vmware + compute eventlet removal patches https://review.opendev.org/c/openstack/nova/+/973468 | |
| 19:27:13 | opendevreview | Balazs Gibizer proposed openstack/nova master: [hacking]Do not mock threading.Event https://review.opendev.org/c/openstack/nova/+/971454 | |
| 19:27:13 | opendevreview | Balazs Gibizer proposed openstack/nova master: Libvirt event handling without eventlet https://review.opendev.org/c/openstack/nova/+/965949 | |
| 19:27:14 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use an executor to delay STOPPED events https://review.opendev.org/c/openstack/nova/+/974445 | |
| 19:27:14 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 19:27:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Test with oslo.vmware + compute eventlet removal patches https://review.opendev.org/c/openstack/nova/+/973468 | |
| 21:07:02 | opendevreview | Steve Baker proposed openstack/nova master: Add VNC console support for the Ironic driver https://review.opendev.org/c/openstack/nova/+/942528 | |
| 22:15:16 | opendevreview | Steve Baker proposed openstack/nova master: Use the correct name for the ironic check job https://review.opendev.org/c/openstack/nova/+/974781 | |
| 22:44:57 | opendevreview | melanie witt proposed openstack/nova master: DNM test https://review.opendev.org/c/openstack/nova/+/974792 | |
| #openstack-nova - 2026-01-27 | |||
| 06:30:30 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Use firmware auto-selection by libvirt https://review.opendev.org/c/openstack/nova/+/969132 | |
| 06:33:23 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add capability to load smm feature from existing xml https://review.opendev.org/c/openstack/nova/+/969131 | |
| 06:33:40 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Use firmware auto-selection by libvirt https://review.opendev.org/c/openstack/nova/+/969132 | |
| 07:39:19 | gokhan | hello folks, I want to get your thoughts about this patch https://review.opendev.org/c/openstack/nova/+/973750 . It seems https://github.com/openstack/nova/commit/5a55a78d510b86975f0f4f8f43ee1feef7206244#diff-47eb12598e353b9e0689707d7b477353200d0aa3ed13045ffd3d017ee7d9e753 commit resolves live migration pre migrate failure. Regarding the fixes in Bug #1899835 (Avoid volume rollback mismatches), which handles cleanup on destination when | |
| 07:39:19 | gokhan | pre_live_migration fails: I’ve observed that many live migration failures (like the one where a volume is in 'backing-up' status) could potentially be caught much earlier. Currently, if a volume is busy or in an unstable state, the failure often occurs during pre_live_migration, triggering the complex rollback logic we've been refining. Would it make sense to implement a proactive volume status check at the Nova API level (or early in th | |
| 07:39:20 | gokhan | e Conductor) before the RPC call to the destination node? | |
| 07:47:47 | gibi | gokhan: I agree that if there is a way to catch "bad" volume states early in the complex live migration process then that can decrease the likelyhood of a problematic rollback. | |
| 07:56:18 | gokhan | thanks gibi We have submitted a patch (https://review.opendev.org/c/openstack/nova/+/973750) for catching in api side. We would like to confirm if this is the recommended approach. | |
| 08:02:32 | gokhan | Since melwitt has previously worked on similar issues—specifically the 'Avoid volume rollback mismatches' patches—I would highly value her insights and comments on this matter to ensure we're aligned with the existing logic. | |
| 08:03:50 | gibi | gokhan: yeah, I'm reading you patch now... | |
| 08:04:47 | gokhan | thanks gibi :) | |
| 08:10:58 | gibi | gokhan: I think the direction is OK. I left comments about the test coverage, and pinged melwitt in the review | |
| 08:15:56 | gokhan | Thanks for the review gibi, we will address your comments and upload a new patch set soon. | |
| 08:17:14 | opendevreview | Seyeong Kim proposed openstack/nova master: libvirt: Support boot_index for multiple block devices https://review.opendev.org/c/openstack/nova/+/963665 | |
| 11:04:54 | tobias-urdin | can somebody refresh my memory, when doing live migration for an instance with cpu_policy=dedicated and cpu_thread_policy=require and cpu_dedicated_set set, does scheduler take into account remapping PCPU for example on source cores 0,40 to something like 1,41 if 0,40 was not free on destination compute? | |
| 11:05:38 | tobias-urdin | gibi: ^ iirc it should work? (caracal) | |
| 11:12:16 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron extensions https://review.opendev.org/c/openstack/nova/+/962270 | |
| 11:12:16 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron floating IPs https://review.opendev.org/c/openstack/nova/+/962604 | |
| 11:24:37 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 11:24:59 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron subnets https://review.opendev.org/c/openstack/nova/+/962190 | |
| 11:25:09 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron extensions https://review.opendev.org/c/openstack/nova/+/962270 | |
| 11:30:28 | opendevreview | Esra Ozkan proposed openstack/nova master: Fix Concurrent VM Live Migrate - Volume Backup Error https://review.opendev.org/c/openstack/nova/+/973750 | |
| 11:32:26 | opendevreview | Max proposed openstack/nova master: Add regression test for bug #2137366 https://review.opendev.org/c/openstack/nova/+/974831 | |
| 11:32:26 | opendevreview | Max proposed openstack/nova master: fix: delete attachments after rescheduling delete https://review.opendev.org/c/openstack/nova/+/974832 | |
| 11:36:19 | opendevreview | Esra Ozkan proposed openstack/nova master: Fix Concurrent VM Live Migrate - Volume Backup Error https://review.opendev.org/c/openstack/nova/+/973750 | |
| 11:36:23 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron floating IPs https://review.opendev.org/c/openstack/nova/+/962604 | |
| 12:04:37 | gibi | tobias-urdin: yeah I think it should work but obviously we need more data. | |
| 12:09:23 | gibi | tobias-urdin: I think this is the funtional test that at least partially covering it https://github.com/openstack/nova/blob/d840c63a18ad951382be821f90a6b28c627911fa/nova/tests/functional/libvirt/test_numa_live_migration.py#L197 | |
| 12:09:59 | sean-k-mooney | tobias-urdin: it does in any release the properly supprot numa live migration which i think was added in train or wallaby | |
| 12:10:30 | sean-k-mooney | ya it was train | |
| 12:10:49 | sean-k-mooney | https://specs.openstack.org/openstack/nova-specs/specs/train/implemented/numa-aware-live-migration.html | |
| 12:33:17 | tobias-urdin | thanks, not sure why i had a vague memory of it having to be equal on both sides, probably pre-train days something then :) | |
| 12:33:57 | sean-k-mooney | yep pre-train the live migraiton woudl be allowed but the vms xml would not be updated until it was hard rebboted | |
| 12:34:14 | sean-k-mooney | asumign the cores existed if they didnt the live migraiton woudl fail but only in that case | |
| 12:34:34 | sean-k-mooney | we put a block in place and backported pre train with a workaround to reenable it as part of supprotign it properly | |
| 12:35:00 | sean-k-mooney | basiclly pre train we said only do live migration with numa if your usign hugepage without cpu pinning | |
| 12:42:30 | opendevreview | Stephen Finucane proposed openstack/nova master: Follow-up for I85a7729ee65f8987ed0239f80d8d0082a414ab8f https://review.opendev.org/c/openstack/nova/+/974843 | |
| 13:53:26 | opendevreview | Merged openstack/nova master: api: Simplify servers views (1/3) https://review.opendev.org/c/openstack/nova/+/956231 | |
| 14:00:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use an executor to delay STOPPED events https://review.opendev.org/c/openstack/nova/+/974445 | |
| 14:00:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 14:00:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Test with oslo.vmware + compute eventlet removal patches https://review.opendev.org/c/openstack/nova/+/973468 | |
| 14:00:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: SubclassSignatureTestCase to use NoDBTestCase as base https://review.opendev.org/c/openstack/nova/+/974861 | |
| 14:11:25 | gibi | sean-k-mooney: bauzas: gmaan: https://review.opendev.org/c/openstack/nova/+/974445 this is ready for serious review now. I have couple of remaining todos but nothing is expected to change in the core of the patch. In the other hand the core of the patch needs deep review | |
| 14:12:47 | bauzas | ack, I definitely need a coffee for reviewing it | |
| 14:13:28 | gibi | don't worry if it take two :) | |
| 14:13:34 | opendevreview | Merged openstack/nova master: api: Simplify servers views (2/3) https://review.opendev.org/c/openstack/nova/+/956232 | |
| 14:17:54 | opendevreview | Merged openstack/nova master: api: Simplify servers views (3/3) https://review.opendev.org/c/openstack/nova/+/956233 | |
| 14:23:37 | sean-k-mooney | gibi: ok ill try an take a look today or tomorrow | |
| 14:56:47 | gibi | thanks both of you :) | |
| 15:13:54 | MaxLamprecht[m] | Hey folks, I'm currently debugging many out of sync cases between nova and cinder BDMs/attachments in our environment. Mostly these are caused by parallel operations/lock-contention or other reasons. I added a comment... (full message at <https://matrix.org/oftc/media/v1/media/download/AYgeZZCT43vzk9WSn7ySGLYl0AqCdodcgtuHH4PEsRyizpNZ9dfQRaY5IGWjRMbtweIwNYxNRndku0zt8aS2xE9CecSB29CAAG1hdHJpeC5vcmcvb0drQk95eXZDTkVTUmphRGJPSk5GYlRZ>) | |