| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-01-30 | |||
| 18:28:46 | gmaan | sean-k-mooney: checking your reply | |
| 18:42:24 | gmaan | sean-k-mooney: I think catching exception and reseting the internal state is more safe and better | |
| 18:43:37 | gmaan | i know release_write_lock does not raise exception other then other writter try to unlock them but 1. its 3rd party so changes can happen there 2. resting internal state before actually releasing lock seems odd to m,e | |
| 18:46:40 | gmaan | gibi: let's discuss on Monday about the DB/object changes for new RPC topic. I was thinking the same as you commented but proposed the changes because old field 'topic' was in DB/object. I will comment in review too. | |
| 18:49:00 | gmaan | dansmith: maybe you remember if any rational or it is from starting, we have RPC 'topic' field in DB/Object but it is not user facing or changable things - https://github.com/openstack/nova/blob/59a7093915298973c72b6d1749a6acd27e0045a9/nova/objects/service.py#L328 | |
| 18:49:27 | gmaan | as I am adding a new RPC server for graceful shutdown, I proposed to add that new topic field 'topic_alt' in DB/object - https://review.opendev.org/c/openstack/nova/+/975240/2 | |
| 18:50:01 | sean-k-mooney | gmaan: ok ill do that quickly then before i leave | |
| 18:50:17 | dansmith | I don't remember specifically, I suspect it's a holdover from much longer ago where "everything was pluggable" | |
| 18:50:48 | gmaan | ohk, In that case, we can leave this old field but not add the new topic field as no use case as of now | |
| 18:50:51 | gmaan | sean-k-mooney: ack | |
| 18:51:27 | dansmith | yeah doesn't appear to be used anywhere | |
| 18:52:20 | gmaan | k, will abandon my change, if we need it can be added in future anyways | |
| 18:52:31 | dansmith | ah, looks like it may have been a nova-network thing | |
| 18:52:43 | dansmith | like, to get the network manager for a host, which may be shared | |
| 18:53:29 | dansmith | gmaan: https://github.com/openstack/nova/blob/icehouse-eol/nova/network/manager.py#L249 | |
| 18:53:34 | dansmith | that's a deep cut :) | |
| 18:55:12 | gmaan | ohk, I see that. I tried to find history in but did not go that old. :) thanks | |
| 18:55:54 | dansmith | objects were maturing around icehouse so I usually use that as the point to look for pre-object histopry :) | |
| 18:56:44 | gmaan | i remember icehouse that was the release I started my upstream work :) | |
| 18:56:58 | dansmith | you're such a newb ;P | |
| 18:57:22 | gmaan | :) | |
| 19:19:40 | opendevreview | sean mooney proposed openstack/nova master: FairLockGuard: Support cross-thread sharing and nesting https://review.opendev.org/c/openstack/nova/+/973438 | |
| 19:41:58 | opendevreview | Ghanshyam proposed openstack/nova master: Add 2nd RPC server for compute service https://review.opendev.org/c/openstack/nova/+/975242 | |
| 19:43:35 | opendevreview | Ghanshyam proposed openstack/nova master: Add 2nd RPC server for compute service https://review.opendev.org/c/openstack/nova/+/975242 | |
| #openstack-nova - 2026-01-31 | |||
| 00:36:25 | opendevreview | melanie witt proposed openstack/nova master: unified limits: Fix openstacksdk usage for endpoint discovery https://review.opendev.org/c/openstack/nova/+/975106 | |
| 03:02:21 | opendevreview | melanie witt proposed openstack/nova master: unified limits: Fix openstacksdk usage for endpoint discovery https://review.opendev.org/c/openstack/nova/+/975106 | |
| 03:02:21 | opendevreview | melanie witt proposed openstack/nova master: testing: Run functional tests under [testenv:cover] https://review.opendev.org/c/openstack/nova/+/975346 | |
| #openstack-nova - 2026-02-02 | |||
| 08:02:31 | obre | Hi! There is a BP (add-amx-traits) thats been lying around since 2023, which describes a feature I would like to see in nova, and that I guess Im able to implement. The BP is not approved, and posted by someone else. Should I in some way update the BP and assign it to myself, or should I create a new one to get it approved and then Implemented? | |
| 08:17:39 | mkuroha | gibi: Please take another look when you have time. I’ve addressed your comments. https://review.opendev.org/c/openstack/nova/+/948304 | |
| 08:41:21 | ykarel | bauzas, auniyal can you check os-vif release patch https://review.opendev.org/c/openstack/releases/+/975181 | |
| 10:03:22 | elodilles | auniyal bauzas Uggla : hi, sorry for pinging, but there are a couple of release patches that would require a release liaison review o:) if you have some spare time, then please take a look: https://review.opendev.org/q/project:openstack/releases+reviewer:self+is:open | |
| 11:15:59 | gibi | gmaan: thanks for the review on the eventlet removal side. I see you went back and forth about waiting for the deadline of the currently waited for task. I think I have a way out of that debate. Currently the libvirt driver will cancel() all the outstanding tasks on the executor then shut it down. So at shutdown the executor can be awaken, check if the current task is cancelled or not then decide | |
| 11:16:05 | gibi | what to do. This way a cancelled task's deadline does not need to be waited for | |
| 12:01:16 | opendevreview | Merged openstack/nova stable/2025.2: Reproduce bug/2130881 https://review.opendev.org/c/openstack/nova/+/975162 | |
| 12:48:06 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use an executor to delay STOPPED events https://review.opendev.org/c/openstack/nova/+/974445 | |
| 12:48:06 | opendevreview | Balazs Gibizer proposed openstack/nova master: Cleanup libvirt driver at service stop https://review.opendev.org/c/openstack/nova/+/975128 | |
| 12:48:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix full executor warning on noname executor https://review.opendev.org/c/openstack/nova/+/975172 | |
| 12:48:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 12:48:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Test with oslo.vmware + compute eventlet removal patches https://review.opendev.org/c/openstack/nova/+/973468 | |
| 12:49:05 | gibi | gmaan: ^^ improvement is up. I will still need to do a cleanup round on the patches (e.g. moving prints to log) | |
| 13:58:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use an executor to delay STOPPED events https://review.opendev.org/c/openstack/nova/+/974445 | |
| 13:58:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Cleanup libvirt driver at service stop https://review.opendev.org/c/openstack/nova/+/975128 | |
| 13:58:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix full executor warning on noname executor https://review.opendev.org/c/openstack/nova/+/975172 | |
| 13:58:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 13:58:04 | 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:05:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Cleanup libvirt driver at service stop https://review.opendev.org/c/openstack/nova/+/975128 | |
| 14:05:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix full executor warning on noname executor https://review.opendev.org/c/openstack/nova/+/975172 | |
| 14:05:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 14:05:46 | 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:05:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove spawn_after https://review.opendev.org/c/openstack/nova/+/975396 | |
| 14:06:57 | gibi | gmaan: and now the cleanups are also done | |
| 14:20:55 | gibi | gmaan: hold on I'm seeing issuess | |
| 14:32:52 | dansmith | gibi: did you see my response on the async detach patch and asking you for clarification? | |
| 14:33:20 | gibi | dansmith: not yet, but I will check now... | |
| 14:45:18 | gibi | dansmith: I replied | |
| 14:46:09 | dansmith | ack | |
| 14:48:21 | gibi | you basically answered my question, we have no issue here, I only have a minor request to translate 500 to 503 | |
| 14:50:48 | opendevreview | Merged openstack/nova stable/2025.2: Init virt driver before use https://review.opendev.org/c/openstack/nova/+/975163 | |
| 15:26:32 | Uggla | Reminder, nova meeting in ~30mn | |
| 15:51:26 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use an executor to delay STOPPED events https://review.opendev.org/c/openstack/nova/+/974445 | |
| 15:51:27 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove spawn_after https://review.opendev.org/c/openstack/nova/+/975396 | |
| 15:51:27 | opendevreview | Balazs Gibizer proposed openstack/nova master: Cleanup libvirt driver at service stop https://review.opendev.org/c/openstack/nova/+/975128 | |
| 15:51:28 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix full executor warning on noname executor https://review.opendev.org/c/openstack/nova/+/975172 | |
| 15:51:28 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 15:51:29 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Test with oslo.vmware + compute eventlet removal patches https://review.opendev.org/c/openstack/nova/+/973468 | |
| 15:52:51 | gibi | gmaan: ^^ this should better now, I promise :) | |
| 15:56:38 | opendevreview | Merged openstack/nova stable/2025.1: Reproduce bug/2130881 https://review.opendev.org/c/openstack/nova/+/975164 | |
| 16:01:58 | Uggla | #startmeeting nova | |
| 16:01:58 | opendevmeet | Meeting started Mon Feb 2 16:01:58 2026 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:01:58 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:01:58 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:02:06 | Uggla | Hello everyone | |
| 16:02:09 | erlon | \o | |
| 16:02:12 | gibi | o/ | |
| 16:02:13 | erlon | hey | |
| 16:02:18 | lajoskatona | o/ | |
| 16:02:35 | bauzas | o/ | |
| 16:02:39 | gmaan | o/ | |
| 16:02:39 | elodilles | o/ | |
| 16:03:04 | fwiesel | \o | |
| 16:03:10 | dansmith | oj | |
| 16:03:41 | Uggla | Let's start | |
| 16:03:44 | Uggla | #topic Bugs (stuck/critical) | |
| 16:03:50 | Uggla | #info No Critical bug | |
| 16:03:58 | tkajinam | o/ | |
| 16:03:58 | Uggla | #topic Gate status | |
| 16:04:09 | Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | |
| 16:04:15 | Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | |
| 16:04:21 | Uggla | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status | |
| 16:04:29 | Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | |
| 16:04:35 | Uggla | #info Please try to provide a meaningful comment when you recheck | |
| 16:04:56 | Uggla | Shame on me, I have not looked at the gate today, anything special with it ? | |
| 16:05:24 | gmaan | nova-alt-configurations is fixed now | |
| 16:05:44 | gmaan | tempest change is merged, so feel free to recheck for this failure | |
| 16:05:46 | gmaan | #link https://review.opendev.org/c/openstack/tempest/+/975356 | |
| 16:07:15 | Uggla | ok gmaan, good to know and thanks | |
| 16:07:29 | Uggla | anything else ? | |
| 16:07:53 | bauzas | - | |
| 16:08:22 | Uggla | ok so moving on | |
| 16:08:31 | Uggla | #link https://releases.openstack.org/gazpacho/schedule.html | |