chore: Remove NPM Windows 2022 Tests from the NPM Conformance Tests #3888
Conversation
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
| /azp run Azure Container Networking PR, NPM Scale Test, NPM Conformance Tests |
There was a problem hiding this comment.
Pull Request Overview
This PR removes Windows 2022 NPM conformance tests from the CI pipeline to address flaky timeout issues and streamline testing. The change eliminates Windows-specific test configurations while maintaining Linux NPM testing, which is the only supported NPM version.
- Removes Windows 2022 test matrix configurations from both main and latest-release conformance test pipelines
- Eliminates Windows-specific cluster creation and test execution logic
- Removes commented-out Windows Cyclonus test stage
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| .pipelines/npm/npm-conformance-tests.yaml | Removes Windows 2022 test matrix, cluster creation logic, and test execution code |
| .pipelines/npm/npm-conformance-tests-latest-release.yaml | Removes Windows 2022 test matrix configuration |
| Azure Pipelines successfully started running 3 pipeline(s). |
| /azp run Azure Container Networking PR, NPM Scale Test, NPM Conformance Tests |
| Azure Pipelines successfully started running 3 pipeline(s). |
rayaisaiah commented Aug 5, 2025
| Scale: https://msazure.visualstudio.com/One/_build/results?buildId=132765236&view=results |
| /azp run Azure Container Networking PR |
| Azure Pipelines successfully started running 1 pipeline(s). |
vakalapa approved these changes Aug 6, 2025
jpayne3506 approved these changes Aug 6, 2025
rayaisaiah deleted the isaiahraya/remove-npm-conformance-test-windows-22 branch rayaisaiah added a commit that referenced this pull request Aug 11, 2025
…3888) * removing windows conformance tests part 1 * removing windows conformance tests part 2 * removed unused segament * typo * keep latest release the same * removed 1 of the az aks get-credentials
4 tasks
github-merge-queue bot pushed a commit that referenced this pull request Aug 11, 2025
…ce Tests + Clean Up NPM release/v1.4 (#3917) * chore: Remove NPM Windows 2022 Tests from the NPM Conformance Tests (#3888) * removing windows conformance tests part 1 * removing windows conformance tests part 2 * removed unused segament * typo * keep latest release the same * removed 1 of the az aks get-credentials * backport windows conformance test removal and image bump as v1.4 is EOL
NihaNallappagari pushed a commit to NihaNallappagari/azure-container-networking that referenced this pull request Sep 4, 2025
…zure#3888) * removing windows conformance tests part 1 * removing windows conformance tests part 2 * removed unused segament * typo * keep latest release the same * removed 1 of the az aks get-credentials
sivakami-projects pushed a commit that referenced this pull request Oct 23, 2025
…3888) * removing windows conformance tests part 1 * removing windows conformance tests part 2 * removed unused segament * typo * keep latest release the same * removed 1 of the az aks get-credentials
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 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.
Reason for Change:
Removes the NPM Windows 2022 tests from the NPM conformance test pipeline runs.
Issue Fixed:
Recently the NPM conformance tests have experienced a
[TIMEDOUT]error for the Windows 2022 tests. This is causing NPM-related PRs to be delayed due to flaky/inconsistent pipeline runs. We do not need to verify NPM Windows scenarios in the conformance test as we aren't supporting Windows or planning to ever cut a new release. This change streamlines the NPM conformance test runs to test NPM Linux which is the only supported NPM version outside of NPM-Lite which runs different tests to validate.Requirements:
Notes: