Skip to content

cloudtest: skip WSLC test suite on Windows Server images#40559

Open
benhillis wants to merge 3 commits into
masterfrom
benhill/skip-wslc-tests-on-server
Open

cloudtest: skip WSLC test suite on Windows Server images#40559
benhillis wants to merge 3 commits into
masterfrom
benhill/skip-wslc-tests-on-server

Conversation

@benhillis
Copy link
Copy Markdown
Member

Summary

Skip the WSLC TAEF group on Windows Server cloudtest images. The WSLC suite consistently times out on Server SKUs because of a VMBus PowerOff hang during VM teardown, which exhausts the cloudtest job timeout and blocks downstream signal.

Change

cloudtest/CMakeLists.txt: introduce CLOUDTEST_SERVER_IMAGES (currently just the WS2022 datacenter image) and only emit the wslc add_test_group when the image is not in that list.

  • Server images: wsl1 + wsl2 jobs run as before; wslc job is skipped.
  • Client images (rs_prerelease, win11-23h2, win10-22h2): unchanged — still run all three groups.
  • Local test.bat runs are unaffected (no filter applied there).

Reverting

When the host issue is fixed, remove the image from CLOUDTEST_SERVER_IMAGES (or delete the list and the guard) to re-enable WSLC coverage on Server.

The WSLC TAEF job consistently times out on Windows Server images
because of a VMBus PowerOff hang during VM teardown on Server SKUs.

Skip the wslc TAEF group on Server images until the host issue is
resolved. The wsl1 and wsl2 groups continue to run on Server. Client
images (rs_prerelease, win11-23h2, win10-22h2) are unchanged and still
run all three groups.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@benhillis benhillis requested a review from a team as a code owner May 15, 2026 18:55
Copilot AI review requested due to automatic review settings May 15, 2026 18:55
OneBlue
OneBlue previously approved these changes May 15, 2026
Copy link
Copy Markdown
Collaborator

@OneBlue OneBlue left a comment

Choose a reason for hiding this comment

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

I think this is OK for now, let's try to prioritize getting the server fix available in our CI so we can revert this ASAP

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 updates the CloudTest CMake configuration to skip generating the WSLC (TAEF @TestCategory='WSLC') test group on Windows Server CloudTest images to avoid consistent timeouts caused by VM teardown hangs on Server SKUs.

Changes:

  • Introduce a CLOUDTEST_SERVER_IMAGES list to identify Windows Server CloudTest images.
  • Guard the wslc add_test_group emission so it is skipped when the current image is in CLOUDTEST_SERVER_IMAGES.

Comment thread cloudtest/CMakeLists.txt Outdated
Copy link
Copy Markdown
Collaborator

@OneBlue OneBlue left a comment

Choose a reason for hiding this comment

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

I think copilot's right on the variable issue, but LGTM once fixed

Use "${image}" instead of bare image in the IN_LIST condition for
clarity and to avoid relying on if() implicit variable lookup.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Companion change to the cloudtest/CMakeLists.txt skip: tag the
fe_release image as `server: true` and gate the WSLC test job on
`image.server` so the pipeline does not try to load a TestMap.xml
that is no longer generated for the WS22+wslc combination.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 15, 2026 19:56
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 2 out of 2 changed files in this pull request and generated no new comments.

@benhillis
Copy link
Copy Markdown
Member Author

Validated against the nightly pipeline: the wslc tests - fe_release job now resolves to skipped cleanly while all other test phases run normally.

@benhillis benhillis enabled auto-merge (squash) May 15, 2026 23:23
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