Skip to content

ci: skip hyper-v vbs tests on amd - #5069

Merged
Trevor Jones (tjones60) merged 1 commit into
microsoft:mainfrom
tjones60:no_amd_vbs
Nov 13, 2026
Merged

Trevor Jones (tjones60) merged 1 commit into
microsoft:mainfrom
tjones60:no_amd_vbs

Conversation

@tjones60

Copy link
Copy Markdown
Contributor

Hyper-V VBS OpenHCL tests always fail on AMD, so skip them for now. We will continue to get coverage on Intel, but leave the unstable tag on tests since it is still true that any failures are not actionable.

@tjones60
Trevor Jones (tjones60) requested a review from a team as a code owner November 12, 2026 23:36
Copilot AI balanced review requested due to automatic review settings November 12, 2026 23:36

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.

🟢 Approval recommended

No active review issues are reported for the targeted AMD SNP CI test-filter change.

No open issues 🎉

What changed in this PR

This PR skips failing Hyper-V VBS tests in the Windows AMD SNP CI job while preserving Intel coverage and existing unstable test markings.

Changes:

  • Excludes tests matching both hyperv and vbs, alongside the existing PCAT exclusion.
  • Documents the AMD failures on recent prerelease builds and adds a reminder to re-enable the tests once fixed.
File Description
flowey/​flowey_hvlite/​src/​pipelines/​checkin_gates.rs Updates the AMD SNP test filter and explains the temporary exclusion.

Review effort: Balanced

nextest_filter_expr: format!("{cvm_filter} & !test(pcat)"),
// - PCAT VMs are unstable on AMD
// - Hyper-V VBS VMs fail on AMD with recent prerelease builds
// TODO: re-enable once fixed.

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.

@tjones60
Trevor Jones (tjones60) enabled auto-merge (squash) November 13, 2026 00:37
@tjones60
Trevor Jones (tjones60) merged commit 7d90c41 into microsoft:main Nov 13, 2026
108 of 112 checks passed
@github-actions

Copy link
Copy Markdown

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