| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-12-02 | |||
| 14:37:41 | opendevreview | Dhruv Shah proposed openstack/nova master: [trivial][doc] Fix auth_url definitions in install guides https://review.opendev.org/c/openstack/nova/+/969253 | |
| 14:41:22 | opendevreview | Dhruv Shah proposed openstack/nova master: [trivial][doc] Fix auth_url definitions in install guides https://review.opendev.org/c/openstack/nova/+/969253 | |
| 15:03:23 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Use firmeare auto-selection by libvirt https://review.opendev.org/c/openstack/nova/+/969132 | |
| 15:35:09 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add capability to load loader and nvram from xml https://review.opendev.org/c/openstack/nova/+/969086 | |
| 15:35:09 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add basic xml generation for firmware auto selection https://review.opendev.org/c/openstack/nova/+/969085 | |
| 15:35:10 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Use firmeare auto-selection by libvirt https://review.opendev.org/c/openstack/nova/+/969132 | |
| 15:35:10 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add capability to load ssm feature from existing xml https://review.opendev.org/c/openstack/nova/+/969131 | |
| 15:35:11 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Extend functional test coverage of UEFI boot guests https://review.opendev.org/c/openstack/nova/+/969263 | |
| 16:07:12 | melwitt | gibi: hey, I had pushed an update to https://review.opendev.org/c/openstack/nova/+/964848 that you reviewed to add a reno, fyi when you get a chance | |
| 16:09:14 | dansmith | oh yeah I want to hit that too | |
| 16:34:48 | opendevreview | sean mooney proposed openstack/nova-specs master: add spec for resouce notifications https://review.opendev.org/c/openstack/nova-specs/+/967712 | |
| 16:35:13 | opendevreview | Ivan Anfimov proposed openstack/nova master: Remove openSUSE/SLES from install guide https://review.opendev.org/c/openstack/nova/+/949324 | |
| 16:35:17 | opendevreview | Ivan Anfimov proposed openstack/nova master: Remove openSUSE/SLES from install guide https://review.opendev.org/c/openstack/nova/+/949324 | |
| 16:35:42 | opendevreview | Ivan Anfimov proposed openstack/nova master: Remove openSUSE/SLES from install guide https://review.opendev.org/c/openstack/nova/+/949324 | |
| 16:37:48 | opendevreview | Ivan Anfimov proposed openstack/nova master: docs: update installation documentation https://review.opendev.org/c/openstack/nova/+/938680 | |
| 16:37:52 | opendevreview | Ivan Anfimov proposed openstack/nova master: docs: update installation documentation https://review.opendev.org/c/openstack/nova/+/938680 | |
| 18:14:43 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Add backlog spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | |
| 19:01:45 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 19:01:46 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron extensions https://review.opendev.org/c/openstack/nova/+/962270 | |
| 19:01:46 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron subnets https://review.opendev.org/c/openstack/nova/+/962190 | |
| 19:01:47 | opendevreview | Lajos Katona proposed openstack/nova master: WIP: Use SDK for Neutron Ports https://review.opendev.org/c/openstack/nova/+/969298 | |
| 19:01:47 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron floating IPs https://review.opendev.org/c/openstack/nova/+/962604 | |
| 19:05:02 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose Spec 1 of Graceful shutodwn for 2026.1 https://review.opendev.org/c/openstack/nova-specs/+/969299 | |
| 19:05:54 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose Spec 1 of Graceful shutodwn for 2026.1 https://review.opendev.org/c/openstack/nova-specs/+/969299 | |
| 21:35:00 | opendevreview | Jay Faulkner proposed openstack/nova master: WIP: [ironic] Use states constants from Ironic, update test for ddt https://review.opendev.org/c/openstack/nova/+/969321 | |
| 22:19:32 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Add backlog spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | |
| 22:55:47 | gmaan | gibi: dansmith sean-k-mooney ^^ updated spec for periodic tasks, please recheck | |
| 22:58:54 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose Spec 1 of Graceful shutodwn for 2026.1 https://review.opendev.org/c/openstack/nova-specs/+/969299 | |
| 22:59:48 | gmaan | dansmith: to propose the spec1 from backlog for this cycle, I am copy the same content from backlog and proposing for this cycle spec ^^, is that right way or any other better way to propose it per cycle? | |
| 23:02:28 | gmaan | or instead of copy the whole content, we can just include a single line referring to backlog spec and tell what all it will implement? | |
| 23:27:03 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Add backlog spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | |
| #openstack-nova - 2025-12-03 | |||
| 01:45:24 | opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | |
| 01:45:24 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 01:45:25 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 01:45:25 | opendevreview | melanie witt proposed openstack/nova master: TPM: prepare to bump service version for live migration https://review.opendev.org/c/openstack/nova/+/962051 | |
| 01:45:26 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 01:45:26 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 01:45:27 | opendevreview | melanie witt proposed openstack/nova master: TPM: opt-in to new TPM secret security via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| 01:45:27 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 01:45:28 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 02:32:55 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Drop redundant chown of tpm data directory https://review.opendev.org/c/openstack/nova/+/962446 | |
| 02:34:06 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Extend functional test coverage of UEFI boot guests https://review.opendev.org/c/openstack/nova/+/969263 | |
| 02:34:17 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add basic xml generation for firmware auto selection https://review.opendev.org/c/openstack/nova/+/969085 | |
| 02:34:27 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add capability to load loader and nvram from xml https://review.opendev.org/c/openstack/nova/+/969086 | |
| 02:36:01 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add capability to load loader and nvram from xml https://review.opendev.org/c/openstack/nova/+/969086 | |
| 02:36:11 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add capability to load ssm feature from existing xml https://review.opendev.org/c/openstack/nova/+/969131 | |
| 02:36:22 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Use firmeare auto-selection by libvirt https://review.opendev.org/c/openstack/nova/+/969132 | |
| 02:36:58 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Use firmeare auto-selection by libvirt https://review.opendev.org/c/openstack/nova/+/969132 | |
| 03:37:27 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 03:37:27 | opendevreview | melanie witt proposed openstack/nova master: TPM: prepare to bump service version for live migration https://review.opendev.org/c/openstack/nova/+/962051 | |
| 03:37:28 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 03:37:28 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 03:37:29 | opendevreview | melanie witt proposed openstack/nova master: TPM: opt-in to new TPM secret security via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| 03:37:29 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 03:37:30 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 11:35:58 | sean-k-mooney | DominikDanelski[m]: so you shoudl rebase on top of https://review.opendev.org/c/openstack/nova/+/969251 | |
| 11:38:50 | sean-k-mooney | i dont have time to do a detailed review but the content looks reasonable at first glance. the only think i wasould imedially say could be imporved is expelcity defining the flavor to be use dith the server. your currently relying impleictly on the behavior fo self._build_server(networks='none') so without diging inot the defautl flavor that that is usign tis not apprent why | |
| 11:38:52 | sean-k-mooney | it woudl violate quota | |
| 11:52:12 | opendevreview | Will Szumski proposed openstack/nova master: WIP: Try and reproduce nova scheduler race https://review.opendev.org/c/openstack/nova/+/969380 | |
| 12:06:57 | MaxLamprecht[m] | Hey, I have https://review.opendev.org/c/openstack/nova/+/934984(+regression test), which has already +2 by sean-mooney. Anyone available for more review/merge? | |
| 12:08:02 | MaxLamprecht[m] | * have https://review.opendev.org/c/openstack/nova/+/934984 (+regression test), | |
| 12:08:47 | MaxLamprecht[m] | Additionally I have a really small MR that adds missing db indices for better db performance. https://review.opendev.org/c/openstack/nova/+/958556 | |
| 13:24:08 | opendevreview | Dominik proposed openstack/nova master: Regression test for Placement allocations remaining during failed schedule (bug #2132020) https://review.opendev.org/c/openstack/nova/+/969251 | |
| 13:40:12 | opendevreview | sean mooney proposed openstack/nova-specs master: add spec for resouce notifications https://review.opendev.org/c/openstack/nova-specs/+/967712 | |
| 15:25:36 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Add backlog spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | |
| 15:28:09 | gmaan | gibi: sean-k-mooney dansmith ^^ updated it, no change in proposal just fixed the sean-k-mooney comments on improving the paragraphs. | |
| 15:29:41 | gmaan | gibi: i saw you commented on the spec1 proposal for this cycle, I think we should fix those in backlog spec? so that spec1 proposal in cycle and backlog spec can match https://review.opendev.org/c/openstack/nova-specs/+/969299/3 | |
| 15:30:47 | gmaan | my idea was merge the backlog spec after fixing all things we want and then propose spec1 and spec2 per cycle with no change in proposal | |
| 15:42:24 | ratailor | gmaan, could you please review tempest test as suggested in https://review.opendev.org/c/openstack/nova-specs/+/929780/comments/342e3188_90765518 and please provide your suggestion if any improvement is required. | |
| 15:42:38 | ratailor | https://review.opendev.org/c/openstack/tempest/+/953331 | |
| 16:22:57 | gmaan | gibi: sean-k-mooney ignore my last comment about spec1 proposal , I am checking the comemnt and will propose the updated version | |
| 16:23:18 | sean-k-mooney | ack | |
| 16:36:12 | gmaan | ratailor: ack, will check | |
| 16:48:09 | opendevreview | Lajos Katona proposed openstack/nova master: blueprint: iothreads-for-instances https://review.opendev.org/c/openstack/nova/+/939254 | |
| 16:50:40 | opendevreview | Lajos Katona proposed openstack/nova master: blueprint: iothreads-for-instances https://review.opendev.org/c/openstack/nova/+/939254 | |
| 17:12:46 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Collect result of _live_migration_operation https://review.opendev.org/c/openstack/nova/+/969501 | |
| 17:12:50 | dansmith | gibi: ^ | |
| 17:16:15 | opendevreview | Dan Smith proposed openstack/nova master: Fix missing guest.migrate(parallel=) mock arg https://review.opendev.org/c/openstack/nova/+/969503 | |
| 17:16:45 | dansmith | and this ^ should fix the actual failure. We'll need to swap the order of those before merge of course, but in this sequence you can see the problem | |
| 17:26:15 | dansmith | I'll create a bug for this after my current call, btw | |
| 17:43:08 | gibi | dansmith: looking | |
| 17:49:59 | gibi | this is the eventlet change in that codepath https://review.opendev.org/c/openstack/nova/+/948072/17/nova/virt/libvirt/driver.py | |
| 17:50:52 | gibi | I'm wondering how the eventlet version raised the exception if if it is not waited for the spawned greenlet to finish | |
| 17:51:57 | gibi | I'm pretty fried already so I probably not seeing someting obvious here | |
| 17:56:56 | dansmith | I'm still on another call, but maybe a CastAsCall difference? perhaps that's the change here and not something that would be actually different at runtime? but also, it seems bad for us to not capture such an error at runtime | |
| 17:57:04 | dansmith | so perhaps the actual miss has been longer than eventlet removal | |
| 17:57:22 | dansmith | but it's definitely different with the new future stuff, and hides errors we were catching before (as demonstrated) | |
| 17:58:24 | gibi | yeah there is spawn synchronous fixture (like CastAsCall for RPC) so that can be involved | |
| 17:58:33 | dansmith | er, right | |
| 17:58:43 | gibi | hm that gave me a hunch | |
| 18:00:58 | gibi | there was a test fixture for this sync spanw for testing and it was actually broken (or I assumed at that time that it was broken) and I changed the behavior of it (thought fixing it) during the eventlet removal. This can be a fallout of that... | |
| 18:01:04 | gibi | now I just need to find it | |
| 18:02:44 | opendevreview | Merged openstack/nova-specs master: Spec for asynchronous volume-attach API https://review.opendev.org/c/openstack/nova-specs/+/958900 | |
| 18:05:57 | gibi | dansmith: I think this is the old code that made it work https://review.opendev.org/c/openstack/nova/+/948072/17/nova/tests/fixtures/nova.py#b1322 | |
| 18:06:23 | gibi | nope,ignore me | |
| 18:06:40 | gibi | link and add_done_callback is behaving the same | |
| 18:11:44 | gibi | OK I'm lost. I will try tomorrow with fresh minds | |
| 18:12:09 | dansmith | no problem, at least we know the problem now | |
| 18:21:27 | opendevreview | Will Szumski proposed openstack/nova master: WIP: Adds regression test for bug LP#2132984 https://review.opendev.org/c/openstack/nova/+/969380 | |
| 19:03:22 | opendevreview | Merged openstack/nova-specs master: Add backlog spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | |