| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-01-22 | |||
| 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: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 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: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: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: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: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: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: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: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 | Hi there! | |
| 08:52:47 | zigo | Has any of you already encounter this ? | |
| 08:52:47 | zigo | https://bugs.launchpad.net/nova/+bug/2138923 | |
| 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 nova.cmd, nova.conf, nova.console https://review.opendev.org/c/openstack/nova/+/705657 | |
| 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: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:35 | opendevreview | Stephen Finucane proposed openstack/nova master: typing: Correct import issues https://review.opendev.org/c/openstack/nova/+/974221 | |
| 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: Scheduler changes https://review.opendev.org/c/openstack/nova/+/971176 | |
| 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: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 nova.cmd, nova.conf, nova.console https://review.opendev.org/c/openstack/nova/+/705657 | |
| 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: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:13 | opendevreview | Stephen Finucane proposed openstack/nova master: typing: Correct import issues https://review.opendev.org/c/openstack/nova/+/974221 | |
| 15:45:14 | opendevreview | Stephen Finucane proposed openstack/nova master: mypy: Enable incremental checks https://review.opendev.org/c/openstack/nova/+/974222 | |
| 15:45:14 | opendevreview | Stephen Finucane proposed openstack/nova master: Add ruff-check https://review.opendev.org/c/openstack/nova/+/974441 | |
| 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: DNM:Test with oslo.vmware + compute eventlet removal patches https://review.opendev.org/c/openstack/nova/+/973468 | |
| 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 | |
| 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 | |
| 19:32:04 | rm_work[m] | anyone offhand have any ideas why placement might take a long time to show a node within openstack allocation candidate list after it goes available in ironic? it's usually about 5 minutes in normal operation and sometimes we see it take up to 20 minutes :( | |
| 19:33:57 | rm_work[m] | is there some setting we can tweak? is this a periodic task that is running too slowly? | |
| 19:34:26 | sean-k-mooney | melwitt: +1 for now ill try and take a look properly on monday but it looks ok at first glance | |
| 19:34:52 | melwitt | thanks sean-k-mooney :) | |
| 19:35:05 | sean-k-mooney | rm_work[m]: is the RP not there or is it there but disabled? | |
| 19:35:54 | sean-k-mooney | rm_work[m]: my guess of the top of my head is that its related to the update_aviable_resouces periodic task tiem of 5 mins and the delay is just in the virt driver runnign that and noticing the new node | |
| 19:38:07 | rm_work[m] | ah ok so we could maybe tune that task down to 1m or something? | |
| 19:38:08 | gmaan | melwitt: yeah, I was reviewing that also, lgtm. thanks for refactoring. | |
| 19:38:23 | melwitt | thanks gmaan | |
| 19:39:26 | sean-k-mooney | rm_work[m]: perhaps this is the optioni mentioned https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.update_resources_interval | |
| 19:39:35 | rm_work[m] | i think the BAD case (the 20m I talked about) might be when we had other issues going on slowing things down, so maybe it's as simple as setting the update_aviable_resouces periodic to run faster | |
| 19:39:44 | sean-k-mooney | rm_work[m]: what exactly do you mean by aviable by the way | |
| 19:39:52 | sean-k-mooney | do you meain aviable in the ironic node list | |
| 19:40:02 | rm_work[m] | the ironic state yeah | |
| 19:40:52 | sean-k-mooney | ack i dont recall if there is a specifric addtioanl dely in the ironic case but maybe JayF woudl know | |
| 19:40:59 | rm_work[m] | he said he didn't :D | |
| 19:41:01 | JayF | I sent rm_work[m] to you :) | |