| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-04-21 | |||
| 11:27:48 | sean-k-mooney | even if the code is effectivly equvlent https://review.opendev.org/c/openstack/nova/+/971176/9/nova/virt/hardware.py is an invaisve rewrite | |
| 11:35:28 | opendevreview | Anton Iacobaeus proposed openstack/nova-specs master: Intel TDX support in libvirt driver https://review.opendev.org/c/openstack/nova-specs/+/979608 | |
| 11:38:30 | DominikDanelski[m] | sean-k-mooney: Did you see my reply in https://review.opendev.org/c/openstack/nova/+/971176/1/nova/virt/hardware.py#b2509? I think the change to the overall logic is not as significant as it may look from the lines of diff, but I asked about your input there and I'm open to changes if you provide direction for them. | |
| 11:47:04 | sean-k-mooney | i did but again because it fundemtnally change the organstaion of the code i have not had tiem to fully unlopad all my expecations fo how that code shoudl work adn rereview it based on yoru changes | |
| 11:47:19 | sean-k-mooney | as i said that refactoring makes it a lot harder to review your change | |
| 11:50:42 | sean-k-mooney | if you wanted to make that easier to review you would do the refactor in its own patch that does not change any exisitng behivor. although im not entirly conveice that is the correct move. it would make it much clear what is jsut a refactorign vs behvior change | |
| 12:13:50 | opendevreview | Esra Ozkan proposed openstack/nova master: Check volume status before live migration https://review.opendev.org/c/openstack/nova/+/973750 | |
| 12:22:15 | opendevreview | Elod Illes proposed openstack/nova stable/2025.2: fix: device_by_alias should respect config type https://review.opendev.org/c/openstack/nova/+/984920 | |
| 12:24:08 | sean-k-mooney | Uggla: so its a little disapointing that the lack fo openstack client and sdk? supprot for nova share attatchment was not coverd in teh manilla cross project session | |
| 12:24:25 | sean-k-mooney | Uggla: i dont think we shoudl apcpate any new feature there really until that gap is closed | |
| 12:24:59 | sean-k-mooney | i dont object to any of the feature but without afctully coing that gap so you dont have to use curl im not sure we should keep exteneding it | |
| 12:25:18 | sean-k-mooney | oh... | |
| 12:25:22 | sean-k-mooney | sorry i msiste it | |
| 12:25:32 | Uggla | sean-k-mooney, good news the sdk and client were merged, it was the first topic of yesterday meeting. | |
| 12:25:47 | sean-k-mooney | ya i ddint see that initally | |
| 12:25:50 | sean-k-mooney | all good then | |
| 12:26:21 | Uggla | sean-k-mooney, gouthamr take over this and manage to cross the finish line. | |
| 12:27:15 | Uggla | sean-k-mooney no worries. | |
| 12:29:21 | Uggla | sean-k-mooney, also carloss presented good progress on the testing side. | |
| 12:30:42 | sean-k-mooney | shoudl i ask if said testing found bugs or not | |
| 12:33:16 | Uggla | sean-k-mooney, I would say yes at least one I'm aware of it --> 984802: Set instance to ERROR state on start_instance power_on failure | https://review.opendev.org/c/openstack/nova/+/984802 | |
| 12:40:08 | carloss | sean-k-mooney: yep, the change Uggla linked was an issue gouthamr ran into. but so far, in the scenario (end to end) tests I have implemented, no further bugs found | |
| 12:40:37 | carloss | this is a job run using manila's cephfs native: https://zuul.opendev.org/t/openstack/build/f761d9016979435abe69c849a645ebc2 | |
| 12:41:27 | carloss | I'm still figuring out why LVM (with NFS) is failing on CI because it's passing in a local env I have - so I think there's some mismatch in the env I own compared to the CI | |
| 12:41:57 | carloss | the share attachment tests are under the TestShareAttachmentCEPHFS class | |
| 12:44:36 | sean-k-mooney | Uggla: oh the resoume on host boot feature | |
| 12:45:00 | sean-k-mooney | ya im betting we didnt re attach the share or something? | |
| 12:45:04 | sean-k-mooney | that woudl have been easy to miss | |
| 12:48:22 | opendevreview | Merged openstack/nova master: objects: Prepare for oslo.versionedobjects 3.10.0 https://review.opendev.org/c/openstack/nova/+/985506 | |
| 12:54:31 | DominikDanelski[m] | sean-k-mooney: OK, so I propose that we split the change in 2: first the behaviour change without the signature change (extending the function's behaviour at the expense of making it long), then the split which would end in the current state. It would be hard to show the refactor without the reason it's being done (extending behaviour). | |
| 13:56:10 | opendevreview | Karol Klimaszewski proposed openstack/nova-specs master: Spec for SPDK-based nova-provisioned storage backend https://review.opendev.org/c/openstack/nova-specs/+/985676 | |
| 13:59:39 | tkajinam | as it's not likely we have time to discuss this... I wonder if that namespace package thing is what we broken when we removed pkg_resources ? | |
| 14:00:00 | tkajinam | I intended to keep that functionality but that was later broken by canonical folks insisting "this doesn't work for us" | |
| 14:00:43 | tkajinam | I might be confused by https://review.opendev.org/c/openstack/nova/+/939827 , though | |
| 14:01:20 | tkajinam | ok it seems nova.scheduler.filters has never been exposed as a namespace package | |
| 14:01:54 | tkajinam | we probably want to use the entrypoints approach instead | |
| 14:20:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: test if CLI testing hitting scatter gather https://review.opendev.org/c/openstack/nova/+/985681 | |
| 14:30:56 | opendevreview | Karol Klimaszewski proposed openstack/nova-specs master: Spec for SPDK-based nova-provisioned storage backend https://review.opendev.org/c/openstack/nova-specs/+/985676 | |
| 15:04:21 | cardoe | Thanks all for having the Ironic folks. | |
| 15:15:09 | opendevreview | Goutham Pacha Ravi proposed openstack/nova-specs master: Add spec for virtiofs hot attach and detach https://review.opendev.org/c/openstack/nova-specs/+/983816 | |
| 16:43:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: test if CLI testing hitting scatter gather https://review.opendev.org/c/openstack/nova/+/985681 | |
| 18:07:56 | cardoe | dansmith: I think my comments were misconstrued a little bit and I'd like to clear things up. | |
| 18:09:17 | cardoe | There's a number of users / operators of the nova-ironic driver that I'm aware of. That driver hasn't been a prime focus of the nova cores team so a lot of times patches kinda stall out a little bit. The testing for that driver in Zuul isn't the best as was highlighted during the call. | |
| 18:10:16 | cardoe | So what I'm doing is trying to get those users / operators to share some changes together that are important to them and trying to figure out the best way for us to voice that "this bug/issues is important to that group" | |
| 18:11:29 | cardoe | The reason for the repo idea is that more than once it's happened that I've spoken to one of them and they've said "oh we've iterated on the patch a little bit and use a new version". So just trying to come up with a way for them to do that more in the open. | |
| 18:17:52 | dansmith | cardoe: sounds pretty much like what you said on the call (although the context around it at the time is the important but). Sounds like a fork to me, which doesn't seem very constructive... | |
| 18:18:24 | dansmith | but, you don't need to convince me, I responded to your comment in the moment in the context of the moment | |
| 18:21:42 | cardoe | Can you give me suggestions how we get fixes that are important to the users of nova-ironic landed? | |
| 18:23:07 | opendevreview | Bryan Fraschetti proposed openstack/nova master: Reject to attach and detach volume while task_state is not None https://review.opendev.org/c/openstack/nova/+/758918 | |
| 18:23:08 | opendevreview | Bryan Fraschetti proposed openstack/nova master: Feat: Add Command for Forceful BDM Deletion https://review.opendev.org/c/openstack/nova/+/985717 | |
| 18:23:08 | opendevreview | Bryan Fraschetti proposed openstack/nova master: add task state for attach volume and detach volume https://review.opendev.org/c/openstack/nova/+/985718 | |
| 18:23:09 | opendevreview | Bryan Fraschetti proposed openstack/nova master: Reject Volume Attachment if task_state not Attaching https://review.opendev.org/c/openstack/nova/+/985719 | |
| 18:25:38 | sean-k-mooney | ok im goign to call it a day. ill be in the watcher session most fo tomorrow but if im meeded for a session ping me and ill try an attend | |
| 18:33:16 | cardoe | https://bugs.launchpad.net/nova/+bug/1884217 https://bugs.launchpad.net/nova/+bug/1732506 (mentioned by johngarbutt during the session) https://bugs.launchpad.net/nova/+bug/2127087 https://bugs.launchpad.net/nova/+bug/2085432 https://bugs.launchpad.net/nova/+bug/2131095 https://bugs.launchpad.net/nova/+bug/2148059 https://bugs.launchpad.net/nova/+bug/2117700 | |
| 18:33:47 | cardoe | That's just the list of bugs that I'm aware of that operators have mentioned | |
| 18:35:52 | cardoe | I don't have a list of gerrit patches handy | |
| 18:44:04 | melwitt | gmaan: dunno if you might be able to shed any light on this, but in CI the nova.tests.functional.test_servers.ServerMovingTestsWithNestedResourceRequests.test_resize_reschedule_uses_host_lists_3_fails test has been flaking out and I can't see why, | |
| 18:44:43 | melwitt | the last messages in the traceback are graceful shutdown stuff and I assume they're not related, but I wondered what you might think. example: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_af0/openstack/af0437afd3354ef990fcb3d6aa47f80a/testr_results.html | |
| 20:12:41 | opendevreview | Goutham Pacha Ravi proposed openstack/nova-specs master: Add spec for virtiofs hot attach and detach https://review.opendev.org/c/openstack/nova-specs/+/983816 | |
| 21:52:47 | gmaan | let me check | |
| 22:13:32 | gmaan | melwitt: it seems all nova services are shutdown in between of tests. | |
| 22:14:29 | gmaan | functional tests override the graceful shutdown to 0 so shutdown will be immediately without waiting for things to finish and that is why server is stuck in that state | |
| 22:14:39 | gmaan | not sure why shutdown happen | |
| 22:15:40 | melwitt | gmaan: thanks for looking. so this is unexpected? that is a good clue for me to continue debugging | |
| 22:16:06 | gmaan | yeah, shutdown itself should not be initiated in between of test | |
| 22:57:32 | opendevreview | Goutham Pacha Ravi proposed openstack/nova-specs master: Add virtiofs live migration spec https://review.opendev.org/c/openstack/nova-specs/+/983801 | |
| 23:01:43 | opendevreview | Goutham Pacha Ravi proposed openstack/nova-specs master: Add spec for virtiofs live migration https://review.opendev.org/c/openstack/nova-specs/+/983801 | |
| 23:01:50 | opendevreview | Goutham Pacha Ravi proposed openstack/nova-specs master: Add virtiofs cold migration spec https://review.opendev.org/c/openstack/nova-specs/+/985738 | |
| #openstack-nova - 2026-04-22 | |||
| 00:08:09 | opendevreview | melanie witt proposed openstack/nova master: Increase wait retries for resize reschedule functional test https://review.opendev.org/c/openstack/nova/+/985741 | |
| 00:09:41 | melwitt | gmaan: just an update, the graceful shutdown was unrelated and was just the test cleanup happening after the wait for server state failed ^ | |
| 01:00:06 | gmaan | melwitt: ohk, thanks for fix, +2 | |
| 01:01:02 | melwitt | thanks gmaan | |
| 06:25:09 | opendevreview | Rico Lin proposed openstack/nova master: compute: add [compute] skip_inventory_in_use_errors https://review.opendev.org/c/openstack/nova/+/985750 | |
| 06:26:30 | opendevreview | Rico Lin proposed openstack/nova master: compute: add skip_inventory_in_use_errors opt https://review.opendev.org/c/openstack/nova/+/985750 | |
| 08:01:57 | opendevreview | Merged openstack/nova master: Increase wait retries for resize reschedule functional test https://review.opendev.org/c/openstack/nova/+/985741 | |
| 09:02:49 | elodilles | bauzas Uggla : i know this is PTG week, but i've prepared nova release patches, mainly because of we could do a final 2024.2 Dalmatian release before it moves to EOL. could you please review them? https://review.opendev.org/q/project:openstack/releases+is:open+intopic:nova | |
| 09:03:22 | Uggla | Hi elodilles, I'm gonna have a look | |
| 09:06:30 | elodilles | thx in advance o/ | |
| 09:26:00 | opendevreview | ribaudr proposed openstack/nova master: ironic: Optimize get_num_instances to avoid unnecessary API calls https://review.opendev.org/c/openstack/nova/+/985785 | |
| 09:32:21 | bauzas | elodilles: ack, will look | |
| 09:48:46 | elodilles | thanks, too o/ | |
| 10:15:45 | Uggla | bauzas, elodilles, if you can have a look at 985791: Add Nova deadlines to the 2026.2 Hibiscus schedule | https://review.opendev.org/c/openstack/releases/+/985791 (no hurries) | |
| 10:29:47 | elodilles | Uggla: ack, will do | |
| 11:22:48 | opendevreview | sean mooney proposed openstack/nova master: [dnm] testing with python 3.14 https://review.opendev.org/c/openstack/nova/+/985802 | |
| 11:23:40 | sean-k-mooney | Uggla: gmaan sambork so i hacked up some py 3.14 testing yesteday and got some interesting result on https://review.opendev.org/c/openstack/devstack/+/985735 | |
| 11:24:01 | sean-k-mooney | we can boot vms but some things fail | |
| 11:24:11 | sean-k-mooney | for example some of the novnc tests | |
| 11:24:30 | sean-k-mooney | the dnm to nova above shoudl give us better results fro our edgecases | |
| 11:25:07 | sean-k-mooney | basicly i have a WIP serise for devtack to use uv to provide python and replace pip with uv pip | |
| 11:25:30 | sean-k-mooney | what that allows use to do is use basicly any python verion on any distro within reason | |
| 11:26:23 | sean-k-mooney | so with that dnm all our jobs regrdless of the disto will run on python 3.14 and give ues a good overviewr of compatiablity | |
| 11:27:42 | sean-k-mooney | gibi: tkajinam sambork so the novnc issue iw the pikel issue we were discussing in the eventlet removeal room https://zuul.opendev.org/t/openstack/build/6f2857af77aa41eabb9f28a077081d89/log/controller/logs/screen-n-novnc-cell1.txt#16 | |
| 11:32:09 | sean-k-mooney | we can see that tempest.api.compute.volumes.test_attach_volume.AttachVolumeMultiAttachTest.test_snapshot_volume_backed_multiattach and other complex test passed so for the most part we seam to be in a relitively good place | |
| 11:55:18 | sambork | nice findings, I'm curious if this will pass with threading mode (because patch for proxies runing with threading are still under review https://review.opendev.org/c/openstack/nova/+/976089) | |
| 11:55:48 | sean-k-mooney | you could add a second dnm on top of your patch | |
| 11:55:58 | sean-k-mooney | you just need to depend on the devstack dnm | |
| 11:56:15 | sean-k-mooney | but ya im not sure if the pikel issue is eventlet related or not | |
| 11:56:40 | sean-k-mooney | sambork: or actully once my currnt dnm reports back | |
| 11:56:46 | sean-k-mooney | i can rebase it on yoru review | |
| 11:57:09 | sean-k-mooney | i just eamiled the list so other project can tyr this themselve if they want | |