| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-09-30 | |||
| 14:33:50 | gibi | but no proof yet | |
| 14:40:59 | dansmith | gibi: probably need to backport that bug fix yeah? | |
| 14:58:53 | gibi | yeah it feel backportable | |
| 15:18:07 | opendevreview | Dan Smith proposed openstack/nova-specs master: Add cross-cell-live-migration spec https://review.opendev.org/c/openstack/nova-specs/+/1008081 | |
| 15:25:12 | Uggla | Reminder: upstream triage in ~5mn (https://meet.google.com/zjr-rxus-hzj) | |
| 17:01:30 | opendevreview | Stephen Finucane proposed openstack/os-vif master: Remove use of testscenarios https://review.opendev.org/c/openstack/os-vif/+/1008029 | |
| 17:01:31 | opendevreview | Stephen Finucane proposed openstack/os-vif master: tox: Add functional-py* targets https://review.opendev.org/c/openstack/os-vif/+/1008097 | |
| 17:20:45 | JayF | Uggla: https://review.opendev.org/c/openstack/placement/+/990761 adamcarthur5 from my team was asking about the best way to get this on the nova radar; and if it'll need PTG time. He's the primary maintainer of openstack-exporter for prometheus, which needs the bulk read API to operate performantly | |
| 17:37:47 | dansmith | JayF: I was waiting for something else so I added some radar pings | |
| 17:39:11 | dansmith | I dunno what Uggla's POV is but unless I'm missing a bunch about the gaps (entirely possible) it could benefit from a PTG discussion about whether we even want the "mysqldump | jsonify" workflow as an option | |
| 17:41:37 | JayF | What's a radar ping? | |
| 17:42:06 | adamcarthur5 | Hi dansmith - I am planning on writing the patches for this change. I will answer your comments today. To be honest, I just got a patch up as a quick draft, to try and get some feedback. I'll pad things out. | |
| 17:42:35 | dansmith | JayF: you said "get it on the nova radar" | |
| 17:42:40 | JayF | oooooh | |
| 17:43:01 | dansmith | adamcarthur5: ack, it's pretty light on detail, so... | |
| 19:41:56 | opendevreview | minwoo seo proposed openstack/nova master: libvirt: Add volume migration bandwidth option to nova.conf https://review.opendev.org/c/openstack/nova/+/999841 | |
| 21:57:55 | 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 | |
| 22:09:38 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Add hot attach/detach support for virtiofs shares https://review.opendev.org/c/openstack/nova/+/1008154 | |
| #openstack-nova - 2026-10-01 | |||
| 00:10:41 | opendevreview | Adam McArthur proposed openstack/placement master: Placement spec for bulk read API of resource providers https://review.opendev.org/c/openstack/placement/+/990761 | |
| 02:14:04 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Add hot attach/detach support for virtiofs shares https://review.opendev.org/c/openstack/nova/+/1008154 | |
| 07:32:21 | opendevreview | Juan Larriba proposed openstack/nova master: Add OpenTelemetry tracing middleware for Nova https://review.opendev.org/c/openstack/nova/+/982871 | |
| 07:34:31 | opendevreview | Juan Larriba proposed openstack/nova master: Add OpenTelemetry tracing middleware for Nova https://review.opendev.org/c/openstack/nova/+/982871 | |
| 07:39:09 | opendevreview | Juan Larriba proposed openstack/nova master: Add OpenTelemetry tracing middleware for Nova https://review.opendev.org/c/openstack/nova/+/982871 | |
| 08:11:43 | opendevreview | Fabian Wiesel proposed openstack/nova master: Clean up stuck/leaked volumes in BFV scenarios on failed build or early deletion https://review.opendev.org/c/openstack/nova/+/1008215 | |
| 08:16:36 | opendevreview | Fabian Wiesel proposed openstack/nova master: Clean up stuck/leaked volumes in BFV scenarios on failed build or early deletion https://review.opendev.org/c/openstack/nova/+/1008215 | |
| 09:19:18 | opendevreview | Fabian Wiesel proposed openstack/nova master: Clean up stuck/leaked volumes in BFV scenarios on failed build or early deletion https://review.opendev.org/c/openstack/nova/+/1008215 | |
| 09:29:06 | bauzas | gibi: thanks gibi for your excellent comments on https://review.opendev.org/c/openstack/nova/+/1006786/3 currently working on another revision | |
| 10:16:06 | opendevreview | Sylvain Bauza proposed openstack/nova master: Prevent concurrent attach_volume with task_state guard https://review.opendev.org/c/openstack/nova/+/1006786 | |
| 10:16:06 | opendevreview | Sylvain Bauza proposed openstack/nova master: Guard against deleting a concurrent request's BDM on timeout https://review.opendev.org/c/openstack/nova/+/1005727 | |
| 10:16:07 | opendevreview | Sylvain Bauza proposed openstack/nova master: Reject stale reserve_block_device_name RPC on compute https://review.opendev.org/c/openstack/nova/+/1006011 | |
| 10:31:30 | gibi | bauzas: ack | |
| 10:31:40 | gibi | sorry for not spotting that sooner | |
| 10:36:14 | bauzas | don't be sorry, I never saw it | |
| 11:55:55 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2026.2: CellDatabases: replace shared lock with thread-local cell routing https://review.opendev.org/c/openstack/nova/+/1008257 | |
| 11:56:52 | gibi | melwitt: bauzas: elodilles: ^^ this helped avoiding the functional / cover job hanging so lets add it to 2026.2 as well ^^ | |
| 11:57:59 | gibi | I +Ad https://review.opendev.org/c/openstack/nova/+/1007996 making tox cover voting again, I will propose a backport for that as well once it lands on master | |
| 12:33:40 | elodilles | gibi: +2'd | |
| 12:36:13 | gibi | cool, thanks | |
| 12:36:34 | elodilles | np o/ | |
| 12:57:22 | gibi | bauzas: there is a new ~bug in https://review.opendev.org/c/openstack/nova/+/1006786 sorry :) | |
| 12:57:33 | bauzas | doh | |
| 12:57:47 | gibi | but we are getting there | |
| 12:58:00 | bauzas | sorry I'm neither a notification nor a BDM SME :) | |
| 12:58:12 | Uggla | dansmith, adamcarthur5, I agree Dan, we already had such kind of request and this is tricky regarding performances. So we might not need a X session with Ironic, but discussing the use case at the PTG sounds a good start. | |
| 12:58:26 | gibi | no worries this new one is just a try block restructuring issue :) | |
| 13:00:52 | bauzas | gibi: thanks, I'll provide a new rev :) | |
| 13:00:56 | bauzas | gibi++ | |
| 13:01:14 | gibi | and I will re-review it :) | |
| 13:19:19 | opendevreview | Sylvain Bauza proposed openstack/nova master: Reject stale reserve_block_device_name RPC on compute https://review.opendev.org/c/openstack/nova/+/1006011 | |
| 13:19:19 | opendevreview | Sylvain Bauza proposed openstack/nova master: Prevent concurrent attach_volume with task_state guard https://review.opendev.org/c/openstack/nova/+/1006786 | |
| 13:31:36 | gibi | bauzas: thanks +2 | |
| 15:08:33 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run periodics outside compute test callbacks https://review.opendev.org/c/openstack/nova/+/1007150 | |
| 15:36:16 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run periodics outside compute test callbacks https://review.opendev.org/c/openstack/nova/+/1007150 | |
| 16:06:56 | gibi | gmaan: smabork: I respin https://review.opendev.org/c/openstack/nova/+/1007150 with the run_periodics improvement | |
| 16:07:12 | gibi | I've respun (?) | |
| 16:08:09 | gibi | anyhow I fixed the oustanding comment in that patch :) | |
| 16:57:56 | gmaan | gibi: ack | |
| 17:25:56 | opendevreview | Merged openstack/nova master: Add pyroute2 in doc dependencies https://review.opendev.org/c/openstack/nova/+/1004781 | |
| 17:49:40 | opendevreview | Merged openstack/nova stable/2026.2: CellDatabases: replace shared lock with thread-local cell routing https://review.opendev.org/c/openstack/nova/+/1008257 | |
| 17:49:42 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Stop hanging the test if threadpool executor is stuck in cleanup https://review.opendev.org/c/openstack/nova/+/1006351 | |
| 18:36:59 | dansmith | melwitt: re-reading your comment about naming it V2, I thought you had maybe not grokked (or not grokked when you wrote that) that this'd be a permanent replacement eventually | |
| 18:37:27 | dansmith | but maybe you _did_ understand that and still don't think that it needs "v2" or "new" or "now better than ever" in the name | |
| 18:37:58 | dansmith | it seems like it might be hard to explain or confuse people if we don't, but still valid if that's your preference | |
| 18:38:40 | dansmith | I'd think it becomes one of those things where it's hard for them to remember which is the newer one they should be building on when they just look at the name. obviously if they look at the microversion history it'll be clear, but... | |
| 18:39:48 | melwitt | dansmith: yes I did understand it to be an eventual replacement. I don't have a rigid opinion, I was just thinking about when it _is_ the only available API it being named "V2" in the rest API is kinda eh ... but again not feeling that certain about it. it's true we already have like "block_device_mapping_v2" and it would be like that | |
| 18:41:09 | dansmith | yes, I both hate the "only one and it's v2" and I also hate block_device_mapping_v2 as well | |
| 18:41:35 | melwitt | if the API were not intended to be an eventual replacement I would not be supportive of the new API idea heh | |
| 18:41:37 | dansmith | it's just one of those things where the v2 is an ugly wart but "block_device_mapping" and device_mapping_block" would be way too confusing trying to remember which one was new | |
| 18:42:10 | dansmith | anyway, just wanted to make sure we're on the same page, and sounds like we are | |
| 18:42:23 | melwitt | sure | |
| 19:21:47 | opendevreview | Michael Sherman proposed openstack/nova master: ironic: Reproduce node leak after local delete https://review.opendev.org/c/openstack/nova/+/1008334 | |
| 19:47:23 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Reduce the unnecessary executors shutdown logs in test jobs https://review.opendev.org/c/openstack/nova/+/1004160 | |
| 20:06:47 | opendevreview | Merged openstack/nova master: Revert "Make tox cover non voting to unblock RC1" https://review.opendev.org/c/openstack/nova/+/1007996 | |
| 20:41:46 | adamcarthur5 | assumptions when I benchmark. | |
| 20:41:46 | adamcarthur5 | Uggla dansmith sean-k-mooney thank you for your comments, here and on the review. I will work through them. In terms of the PTG, is there an ideal date/time that I should suggest in the etherpad? In terms of performance, I agree with you. I am happy to work on performance testing for such a change, I would just need some help for making the right | |
| 20:42:36 | adamcarthur5 | If benchmarking potential changes would be something useful pre-PTG, I can work on that? I.e. maybe the implementation is not quite right, but we can get a sense of performance? Does that make sense? | |
| 20:54:40 | JayF | adamcarthur5: I can't speak for how nova does things, but if you're getting reviews on your spec and making design progress async, a PTG session may be less needed. I'd suggest prioritizing quick roundtrips on those reviews. | |
| 20:55:07 | adamcarthur5 | JayF I was basing it off what they said early in the chat ^^^^ | |
| 20:55:19 | adamcarthur5 | But yes quick turn around on the reviews is also the plan | |
| 20:55:47 | JayF | I mainly wanted you to know if your spec is getting reviewed, you're already making good progress :) | |
| 20:56:07 | adamcarthur5 | 👍 | |
| 21:13:10 | opendevreview | Adam McArthur proposed openstack/nova master: Use safe allocation deletion on instance delete https://review.opendev.org/c/openstack/nova/+/1003208 | |
| #openstack-nova - 2026-10-02 | |||
| 00:12:56 | gmaan | gibi: sambork these two are ready https://review.opendev.org/c/openstack/nova/+/1006351 https://review.opendev.org/c/openstack/nova/+/1004160 | |
| 00:15:19 | opendevreview | Ghanshyam Maan proposed openstack/nova stable/2026.2: Revert "Make tox cover non voting to unblock RC1" https://review.opendev.org/c/openstack/nova/+/1008367 | |
| 00:19:32 | gmaan | gibi: elodilles ^^ as fix is backported, making it voting on 2026.2 too | |
| 02:39:16 | opendevreview | Takashi Kajinami proposed openstack/nova master: DNM: testing https://review.opendev.org/c/openstack/nova/+/1008371 | |
| 06:37:48 | opendevreview | Juan Larriba proposed openstack/nova master: Add OpenTelemetry tracing middleware for Nova https://review.opendev.org/c/openstack/nova/+/982871 | |
| 09:52:44 | gibi | gmaan: thanks I put my positive votes :) | |
| 09:53:00 | gibi | gmaan: when you are up this is also good to go now in my eyes https://review.opendev.org/c/openstack/nova/+/1001369 | |
| 12:17:26 | opendevreview | Stephen Finucane proposed openstack/os-vif master: trivial: Fix indentation in pyproject.toml https://review.opendev.org/c/openstack/os-vif/+/1008450 | |
| 12:32:28 | Uggla | adamcarthur5, JayF, getting reviews is the main issue due to our limited review capacity. That's one of the reasons why we'd like to have a topic at the PTG and, if possible, a spec. The idea is to understand the use case and agree on a direction for addressing it. | |
| 12:35:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix cross-cell cold migration with PCI devices https://review.opendev.org/c/openstack/nova/+/1008071 | |
| 12:35:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Func reproducer for bug/2168874 https://review.opendev.org/c/openstack/nova/+/1007917 | |
| 12:44:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run periodics outside compute test callbacks https://review.opendev.org/c/openstack/nova/+/1007150 | |
| 13:40:14 | opendevreview | Kamil Sambor proposed openstack/nova master: Create an executor wrapper that has task limit per type https://review.opendev.org/c/openstack/nova/+/975924 | |
| 15:08:24 | opendevreview | Merged openstack/os-vif master: trivial: Fix indentation in pyproject.toml https://review.opendev.org/c/openstack/os-vif/+/1008450 | |
| 16:10:46 | gmaan | gibi: done, thanks | |
| 16:10:52 | gmaan | gibi: i think you missed this? https://review.opendev.org/c/openstack/nova/+/1006351/7 | |
| 23:08:53 | opendevreview | taha hosseini proposed openstack/nova master: libvirt: assign virtio storage devices to guest IOThread https://review.opendev.org/c/openstack/nova/+/1008546 | |
| #openstack-nova - 2026-10-03 | |||
| 06:20:33 | opendevreview | taha hosseini proposed openstack/nova master: libvirt: assign virtio storage devices to guest IOThread https://review.opendev.org/c/openstack/nova/+/1008546 | |