| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-10-01 | |||
| 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 | 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:41:46 | adamcarthur5 | assumptions when I benchmark. | |
| 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: Func reproducer for bug/2168874 https://review.opendev.org/c/openstack/nova/+/1007917 | |
| 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: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 hot attach/detach support for virtiofs shares https://review.opendev.org/c/openstack/nova/+/1008154 | |
| 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 | |
| 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_ | Hi all, I'd like to add a topic to the nova-cinder cross-project session at the PTG. | |
| 08:58:55 | yunhokim_ | Topic: booting servers from Cinder-backup-based images | |
| 08:58:55 | yunhokim_ | Blueprint: https://blueprints.launchpad.net/nova/+spec/nova-boot-from-backup-based-image | |
| 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_ | There's no nova-spec yet, but I can write one before the PTG if that would help the discussion. | |
| 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? | |
| 15:21:11 | sean-k-mooney | i wonder if we shoudl propose taging that option as advanced | |
| 15:21:44 | gibi | yeah at least it is no-op as we don't have such tasks | |
| 15:22:37 | sean-k-mooney | there is also https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.periodic_enable nova might work with them disabeld but we certenly dont test that | |
| 15:23:08 | sean-k-mooney | just somethign i have alwasy found odd in our config ref. | |
| 15:23:54 | gibi | yeah would be nice to hide them somehow | |
| 15:24:05 | sean-k-mooney | if we made them advanced https://docs.openstack.org/oslo.config/ocata/cfg.html#advanced-option that woudl at least make it more clear that you proably shoudl not mess it it | |
| 15:24:54 | sean-k-mooney | advanced does nto actully hide them but it puts them at the end of the section with a header waring agaisnt using them | |