Skip to content

vmm_tests: servicing tests can run on nested amd#2742

Merged
benhillis merged 3 commits intomicrosoft:mainfrom
mattkur:turn-on-servicing-tests-nested
Apr 15, 2026
Merged

vmm_tests: servicing tests can run on nested amd#2742
benhillis merged 3 commits intomicrosoft:mainfrom
mattkur:turn-on-servicing-tests-nested

Conversation

@mattkur
Copy link
Copy Markdown
Contributor

@mattkur mattkur commented Feb 4, 2026

The bugs that blocked our servicing tests on AMD in a VM are now resolved and deployed. Remove the block.

@mattkur mattkur requested a review from a team as a code owner February 4, 2026 18:24
Copilot AI review requested due to automatic review settings February 4, 2026 18:24
@mattkur
Copy link
Copy Markdown
Contributor Author

mattkur commented Feb 4, 2026

Note: we may need to update our CI runners to get the fix. Let's see what happens in CI...

But, this works on a default Azure Dev Box VM running on AMD processors.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 removes a test requirement restriction that prevented OpenHCL servicing tests from running on AMD processors in nested virtualization environments. The blocking bugs that necessitated this restriction have been resolved and deployed.

Changes:

  • Removed the special case code in build_requirements() that excluded AMD + Nested execution environments for tests with "servicing" in their name

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 4, 2026

@mattkur
Copy link
Copy Markdown
Contributor Author

mattkur commented Feb 4, 2026

At least one Petri test failed.

Yep, we need to update the CI runners first.

jennagoddard
jennagoddard previously approved these changes Feb 6, 2026
@mattkur mattkur closed this Mar 5, 2026
@mattkur mattkur reopened this Apr 14, 2026
@mattkur mattkur added the release-ci-required Add to a PR to trigger PR gates in release mode label Apr 15, 2026
Copilot AI review requested due to automatic review settings April 15, 2026 02:48
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@benhillis benhillis merged commit b91b1d2 into microsoft:main Apr 15, 2026
65 checks passed
gurasinghMS pushed a commit to gurasinghMS/openvmm that referenced this pull request Apr 17, 2026
The bugs that blocked our servicing tests on AMD in a VM are now
resolved and deployed. Remove the block.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-ci-required Add to a PR to trigger PR gates in release mode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants