| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-01-22 | |||
| 15:21:55 | LarsErikP | james is the only one of them, that's actually present on #ubuntu-server right now :P | |
| 15:24:41 | haleyb | right, and corey is at mozilla | |
| 15:25:06 | haleyb | like i said, for epoxy there won't be much desire | |
| 16:36:32 | melwitt | jkulik: thanks for the reminder, I will try to look soon | |
| 18:33:42 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 20:07:09 | Zhan[m] | while checking the live migration related stuff, I noticed that there are some features/params that are available on libvirt but not yet available in nova (e.g., additional auto converge params, compressions, etc.). I see recently that the parallel connection feature was added in https://review.opendev.org/c/openstack/nova/+/955784, so I'm thinking about just adding all features that are available on libvirt to nova. do we have plans | |
| 20:07:10 | Zhan[m] | regarding this, or if not then maybe I can pick this up? thanks :D | |
| 20:28:04 | dansmith | just because it's in libvirt doesn't necessarily mean it should be in nova, and also, we won't generally add things that are in versions of libvirt that are not yet in an enterprise distro | |
| 20:28:31 | dansmith | best thing would be to identify important missing and relevant things and write a small spec explaining how they would be configured and what benefit they bring | |
| 20:45:30 | Zhan[m] | make sense. in terms of the versions, I'm assuming I should check https://docs.openstack.org/nova/latest/reference/libvirt-distro-support-matrix.html? | |
| 21:11:09 | dansmith | sure, but those are old, so best to look in the code to see what current master is currently targeting (which may be currently still set to the previous release) | |
| 21:25:23 | opendevreview | Merged openstack/nova master: Prevent leaking RPC poller thread between tests https://review.opendev.org/c/openstack/nova/+/974299 | |
| 21:26:23 | gmaan | melwitt: commented on vtpm 'deployment' security change ( 942021), am i missing something on testing the no-vtpm to/from vtpm resize ? they should be supported right? | |
| 21:29:30 | melwitt | gmaan: cool thanks, I will look. basically all resizes are supported _except_ resizes that would need a change in secret ownership. 'deployment' secret owner is the nova service user, 'host' and 'user' secret owner is the normal user | |
| 21:31:21 | melwitt | gmaan: there is a patch at the end of the series which enables resize from TPM 'deployment' => TPM 'host' or 'user' and TPM 'host' or 'user' => TPM 'deployment'. it's just initially not supported bc it is a lot of code and I suspect will be the most difficult part to review | |
| 21:35:32 | gmaan | melwitt: I think that is ok but the test I pointed in review is asserting that the no-vtpm <-> 'deployement' is unsupported unless i missed to understand the tetst | |
| 21:39:10 | melwitt | gmaan: ok I think the confusion is the difference between "instance has no TPM" and "instance has a TPM but secret_security=None bc the user didn't specify one" | |
| 21:39:52 | melwitt | gmaan: if the instance has a TPM and the user did not specify a secret security type (i.e. secret_security=None) then they get the default policy of 'user' | |
| 21:41:15 | melwitt | so TPM 'user' <=> TPM 'deployment' is not allowed for now. no TPM <=> TPM 'deployment' is allowed | |
| 21:43:41 | Zhan[m] | dansmith: spot checking some features, the newest one I found (zlib/zstd compression) is in the NEXT_MIN_{LIBVIRT/QEMU}_VERSION of 2025.2, and it is also in Ubuntu Noble and Debian Trixie so I think it's cool. will run some tests to see how effective they are though. | |
| 21:43:46 | dansmith | melwitt: ohh.. confusing | |
| 21:45:34 | gmaan | melwitt: ohk, i missed to see 'hw:tpm_version' in extra spec of initial server creation. | |
| 21:45:38 | melwitt | yeah. not sure if test names can be improved to something less confusing. at a minimum I could rewrite the docstrings to say more clearly if a test is about TPM <=> TPM vs no TPM <=> TPM | |
| 21:47:52 | gmaan | yeah, that will be helpful, I think renaming test to test_resize_vtpm_server_secret_security_deployment_* can help (*_server* -> _vtpm_server*) | |
| 21:49:02 | melwitt | sure, I can do those. thanks for suggesting | |
| 21:49:25 | gmaan | thanks | |
| #openstack-nova - 2026-01-23 | |||
| 00:18:05 | opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | |
| 00:18:05 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 00:18:06 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 00:18:06 | opendevreview | melanie witt proposed openstack/nova master: TPM: prepare to bump service version for live migration https://review.opendev.org/c/openstack/nova/+/962051 | |
| 00:18:07 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 00:18:07 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 00:18:08 | 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 | |
| 00:18:08 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 00:18:09 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 05:00:30 | opendevreview | Merged openstack/nova master: Replace obsolete PCRE packages https://review.opendev.org/c/openstack/nova/+/970063 | |
| 05:00:43 | opendevreview | Merged openstack/nova master: Faults from cell DB missing in GET /servers/detail https://review.opendev.org/c/openstack/nova/+/699176 | |
| 05:00:57 | opendevreview | Merged openstack/nova master: Use *_OR_ADMIN policy defaults for server shares https://review.opendev.org/c/openstack/nova/+/967709 | |
| 05:01:09 | opendevreview | Merged openstack/nova stable/2025.1: libvirt: Skip unsupported firmware types https://review.opendev.org/c/openstack/nova/+/973821 | |
| 05:01:43 | opendevreview | Merged openstack/nova stable/2025.2: Replace removed os-vif-ovs job https://review.opendev.org/c/openstack/nova/+/972663 | |
| 05:06:46 | opendevreview | Merged openstack/nova stable/2025.1: Replace removed os-vif-ovs job https://review.opendev.org/c/openstack/nova/+/972664 | |
| 05:07:30 | opendevreview | Merged openstack/nova stable/2025.2: Add a regression test for ImagePropsWeigher https://review.opendev.org/c/openstack/nova/+/964287 | |
| 06:27:11 | opendevreview | Esra Ozkan proposed openstack/nova master: Fix Concurrent VM Live Migrate - Volume Backup Error https://review.opendev.org/c/openstack/nova/+/973750 | |
| 06:34:26 | opendevreview | Merged openstack/nova stable/2025.2: Fix fill_metadata usage for the ImagePropertiesWeigher https://review.opendev.org/c/openstack/nova/+/964288 | |
| 06:36:30 | opendevreview | Merged openstack/nova stable/2025.2: Fix string format specifier https://review.opendev.org/c/openstack/nova/+/961336 | |
| 08:52:47 | zigo | https://bugs.launchpad.net/nova/+bug/2138923 | |
| 08:52:47 | zigo | Has any of you already encounter this ? | |
| 08:52:47 | zigo | Hi there! | |
| 08:59:27 | gibi | zigo: does not ring a bell to me | |
| 09:14:52 | obre | Hi! Are there any chance to get this fix in the G-release? https://review.opendev.org/c/openstack/nova/+/916409 | |
| 09:15:00 | obre | It seems to be 90% there | |
| 09:15:01 | obre | ? | |
| 09:16:26 | gibi | obre: I see dansmith has some unanswered comment in the patch. I think the next step there is to handle that | |
| 10:37:46 | sean-k-mooney | zigo: i have only seen that if there is a qemu crash | |
| 10:38:40 | sean-k-mooney | zigo: i.e. the isntance was not found because the domain terminated following the crash | |
| 10:38:53 | sean-k-mooney | it might happen other ways btu that the only time i have seen that | |
| 10:53:49 | opendevreview | Takashi Kajinami proposed openstack/nova stable/2024.2: libvirt: Skip unsupported firmware types https://review.opendev.org/c/openstack/nova/+/974409 | |
| 11:20:53 | zigo | tkajinam: What's your patch about ? UEFI variables ? | |
| 11:23:01 | sean-k-mooney | zigo: are you askign about https://review.opendev.org/c/openstack/nova/+/974409 | |
| 11:23:08 | zigo | yeah | |
| 11:24:03 | sean-k-mooney | its a temory fix to make sure the firmware that is selected by nova is compatibel with the vm devition if your using the stateless firmware option | |
| 11:24:10 | sean-k-mooney | so yes it for uefi guest | |
| 11:24:27 | sean-k-mooney | we are planning to delegate this logic back to libvirt now that it supprots it | |
| 11:24:44 | sean-k-mooney | i.e. allowign libvirt to selct the correct firmware based on the funcitonaltiy you have requested | |
| 11:26:17 | sean-k-mooney | for example if you request secure boot we have to select the secure boot enabeld ovmf uefi firmware or it wont work i belvie the opiste it ture as well if you have it disabled then we need to use the non secure boot version | |
| 11:27:20 | sean-k-mooney | when you enabel stateless vms for confidental compute you similarly need to select the correct firmware file. this is fixing that in nova before we adtop the libvirt feature so it can potitrally be backported | |
| 11:28:06 | sean-k-mooney | well its already being backported but you get the point we exended nova logic before riping it out beacuse we are not backporting the use of the new libvirt feature | |
| 11:29:22 | zigo | Ok, thanks for explaining. | |
| 11:56:31 | opendevreview | Merged openstack/nova stable/2024.2: Fix bug 2000069 https://review.opendev.org/c/openstack/nova/+/970634 | |
| 12:01:57 | DominikDanelski[m] | sean-k-mooney: Sorry, could you go through my replies to your comments in https://review.opendev.org/c/openstack/nova/+/969251?tab=comments ? I think it's very close to being merged after I addressed your questions there, I hope. | |
| 13:46:45 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove networks key from quota schemas https://review.opendev.org/c/openstack/nova/+/974417 | |
| 13:48:29 | stephenfin | sean-k-mooney: gmaan: Could you re-add your +2s to https://review.opendev.org/c/openstack/nova/+/956140/? | |
| 13:48:56 | stephenfin | sean-k-mooney: sorry https://review.opendev.org/c/openstack/nova/+/956141/ for you (need gmaan's +2 on the other one) | |
| 13:49:19 | sean-k-mooney | sure once im off this call ill take a look | |
| 13:49:32 | stephenfin | ta | |
| 13:52:01 | opendevreview | Merged openstack/nova stable/2024.2: Replace removed os-vif-ovs job https://review.opendev.org/c/openstack/nova/+/972665 | |
| 14:04:51 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron subnets https://review.opendev.org/c/openstack/nova/+/962190 | |
| 14:30:34 | opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to remaining top-level modules https://review.opendev.org/c/openstack/nova/+/705658 | |
| 14:30:34 | opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.cmd, nova.conf, nova.console https://review.opendev.org/c/openstack/nova/+/705657 | |
| 14:30:35 | opendevreview | Stephen Finucane proposed openstack/nova master: typing: Correct import issues https://review.opendev.org/c/openstack/nova/+/974221 | |
| 14:30:35 | opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.virt, nova.virt.libvirt https://review.opendev.org/c/openstack/nova/+/974220 | |
| 14:30:36 | opendevreview | Stephen Finucane proposed openstack/nova master: mypy: Enable incremental checks https://review.opendev.org/c/openstack/nova/+/974222 | |
| 15:21:10 | opendevreview | Dominik proposed openstack/nova master: NUMA Topology with Resource Providers: Libvirt NUMA Migrate https://review.opendev.org/c/openstack/nova/+/971177 | |
| 15:21:10 | opendevreview | Dominik proposed openstack/nova master: NUMA Topology with Resource Providers: Scheduler changes https://review.opendev.org/c/openstack/nova/+/971176 | |
| 15:36:04 | sean-k-mooney | DominikDanelski[m]: so melwitt and i chated about this breifly a few weeks ago and we were not configed the repoduce actully repoduce things properly. or that the proposed fix didnt create ohter problems | |
| 15:37:10 | sean-k-mooney | but to be honestl i have lost all context on it since then | |
| 15:45:12 | opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to remaining top-level modules https://review.opendev.org/c/openstack/nova/+/705658 | |
| 15:45:12 | opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.cmd, nova.conf, nova.console https://review.opendev.org/c/openstack/nova/+/705657 | |
| 15:45:13 | opendevreview | Stephen Finucane proposed openstack/nova master: typing: Correct import issues https://review.opendev.org/c/openstack/nova/+/974221 | |
| 15:45:13 | opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.virt, nova.virt.libvirt https://review.opendev.org/c/openstack/nova/+/974220 | |
| 15:45:14 | opendevreview | Stephen Finucane proposed openstack/nova master: Add ruff-check https://review.opendev.org/c/openstack/nova/+/974441 | |
| 15:45:14 | opendevreview | Stephen Finucane proposed openstack/nova master: mypy: Enable incremental checks https://review.opendev.org/c/openstack/nova/+/974222 | |
| 15:45:15 | opendevreview | Stephen Finucane proposed openstack/nova master: Disable flake8 rules https://review.opendev.org/c/openstack/nova/+/974442 | |
| 15:55:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 15:55:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use an executor to delay STOPPED events https://review.opendev.org/c/openstack/nova/+/974445 | |
| 15:55:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Test with oslo.vmware + compute eventlet removal patches https://review.opendev.org/c/openstack/nova/+/973468 | |
| 17:20:41 | gmaan | stephenfin: thanks for followup for 'network' field. I +w on quota ones now and re-approvd other one which were rebased | |
| 17:20:53 | stephenfin | ty! | |
| 18:52:50 | opendevreview | sean mooney proposed openstack/nova master: [WIP] Add PCI device groups for atomic allocation https://review.opendev.org/c/openstack/nova/+/973604 | |
| 19:26:57 | melwitt | dansmith, gmaan: thank you both for reviews :) but I regret to inform you there is a prereq refactor patch underneath it for the service_auth.py commonizing https://review.opendev.org/c/openstack/nova/+/974267 | |