Skip to content

[AI Generated] BugFix: exclude kexec tests on confidential VMs#4554

Open
vyadavmsft wants to merge 2 commits into
mainfrom
bugfix/kexec-cvm-exclusion_240626_125141
Open

[AI Generated] BugFix: exclude kexec tests on confidential VMs#4554
vyadavmsft wants to merge 2 commits into
mainfrom
bugfix/kexec-cvm-exclusion_240626_125141

Conversation

@vyadavmsft

@vyadavmsft vyadavmsft commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

Description

Related Issue

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Documentation update

Checklist

  • Description is filled in above
  • No credentials, secrets, or internal details are included
  • Peer review requested (if not, add required peer reviewers after raising PR)
  • Tests executed and results posted below

Test Validation

Key Test Cases:

Impacted LISA Features:

Tested Azure Marketplace Images:

Test Results

Image VM Size Result
PASSED / FAILED / SKIPPED

@vyadavmsft vyadavmsft requested a review from LiliDeng as a code owner June 24, 2026 19:57
Copilot AI review requested due to automatic review settings June 24, 2026 19:57
@vyadavmsft vyadavmsft force-pushed the bugfix/kexec-cvm-exclusion_240626_125141 branch from 1bc83db to 6aa9f50 Compare June 24, 2026 20:00
@github-actions

Copy link
Copy Markdown

⚠️ AI Test Selection — cancelled

2 test case(s) selected (view run)

Marketplace image: canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest

Count
✅ Passed 0
❌ Failed 0
⏭️ Skipped 0
Total 0
Test case details
Test Case Status Time (s) Message

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 kexec test suite selection criteria to avoid running kexec tests on confidential VM scenarios by adding a simple_requirement(...) gate at the suite level.

Changes:

  • Added a TestSuiteMetadata.requirement intended to exclude CVM runs for the kexec suite.
  • Imported simple_requirement and a security profile helper to express the requirement in feature terms.

Comment thread lisa/microsoft/testsuites/kexec/kexec_suite.py Outdated
@github-actions

Copy link
Copy Markdown

✅ AI Test Selection — PASSED

2 test case(s) selected (view run)

Marketplace image: canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest

Count
✅ Passed 2
❌ Failed 0
⏭️ Skipped 0
Total 2
Test case details
Test Case Status Time (s) Message
verify_kexec_reboot_systemd (lisa_0_0) ✅ PASSED 67.583
verify_kexec_reboot_direct (lisa_0_1) ✅ PASSED 69.910

Copilot AI review requested due to automatic review settings June 24, 2026 20:11
@vyadavmsft vyadavmsft removed the request for review from Copilot June 24, 2026 20:11
@github-actions

Copy link
Copy Markdown

✅ AI Test Selection — PASSED

2 test case(s) selected (view run)

Marketplace image: canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest

Count
✅ Passed 0
❌ Failed 0
⏭️ Skipped 2
Total 2
Test case details
Test Case Status Time (s) Message
verify_kexec_reboot_systemd (lisa_0_0) ⏭️ SKIPPED 0.000 deployment skipped: Requirement mismatch: excluded feature 'Security_Profile' found in capability
verify_kexec_reboot_direct (lisa_0_1) ⏭️ SKIPPED 0.000 deployment skipped: Requirement mismatch: excluded feature 'Security_Profile' found in capability

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.

3 participants