| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-07-20 | |||
| 16:31:46 | gibi | tkajinam: antia: so for TDX we could go with the domain capabilities route? | |
| 16:31:51 | tkajinam | I mean it only checks /sys/module/kvm_amd/parameters/sev and does not check /sys/module/kvm_amd/parameters/sev_es or /sys/module/kvm_amd/parameters/sev_snp | |
| 16:32:19 | tkajinam | gibi, yes, libvirt checks the tdx flag when it declares <tdx support='yes'/> | |
| 16:32:33 | antia | gibi: Yes exactly. Right now it does both the sys fs check and domain capabilities which is not needed | |
| 16:32:51 | gibi | cool then you can get rid of the problem by not relying on reading that file :) | |
| 16:33:15 | tkajinam | yeah using only domain capabilities simplifies the logic and tests largely | |
| 16:33:51 | antia | tried it before and tests were quite annoying to fix, but I think that has been fixed already so should be simple enough | |
| 16:34:20 | gibi | cool | |
| 16:35:03 | tkajinam | one question I got during reviewing the tdx series is whether we need to bump object version again for tdx or can reuse the existing version for snp assuming tdx would be merged this cycle | |
| 16:35:21 | tkajinam | I mean for snp we bumped ImageMeta object version due to hw_mem_encryption_model supporting new amd-sev-snp | |
| 16:35:46 | tkajinam | and I'm wondering if we should bump the version again when hw_mem_encryption_model supports new intel-tdx or we can handle it within the same version | |
| 16:36:45 | gibi | bump the version please | |
| 16:36:50 | gibi | object versions are cheap | |
| 16:36:56 | tkajinam | ok | |
| 16:37:01 | antia | either is fine with me | |
| 16:37:05 | antia | will bump it | |
| 16:37:07 | gibi | thanks | |
| 16:37:30 | tkajinam | quick conclusion :-) | |
| 16:37:44 | gibi | :) | |
| 16:37:50 | tkajinam | nothing else from my end. I reviewed tdx sceries and left a few comments there. | |
| 16:38:03 | antia | thanks for the review! | |
| 16:38:04 | tkajinam | antia, in case you need help to follow existing sev thing then feel free to ping me | |
| 16:38:16 | antia | will do :) | |
| 16:38:28 | gibi | I will keep myself in the loop :) | |
| 16:39:59 | gibi | we can move on | |
| 16:40:55 | tkajinam | yup | |
| 16:42:05 | gibi | bauzas: are you still with us? | |
| 16:43:10 | gibi | OK I'm taking over :) | |
| 16:43:15 | bauzas | sorry back | |
| 16:43:22 | bauzas | #topic Bug scrubbing | |
| 16:43:24 | bauzas | #topic Bug scrubbing | |
| 16:43:40 | bauzas | I haven't looked at the numbers tbh | |
| 16:43:45 | bauzas | moving on so ? | |
| 16:43:52 | bauzas | #topic Open discussion | |
| 16:43:58 | bauzas | anything ? | |
| 16:44:18 | gibi | - | |
| 16:44:23 | tkajinam | a quick one | |
| 16:44:55 | tkajinam | while working on sev-snp work I found several potential cleanups in current libvirt driver code (mainly due to remaining checks for old versions which are non longer supported) | |
| 16:45:03 | tkajinam | https://review.opendev.org/q/hashtag:%22libvirt-cleanups%22+(status:open%20OR%20status:merged) | |
| 16:45:19 | tkajinam | I'll capture these in this hash tag so if you have spare time I appreciate your review :-) | |
| 16:45:33 | gibi | good initiative | |
| 16:45:41 | tkajinam | I think we managed to drop number of unnecessary logics as part of snp work :-) | |
| 16:45:59 | gibi | everything that delete codes worth the review time | |
| 16:46:17 | tkajinam | less code, better life | |
| 16:47:15 | tkajinam | there is also removal of vcpu_pin_set which drops large amount of code just fyi (though I find it a bit complicated for review) | |
| 16:47:16 | tkajinam | https://review.opendev.org/c/openstack/nova/+/996016 | |
| 16:47:21 | tkajinam | that's all from me for today ! | |
| 16:47:42 | gibi | :) | |
| 16:48:00 | gmaan | one review request, enforce_scope releasenotes and one cleanup need 2nd core to check those | |
| 16:48:00 | gibi | (maybe we need a couple of sprints just to fix bugs and land cleanups) | |
| 16:48:02 | gmaan | #link https://review.opendev.org/q/hashtag:%22remove-enforce-scope-flag%22+hashtag:%22nova%22status:open | |
| 16:48:27 | gibi | gmaan: I will check it out now | |
| 16:49:01 | gmaan | thanks | |
| 16:49:42 | bauzas | cool, closing now the meeting, thanks all | |
| 16:49:45 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2026/nova.2026-07-20-16.00.log.html | |
| 16:49:45 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2026/nova.2026-07-20-16.00.txt | |
| 16:49:45 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2026/nova.2026-07-20-16.00.html | |
| 16:49:45 | opendevmeet | Meeting ended Mon Jul 20 16:49:45 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:49:45 | bauzas | #endmeeting | |
| 16:49:51 | antia | thanks | |
| 16:49:52 | gibi | bauzas: thanks | |
| 16:50:10 | elodilles | thanks o/ | |
| 16:50:27 | tkajinam | thank you | |
| 16:51:19 | melwitt | has anyone else noticed in devstack nova-compute no longer logs config options setting when it starts up? | |
| 16:52:50 | dansmith | is that an oslo.service change, perhaps related to the eventlet stuff? | |
| 16:53:51 | melwitt | yeah I'm guessing it has something to do with the change to threading, somehow | |
| 16:56:02 | tkajinam | hmm that's wired | |
| 16:56:25 | tkajinam | for threading we use cotyledon and cotyledon has similar debug logs triggered by log_options opt | |
| 16:59:37 | gmaan | only functional change between oslo.service 4.6.0 and 4.7.0(new one in u-c) is this which is just periodic tasks change https://review.opendev.org/c/openstack/oslo.service/+/982418 | |
| 17:09:09 | melwitt | yeah I dunno, picking a tempest-integrated-compute job from one of the enforce_scope patches I see the lack of conf options logging https://zuul.opendev.org/t/openstack/build/f4cc4f17dd414959b5d04cbfddffe410/log/controller/logs/screen-n-cpu.txt | |
| 17:19:14 | tkajinam | it's wired we see the expected debug logs in nova-scheduler on the other hand | |
| 17:19:31 | tkajinam | maybe nova-compute uses a different service launching method (but I don't feel so now) | |
| 17:19:34 | tkajinam | regarding the file access for sev detection it seems although libvirt reads /sys/module/kvm_amd/parameters/sev it does not use that value when it determines sev elements in capabilities .. we have to fix it along with the missing check for sev_es/snp later | |
| 17:20:30 | tkajinam | for now we should be ok as long as we keep the current file accesses for snp | |
| 17:20:35 | tkajinam | s/snp/sev/ | |
| 18:10:54 | opendevreview | Merged openstack/placement stable/2025.1: [CI][stable-only] Pin tempest for Jammy based job https://review.opendev.org/c/openstack/placement/+/992299 | |
| 18:55:50 | opendevreview | Sylvain Desgrais proposed openstack/nova-specs master: Add spec for allowing security group name spaces https://review.opendev.org/c/openstack/nova-specs/+/998049 | |
| 19:07:42 | opendevreview | Merged openstack/nova master: [Trivial] use the new variable name for RBAC new defaults https://review.opendev.org/c/openstack/nova/+/995580 | |
| 19:08:17 | opendevreview | Merged openstack/nova master: Remove disabling the scope enforcement https://review.opendev.org/c/openstack/nova/+/994950 | |
| 19:08:25 | opendevreview | Merged openstack/placement master: Remove disabling the scope enforecment https://review.opendev.org/c/openstack/placement/+/994790 | |
| 23:06:56 | opendevreview | Merged openstack/os-vif master: Declare Python 3.14 support https://review.opendev.org/c/openstack/os-vif/+/998013 | |
| 23:32:49 | opendevreview | Mike Lowe proposed openstack/nova master: libvirt: Fix false mem-encryption/UEFI conflict with cpu_mode=custom https://review.opendev.org/c/openstack/nova/+/997259 | |
| #openstack-nova - 2026-07-21 | |||
| 03:46:35 | opendevreview | yaguang proposed openstack/nova master: libvirt: enable Hyper-V evmcs on Intel VMX hosts https://review.opendev.org/c/openstack/nova/+/995620 | |
| 05:15:06 | opendevreview | Masanori Kuroha proposed openstack/nova master: Skip deleting allocations on rebuild failure https://review.opendev.org/c/openstack/nova/+/998085 | |
| 05:36:21 | opendevreview | Masanori Kuroha proposed openstack/nova master: Skip deleting allocations on rebuild failure https://review.opendev.org/c/openstack/nova/+/998085 | |
| 05:55:46 | opendevreview | yaguang proposed openstack/nova master: libvirt: enable Hyper-V evmcs on Intel VMX hosts https://review.opendev.org/c/openstack/nova/+/995620 | |
| 07:49:12 | gibi | melwitt: gmaan: dansmith: re log dumping. There was many back and forth about that. nova-compute and nova-scheduler differs in the fact that nova-scheduler uses oslo.service to fork worker processes while nova-compute simply runs in the main process. So there is now missing log dump in that codepath in oslo I assume | |
| 07:57:08 | gibi | filed 2161400 | |
| 07:57:11 | gibi | https://bugs.launchpad.net/nova/+bug/2161400 | |
| 08:51:16 | opendevreview | Balazs Gibizer proposed openstack/placement master: Check overall capacity before generating candidates https://review.opendev.org/c/openstack/placement/+/997617 | |
| 12:12:45 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Add Intel TDX host capability detection https://review.opendev.org/c/openstack/nova/+/994421 | |
| 12:36:06 | gibi | sambork: thanks for taking https://bugs.launchpad.net/nova/+bug/2161400 | |
| 12:36:22 | sambork | np! | |
| 12:37:52 | sambork | gibi, looks like an easy fix!You pointed right to the solution in the bug report, so I should have a patch ready in the next few minutes | |
| 12:39:35 | gibi | cool | |
| 12:39:56 | gibi | that was the only obvious oslo difference between the scheduler and the compute | |
| 12:44:19 | opendevreview | Kamil Sambor proposed openstack/nova master: compute: dump config options at startup https://review.opendev.org/c/openstack/nova/+/998136 | |
| 12:47:10 | gibi | left a comment, we need to check the CI results once we have it ^^ | |
| 13:05:17 | sambork | Thanks! I agree that this will be duplicated, but we can wait for the ci results before I push the fix using utils.concurrency_mode_threading | |
| 14:51:45 | opendevreview | Kamil Sambor proposed openstack/nova master: compute: dump config options at startup https://review.opendev.org/c/openstack/nova/+/998136 | |