| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-10-01 | |||
| 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 | |
| 06:28:26 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Add live migration support for virtiofs shares https://review.opendev.org/c/openstack/nova/+/1008566 | |
| 06:28:26 | 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:26:26 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2027.1 Indri https://review.opendev.org/c/openstack/nova/+/1008569 | |
| 08:24:06 | opendevreview | taha hosseini proposed openstack/nova master: libvirt: assign virtio storage devices to guest IOThread https://review.opendev.org/c/openstack/nova/+/1008546 | |
| 08:48:22 | opendevreview | Merged openstack/nova master: Fix init_host crash migration test under native threading https://review.opendev.org/c/openstack/nova/+/1001369 | |
| #openstack-nova - 2026-10-05 | |||
| 00:50:21 | opendevreview | Masanori Ueno proposed openstack/nova master: doc: Update nova-api restart command for Ubuntu https://review.opendev.org/c/openstack/nova/+/1008734 | |
| 08:51:58 | gibi | gmaan: indeed I missed that. Looking at it now | |
| 08:58:55 | yunhokim_ | There's no nova-spec yet, but I can write one before the PTG if that would help the discussion. | |
| 08:58:55 | yunhokim_ | Background: this was proposed as a Glance spec at the 2026.2 PTG (https://review.opendev.org/c/openstack/glance-specs/+/984902), but it stalled because it also needs Nova/Cinder-side changes, so I'm bringing it to Nova this time. | |
| 08:58:55 | yunhokim_ | Blueprint: https://blueprints.launchpad.net/nova/+spec/nova-boot-from-backup-based-image | |
| 08:58:55 | yunhokim_ | Topic: booting servers from Cinder-backup-based images | |
| 08:58:55 | yunhokim_ | Hi all, I'd like to add a topic to the nova-cinder cross-project session at the PTG. | |
| 08:58:56 | yunhokim_ | Is it OK if I add it to the cross-session section of the etherpad? | |
| 09:03:45 | gibi | Uggla: ^^ | |
| 09:06:10 | Uggla | yunhokim_ can you add your topic at the bottom of the doc https://etherpad.opendev.org/p/nova-2027.1-ptg , I think we will have enough time to cover it during the cinder/nova session. | |
| 10:04:40 | yunhokim_ | Uggla: i've added the topic at the bottom of the etherpad. PTAL, thanks! | |
| 10:04:59 | Uggla | yunhokim_ 👍 | |
| 11:14:21 | opendevreview | Merged openstack/nova stable/2026.2: Revert "Make tox cover non voting to unblock RC1" https://review.opendev.org/c/openstack/nova/+/1008367 | |
| 13:06:17 | opendevreview | Takashi Kajinami proposed openstack/nova master: Replace deprecated sqlalchemy.orm.noload https://review.opendev.org/c/openstack/nova/+/1007632 | |
| 13:20:28 | stephenfin | gibi: jfyi https://review.opendev.org/c/openstack/os-vif/+/1004780 | |
| 13:25:00 | lajoskatona | Uggla: Hi, I opened a bug for the neutron SDK work for Nova: https://bugs.launchpad.net/nova/+bug/2169539 this can make tracking easier | |
| 13:25:32 | lajoskatona | Uggla: today I can't participate the meeting, or only the first ~15mins | |
| 13:47:33 | opendevreview | Lajos Katona proposed openstack/nova master: libvirt: Add test for MTU overwrite during live migration https://review.opendev.org/c/openstack/nova/+/995803 | |
| 13:47:41 | opendevreview | Matěj Mudra proposed openstack/nova master: Add --verbose to nova-manage commands https://review.opendev.org/c/openstack/nova/+/999956 | |
| 13:58:44 | opendevreview | sean mooney proposed openstack/nova-specs master: Specify atomic PCI passthrough groups for Nova https://review.opendev.org/c/openstack/nova-specs/+/975635 | |
| 14:01:10 | opendevreview | Lajos Katona proposed openstack/nova master: libvirt: Preserve source MTU during live migration https://review.opendev.org/c/openstack/nova/+/995809 | |
| 14:01:21 | sean-k-mooney | gibi: no rush but i think ^ has adressed any remaining opens while maintianing the core or the v5 desgin intact, the addtion of `exclude` come directly form my testing with modern nvida gpus, they all seam to use the same pci vendor id and product id for PF and mig VFs making it a pain to list them otherwise. | |
| 14:25:41 | bauzas | dansmith: I don't know if you have either will or capacity for reviewing https://review.opendev.org/c/openstack/nova/+/1005727/ but it would be nice if you could do so for at least the first two patches in the series | |
| 14:26:11 | dansmith | bauzas: yes sorry I've had it open and keep getting pulled into other things | |
| 14:26:17 | dansmith | bauzas: keep nagging | |
| 14:26:23 | bauzas | no worries, we all feel the same | |
| 14:26:56 | bauzas | semi-related I got the chance to be free for 1.5h so I eventually was able to sit down and read your spec | |
| 14:32:16 | dansmith | ack, thanks | |
| 14:54:33 | gibi | folks I've closed https://bugs.launchpad.net/nova/+bug/2165149 as the last hit was 13 days ago. If you see functional failure with "qlalchemy.exc.OperationalError: (sqlite3.OperationalError) database is locked" then please reopen the bug and ping me | |
| 14:56:14 | sean-k-mooney | gibi: nice | |
| 14:56:48 | dansmith | bauzas: sorry I thought more had changed here but it looks like maybe unintentional change of a lot of nearby lines in the second patch that looks intentional/functional but isn't? | |
| 14:58:01 | sean-k-mooney | dansmith: we might eb able to add a signle line to agetns.md to prevent that | |
| 14:58:18 | sean-k-mooney | the issue with the wrapping of the comments is because pep8 requires we use a shortor 72 column limit | |
| 14:58:20 | dansmith | sean-k-mooney: yeah I think we should.. I have an instruction in my local one | |
| 14:58:23 | sean-k-mooney | but nova does nto follow that convention | |
| 14:58:58 | dansmith | sean-k-mooney: well, there are other such things which are purely style preference that get changed | |
| 14:59:26 | dansmith | sean-k-mooney: I'll propose an agents change and tag you | |
| 14:59:56 | sean-k-mooney | acki didnt really look closely at the other changes but ya it the second paragrah im 90% for the comments at least https://peps.python.org/pep-0008/#maximum-line-length | |
| 15:00:13 | sean-k-mooney | ```For flowing long blocks of text with fewer structural restrictions (docstrings or comments), the line length should be limited to 72 characters.``` | |
| 15:04:02 | opendevreview | Dan Smith proposed openstack/nova master: Add agent instructions about gardening https://review.opendev.org/c/openstack/nova/+/1008817 | |
| 15:04:07 | dansmith | sean-k-mooney: ^ | |
| 15:04:19 | sean-k-mooney | :) gardening | |
| 15:04:49 | dansmith | a human referred to this using that term years ago, so it's always what I call it now | |
| 15:05:17 | sean-k-mooney | ack the phasing in the guardrails looks good to me | |
| 15:07:54 | sean-k-mooney | i have started experimenting with https://github.com/DietrichGebert/ponytail/blob/main/skills/ponytail/SKILL.md and https://github.com/cursor/plugins/blob/main/pstack/skills/unslop/SKILL.md lately in my own use | |
| 15:10:05 | sean-k-mooney | i have not used either enouch to say it definitivly helps but anicdtally they have. im reseving judgement unitll i atempt larger work with them | |
| 15:11:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run periodics outside compute test callbacks https://review.opendev.org/c/openstack/nova/+/1007150 | |
| 15:13:27 | sean-k-mooney | gibi: hum that intereesting i didnt think about the fact that the preodic woudl be in a diffent treadh/db context in that case | |
| 15:13:47 | dansmith | oh I like the unslop one | |
| 15:14:18 | dansmith | normally I just stop what I'm doing to verbally berate the agent for being so agentic in is verbiage.. makes me feel better. | |
| 15:14:22 | sean-k-mooney | that one i have found useful even if you jsut copy paste part fo it in your personal agent md | |
| 15:16:39 | gibi | sean-k-mooney: we basically sometimes triggering the periodic job of compute B in the computeA RPC handeler | |
| 15:16:51 | sean-k-mooney | i have only been using those for about 2-3 weeks so again i can say it impoved but it feels better. | |
| 15:17:15 | sean-k-mooney | gibi: oh ok ya that woudl be incorrect | |
| 15:17:36 | sean-k-mooney | escially if a and b were in differnet cells ectra | |
| 15:18:27 | gibi | yeah these tests were complicated :) | |
| 15:20:03 | gibi | gmaan: btw I finally reviewed https://review.opendev.org/c/openstack/nova/+/1006351 I have a bunch of small things that together might make sense to fix | |
| 15:20:24 | sean-k-mooney | general oslo.servce questsion https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.run_external_periodic_tasks, i dont think nova really supprot changing that form its defautl as to my knowlage we dont have any external perodic like that | |
| 15:20:41 | sean-k-mooney | do we really need to expsoe thsoe in our config? | |