test(e2e): run the E2E suite on the self-hosted WSL2 runner - #141
Draft
rominf wants to merge 3 commits into
Draft
Conversation
rominf
marked this pull request as draft
July 23, 2026 14:25
rominf
force-pushed
the
ci-shared-windows-strix-runners
branch
from
July 24, 2026 12:23
bb89ce9 to
cf10418
Compare
rominf
force-pushed
the
worktree-test-hosted-wsl-e2e
branch
from
July 28, 2026 08:32
b1083d5 to
3462382
Compare
Signed-off-by: Roman Inflianskas <Roman.Inflianskas@amd.com>
Signed-off-by: Roman Inflianskas <Roman.Inflianskas@amd.com>
Target the Ubuntu-under-WSL2 runner on the Strix Halo box (the `wsl` label; the native Strix Linux runner carries `native`) instead of registering a throwaway distro on a hosted windows-latest VM. The lane now mirrors the other self-hosted Linux lane step for step: stray-serve reclaim, bounded GPU preflight, toolchain bootstrap, shared runtime pre-warm, and the full suite with no hand filtering. GPU preflight is advisory here because GPU access under WSL is what this lane is proving out: when it is unavailable the capability probe resolves those scenarios to skip and the rest of the suite still runs. Dispatch selector is `strix-wsl`, matching the sibling hardware lanes. Signed-off-by: Roman Inflianskas <Roman.Inflianskas@amd.com>
rominf
force-pushed
the
worktree-test-hosted-wsl-e2e
branch
from
July 29, 2026 13:51
3462382 to
8bff5d1
Compare
rominf
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
runs-on: [self-hosted, linux, strix-halo, wsl])@requires-wsl/@requires-no-wslcapability tags, a WSL system-inspection scenario, and a distinct WSL platform identity in the report gridWhy: WSL-specific behavior had parser and self-test coverage but nothing running inside a real WSL2 host. Nothing was filtered by hand — the capability probe resolves each scenario against the live host, so GPU scenarios report as not applicable if the distro has no GPU access rather than failing.
GPU preflight is advisory on this lane specifically: GPU access under WSL is what the lane is proving out, so a missing
rocm-smiwarns and continues. A GPU that is present but still held by a leftover serve still fails, since that would corrupt the serve scenarios.Risk: medium. The lane is
continue-on-error: trueand the existing required checks are unchanged. Dispatch selector isstrix-wsl, matching the sibling hardware lanes.Test plan
cargo test -p e2e-cucumber --libcargo xtask e2e -- --name 'System inspection recognizes a WSL host'— passes on a real WSL2 host (probe reportedplatform=wsl os=linux gpu=false)cargo fmt --all -- --check, clippy, license/YAML/signature/DCO hooksThe full local workspace test hook still hits the pre-existing WSL2 process-tree failures in
proc_lifecycle::tests::tree_stop_waits_for_descendantsandtree_forced_kill_reaches_sigterm_ignoring_descendant; both fail identically on a cleanmaincheckout.