| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-08-31 | |||
| 16:34:01 | Uggla | gibi please go ahead | |
| 16:34:02 | gibi | when you said 18:13 < Uggla> virtiofs cold migration needs focus too. I focused on testing it last week. | |
| 16:34:10 | gibi | what do you mean? | |
| 16:34:33 | gibi | is there some post FF stuff to land there? | |
| 16:34:43 | dansmith | an FFE was requested | |
| 16:35:02 | dansmith | I thought that it was mostly good to go though after Uggla found and gouthamr fixed (right?) a ceph secret thing | |
| 16:35:21 | gibi | ohh i missed the FFE | |
| 16:35:23 | dansmith | still trying to catch up on the traffic since I was out but that was what I saw | |
| 16:36:08 | sean-k-mooney | not sure if the micvoverison issue has been adressed. tl;dr we shoudl not have one for the migration as there is no api chagne | |
| 16:36:21 | Uggla | yes absolutely a strange behavior with cepfs and migration that happens if you migrate vm depending of the order they created the secret. | |
| 16:36:24 | dansmith | I commented on that | |
| 16:36:34 | dansmith | I assume if we decide to drop it then it'll be trivial to do that | |
| 16:36:55 | dansmith | so I wasn't really including it in "needs focus" :) | |
| 16:37:25 | elodilles | (the FFE mail: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/GIYUU732R5VB6BNQONWPTHR6QF5S2FVT/ ) | |
| 16:37:35 | sean-k-mooney | ack, i have been off since tursday evening so didnt follow up im back tomorrow | |
| 16:38:09 | sean-k-mooney | that was my only real concern | |
| 16:38:23 | gibi | cool, thanks for the summaries. :) | |
| 16:39:27 | Uggla | I'm testing it again, to ensure the bug I found last week is corrected. | |
| 16:40:54 | Uggla | I have also a couple of things remaining to test. But currently the feature looks quite solid. | |
| 16:42:15 | Uggla | anything else you'd like to discuss ? | |
| 16:43:48 | Uggla | if not I think we are done. | |
| 16:44:17 | Uggla | Thanks for joining this meeting. Have a nice day/evening. | |
| 16:44:30 | Uggla | #endmeeting | |
| 16:44:30 | opendevmeet | Meeting ended Mon Aug 31 16:44:30 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:44:30 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-31-16.02.html | |
| 16:44:30 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-31-16.02.txt | |
| 16:44:30 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-31-16.02.log.html | |
| 16:44:32 | tkajinam | thanks. | |
| 16:44:33 | elodilles | thanks Uggla o/ | |
| 16:44:40 | gibi | thanks Uggla | |
| 16:45:43 | lajoskatona | o/ | |
| 16:49:54 | tkajinam | just in case you have a few moments there are a few libvirt driver cleanups with one +2 . https://review.opendev.org/c/openstack/nova/+/997818 https://review.opendev.org/c/openstack/nova/+/995160 and https://review.opendev.org/c/openstack/nova/+/997878 | |
| 16:54:18 | gmaan | gibi: this is the further refactoring (as dansmith commented on the original change) of thread pool executors. dansmith has initial review on that, waiting for you to have a look and I can update it further | |
| 16:57:49 | gibi | gmaan: which one? :) | |
| 16:58:22 | gibi | I guess https://review.opendev.org/c/openstack/nova/+/1001572 | |
| 16:58:33 | gmaan | gibi: yeah this one :) https://review.opendev.org/c/openstack/nova/+/1001572 | |
| 16:58:35 | gibi | I will try to get to it tomorrow | |
| 16:58:36 | gibi | :) | |
| 16:58:39 | gmaan | thanks | |
| 16:59:51 | gmaan | monday morning, need more coffee | |
| 17:43:20 | gouthamr | dansmith: welcome back | |
| 17:47:34 | gouthamr | sean-k-mooney: dansmith : ack on the review comments; I’ll get to them as soon as I get to my laptop | |
| 18:12:02 | gmaan | sean-k-mooney: as you pinged me on this, I think returning 400 is more suitable here, returning 409 need microversion https://review.opendev.org/c/openstack/nova/+/946223 | |
| 18:12:07 | gmaan | i commented on it | |
| 18:27:22 | sean-k-mooney | i would not really agree that all action apis are indivgual interfaces | |
| 18:27:35 | sean-k-mooney | but i dont have a stong perfence | |
| 18:28:18 | sean-k-mooney | its a single endpoing form my persecitie and the return code shoudl not really vary based on the request body once the request is valid | |
| 18:28:37 | sean-k-mooney | but ya either is fine if you prefer the 400 that works for me | |
| 19:08:35 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 19:08:35 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron subnets https://review.opendev.org/c/openstack/nova/+/962190 | |
| 19:08:36 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron extensions https://review.opendev.org/c/openstack/nova/+/962270 | |
| 19:59:36 | opendevreview | Ashish Gupta proposed openstack/nova master: Fix init_host crash migration test under native threading https://review.opendev.org/c/openstack/nova/+/1001369 | |
| 21:25:38 | gmaan | historically, nova action APIs has not been best interface but they are individual operations and interface which is why we have different set of return code per action APIs | |
| 21:26:04 | opendevreview | Merged openstack/nova master: ovo: Drop deprecated item_cls argument https://review.opendev.org/c/openstack/nova/+/1002937 | |
| 22:25:37 | opendevreview | Stephen Finucane proposed openstack/os-vif master: Fix config option registration https://review.opendev.org/c/openstack/os-vif/+/1003173 | |
| 22:25:58 | opendevreview | Stephen Finucane proposed openstack/os-vif master: Fix config option registration https://review.opendev.org/c/openstack/os-vif/+/1003173 | |
| 22:29:40 | opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Replace legacy auth options (2/3) https://review.opendev.org/c/openstack/nova/+/1002657 | |
| 22:29:40 | opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Replace legacy auth options (3/3) https://review.opendev.org/c/openstack/nova/+/1002359 | |
| 22:29:41 | opendevreview | Stephen Finucane proposed openstack/nova master: neutron: Simplify client creation https://review.opendev.org/c/openstack/nova/+/1002658 | |
| 22:29:41 | opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Tweak endpoint_override logic https://review.opendev.org/c/openstack/nova/+/1003174 | |
| 22:29:42 | opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Use discover_version for microversion negotiation https://review.opendev.org/c/openstack/nova/+/1003175 | |
| 22:29:42 | opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Replace legacy auth options (1/3) https://review.opendev.org/c/openstack/nova/+/1003176 | |
| 22:29:43 | opendevreview | Stephen Finucane proposed openstack/nova master: Drop support for v2 password auth https://review.opendev.org/c/openstack/nova/+/1003177 | |
| 22:29:56 | stephenfin | melwitt: I went down a rabbit hole 🤦 | |
| 23:03:27 | opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Use discover_version for microversion negotiation https://review.opendev.org/c/openstack/nova/+/1003175 | |
| 23:03:27 | opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Replace legacy auth options (1/3) https://review.opendev.org/c/openstack/nova/+/1003176 | |
| 23:03:28 | opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Replace legacy auth options (2/3) https://review.opendev.org/c/openstack/nova/+/1002657 | |
| 23:03:28 | opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Replace legacy auth options (3/3) https://review.opendev.org/c/openstack/nova/+/1002359 | |
| 23:03:29 | opendevreview | Stephen Finucane proposed openstack/nova master: neutron: Simplify client creation https://review.opendev.org/c/openstack/nova/+/1002658 | |
| 23:03:29 | opendevreview | Stephen Finucane proposed openstack/nova master: Drop support for v2 password auth https://review.opendev.org/c/openstack/nova/+/1003177 | |
| 23:03:30 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: cinder: Simplify client creation https://review.opendev.org/c/openstack/nova/+/1003188 | |
| #openstack-nova - 2026-09-01 | |||
| 00:51:26 | opendevreview | Merged openstack/os-vif master: Fix config option registration https://review.opendev.org/c/openstack/os-vif/+/1003173 | |
| 04:07:56 | opendevreview | Adam McArthur proposed openstack/nova master: Use safe allocation deletion on instance delete https://review.opendev.org/c/openstack/nova/+/1003208 | |
| 07:11:09 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | |
| 07:11:10 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Reconcile stale Manila share access rules https://review.opendev.org/c/openstack/nova/+/1001916 | |
| 07:38:10 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | |
| 07:38:10 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Reconcile stale Manila share access rules https://review.opendev.org/c/openstack/nova/+/1001916 | |
| 07:48:10 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron subnets https://review.opendev.org/c/openstack/nova/+/962190 | |
| 07:48:11 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron extensions https://review.opendev.org/c/openstack/nova/+/962270 | |
| 08:07:04 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | |
| 08:07:04 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Reconcile stale Manila share access rules https://review.opendev.org/c/openstack/nova/+/1001916 | |
| 08:28:57 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | |
| 08:29:10 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Reconcile stale Manila share access rules https://review.opendev.org/c/openstack/nova/+/1001916 | |
| 08:33:55 | gibi | gmaan: https://github.com/openstack/nova/blob/d3a1c95d06c2d97368207c2728fc262796683ad8/nova/service.py#L435 this combined with the _get_manager_shutdown_timeout returning 0 is a big trap in the functional tests. It means test cases leaving a thread behind that finishes after the test case finished and a new one starts and they are calling things on the manager that calling things on the global | |
| 08:34:01 | gibi | executor state. | |
| 08:40:29 | gibi | I filed https://bugs.launchpad.net/nova/+bug/2165991 | |
| 08:54:51 | gibi | btw new hit on TIMED_OUT and no new hit on DB locked. But I still checking out the rest of the functional-threading failures happend yesterday (zuul is non cooperative): | |
| 08:55:04 | gibi | I mean no new hit | |
| 08:55:41 | gibi | I belive the DB locked issue is real as I saw it locally once or twice in extreme scenarios | |
| 08:55:54 | gibi | (extrem means by introducing lot of additional DB churn) | |
| 08:56:05 | gibi | but still no stable reproducer | |
| 09:03:06 | gibi | I have one suspicious functional-threading fauilure from yesterday https://5125be418d93521dc669-217c5d59ee9178b06ecc76d8426e75cb.ssl.cf1.rackcdn.com/openstack/1ff94c6e70154943aae8d53d58fbcdd5/testr_results.html I will look deeper into it | |
| 09:33:03 | opendevreview | Ashish Gupta proposed openstack/nova master: Fix init_host crash migration test under native threading https://review.opendev.org/c/openstack/nova/+/1001369 | |
| 10:36:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]Ensure shutdown thread is waited on https://review.opendev.org/c/openstack/nova/+/1003245 | |
| 10:36:54 | gibi | gmaan: ^^ fix for the above reported bug | |
| 10:37:13 | gibi | gmaan: I will put a follow up on top with 3 functional test fixed that haning the shutdown | |
| 10:52:44 | gibi | gmaan: also if I see that graceful shutdown logs a task as live_migration_at_dest which RPC handler it is? post...at_dest? rollback...at_dest? | |
| 11:14:57 | gibi | I started logging stack trace at _record_task_start and I see something strange | |
| 11:15:30 | gibi | the _record_task_start live_migration_at_dest actually happens from File "/home/gibi/upstream/git/openstack/nova/nova/compute/manager.py", line 9431, in pre_live_migration | |
| 11:15:59 | gibi | is this correct that we name the task live_migration_at_dest when it is actually a pre_live_migration RPC? | |