Skip to content

[AI Generated] BugFix: Skip verify_initrd_modules on Confidential VMs - #4582

Merged
LiliDeng merged 1 commit into
mainfrom
bugfix/skip-initrd-on-cvm_060726_101459
Jul 7, 2026
Merged

[AI Generated] BugFix: Skip verify_initrd_modules on Confidential VMs#4582
LiliDeng merged 1 commit into
mainfrom
bugfix/skip-initrd-on-cvm_060726_101459

Conversation

@johnsongeorge-w

Copy link
Copy Markdown
Collaborator

Summary

Add CvmDisabled() to the verify_initrd_modules requirement so the test is skipped at the platform-matching level on Confidential VMs. CVM kernels using Unified Kernel Images (UKI) have no standalone initrd under /boot, causing a misleading assertion failure.

Validation Results

Image Result
Canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest PASSED
Canonical 0001-com-ubuntu-confidential-vm-jammy 22_04-lts-cvm latest SKIPPED (requirement mismatch — expected)

CVMs using UKI/azure-fde kernels do not have a standalone initrd file
under /boot, causing verify_initrd_modules to skip with the unhelpful
message 'Could not find initrd file path'.

Add CvmDisabled() to the test requirement so it is skipped at the
requirement matching level with a standard LISA skip message, consistent
with other tests (kvp, drm, azure_disk_encryption).
Copilot AI review requested due to automatic review settings July 6, 2026 19:57
@johnsongeorge-w
johnsongeorge-w requested a review from LiliDeng as a code owner July 6, 2026 19:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the HvModule core test suite to skip verify_initrd_modules during requirement matching on Confidential VMs, avoiding misleading failures on CVM kernels that boot with Unified Kernel Images (no standalone initrd under /boot).

Changes:

  • Adds a SecurityProfile requirement (CvmDisabled()) to verify_initrd_modules so it won’t be scheduled on CVM images.
  • Removes the CVM-specific module allowlist/skip logic inside verify_initrd_modules, since the test is now excluded for CVMs up front.

Comment thread lisa/microsoft/testsuites/core/hv_module.py
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

✅ AI Test Selection — PASSED

4 test case(s) selected (view run)

Marketplace image: redhat rhel 9_5 latest

Count
✅ Passed 3
❌ Failed 0
⏭️ Skipped 1
Total 4
Test case details
Test Case Status Time (s) Message
verify_lis_modules_version (lisa_0_0) ⏭️ SKIPPED 9.947 skipped: This test case requires LIS to be installed
verify_hyperv_modules (lisa_0_2) ✅ PASSED 15.990
verify_initrd_modules (lisa_0_1) ✅ PASSED 22.812
verify_reload_hyperv_modules (lisa_0_3) ✅ PASSED 269.722

@LiliDeng
LiliDeng merged commit 5e80119 into main Jul 7, 2026
62 checks passed
@LiliDeng
LiliDeng deleted the bugfix/skip-initrd-on-cvm_060726_101459 branch July 7, 2026 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants