Skip to content

fix(iot): disable virtual TPM on VMs that don't need it - #12354

Merged
pcdubs merged 1 commit into
virt-s1:mainfrom
djach7:fix/disable-tpm-iot-tests
Aug 14, 2026
Merged

fix(iot): disable virtual TPM on VMs that don't need it#12354
pcdubs merged 1 commit into
virt-s1:mainfrom
djach7:fix/disable-tpm-iot-tests

Conversation

@djach7

@djach7 djach7 commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

IoT tests don't exercise TPM or measured-boot functionality, but virt-install auto-adds a vTPM for UEFI boots on modern os-variants. A Fedora 44 SELinux userspace regression (selinux-policy 45.8-1) breaks swtpm-selinux's CIL policy compilation (RHBZ#2511086), leaving /usr/bin/swtpm mislabeled and unable to exec under libvirt. This fails every UEFI VM boot in CI.

Since the vTPM was never needed, explicitly disable it instead of waiting for the swtpm fix to reach stable repos.

iot-simplified-installer.sh is left as-is because it tests encrypted root (coreos.inst.crypt_root=1) which requires TPM emulation.

See also: fedora-iot/greenboot-rs#193

IoT tests don't exercise TPM or measured-boot functionality, but
virt-install auto-adds a vTPM for UEFI boots on modern os-variants.
A Fedora 44 SELinux userspace regression (selinux-policy 45.8-1)
breaks swtpm-selinux's CIL policy compilation (RHBZ#2511086),
leaving /usr/bin/swtpm mislabeled and unable to exec under libvirt.
This fails every UEFI VM boot in CI.

Since the vTPM was never needed, explicitly disable it instead of
waiting for the swtpm fix to reach stable repos.

iot-simplified-installer.sh is left as-is because it tests encrypted
root (coreos.inst.crypt_root=1) which requires TPM emulation.

See also: fedora-iot/greenboot-rs#193

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: djach7 <djachimo@redhat.com>

@pcdubs pcdubs left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@knecasov knecasov left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you very much for changes!

@djach7

djach7 commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator Author

/test-f44-iot

@djach7

djach7 commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator Author

/test-f45-iot

@djach7

djach7 commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator Author

f45 iot test failures are unrelated to this pr

@sarmahaj sarmahaj left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pcdubs
pcdubs merged commit 7e05e27 into virt-s1:main Aug 14, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants