revert(runners): restore Windows runners to pre-#1108 config (m5zn.metal + Server 2022) - #1125
Merged
shashank-boyapally merged 1 commit intoSep 11, 2026
Conversation
…(m5zn.metal + Server 2022) runfinch#1108 migrated Windows runners from bare-metal m5zn.metal / Server 2022 to a virtualized c7i.2xlarge + nested virtualization / Server 2025 (to save cost). That broke WSL2: distro registration fails with Wsl/Service/RegisterDistro/0x8007007a ("data area too small"), so finch's 'finch vm init' can't attach its disk (VM stays Nonexistent) and every Windows e2e run has failed since (last green 2026-08-20, continuous failures from 2026-08-26 onward - same signature throughout). Revert the Windows-only pieces to their pre-runfinch#1108 (d268124^) state: - instanceType c7i.2xlarge -> m5zn.metal (bare metal; native virtualization for WSL2) - AMI Windows Server 2025 -> 2022 - drop the cpuOptions nestedVirtualization block - restore windows-runner-user-data.yaml (MSI-based WSL 2.4.13 + kernel install, Go 1.23.3, "vEthernet (WSL)" firewall rule) and the test's expected instance type This does NOT include the WSL-pin / auto-download / import experiments; it is a straight revert to the last-known-good Windows runner configuration. mac/linux runners are untouched. Tradeoff: reinstates the m5zn.metal cost that runfinch#1108 removed - the cost saving regressed WSL2, so metal is required for working Windows CI. Signed-off-by: ShashankReddy Boyapally <shasboy@amazon.com>
smandhada16
approved these changes
Sep 11, 2026
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
2 participants
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.
#1108 migrated Windows runners from bare-metal m5zn.metal / Server 2022 to a virtualized c7i.2xlarge + nested virtualization / Server 2025 (to save cost). That broke WSL2: distro registration fails with
Wsl/Service/RegisterDistro/0x8007007a ("data area too small"), so finch's 'finch vm init' can't attach its disk (VM stays Nonexistent) and every Windows e2e run has failed since (last green 2026-08-20, continuous failures from 2026-08-26 onward - same signature throughout).
Revert the Windows-only pieces to their pre-#1108 (d268124^) state:
This does NOT include the WSL-pin / auto-download / import experiments; it is a straight revert to the last-known-good Windows runner configuration. mac/linux runners are untouched.
Tradeoff: reinstates the m5zn.metal cost that #1108 removed - the cost saving regressed WSL2, so metal is required for working Windows CI.
Issue #, if available:
Description of changes:
Testing done:
License Acceptance
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.