Skip to content

[QUIC] Disable Quic tests on AzL3 VM#125665

Merged
ManickaP merged 1 commit intodotnet:mainfrom
ManickaP:quic-azl3-disable
Mar 18, 2026
Merged

[QUIC] Disable Quic tests on AzL3 VM#125665
ManickaP merged 1 commit intodotnet:mainfrom
ManickaP:quic-azl3-disable

Conversation

@ManickaP
Copy link
Copy Markdown
Member

Contributes to #123216

@dotnet-policy-service
Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @karelz, @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

@steveisok steveisok self-requested a review March 17, 2026 17:32
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

Disables System.Net.Quic functional tests on Azure Linux 3 (non-container) environments to avoid known MsQuic failures (QUIC_STATUS_OUT_OF_MEMORY) tracked in #123216.

Changes:

  • Added a new QuicTestBase condition to skip tests on Azure Linux (unless running in a container).
  • Applied the new condition to multiple QUIC functional test classes via ConditionalClass.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/libraries/System.Net.Quic/tests/FunctionalTests/QuicTestBase.cs Adds IsNotAzureLinux3VM condition used to gate QUIC functional tests.
src/libraries/System.Net.Quic/tests/FunctionalTests/QuicStreamTests.cs Adds the new Azure Linux gating condition to the class.
src/libraries/System.Net.Quic/tests/FunctionalTests/QuicStreamConnectedStreamConformanceTests.cs Adds the new Azure Linux gating condition to the class.
src/libraries/System.Net.Quic/tests/FunctionalTests/QuicListenerTests.cs Adds the new Azure Linux gating condition to the class.
src/libraries/System.Net.Quic/tests/FunctionalTests/QuicConnectionTests.cs Adds the new Azure Linux gating condition to the class.
src/libraries/System.Net.Quic/tests/FunctionalTests/MsQuicTests.cs Adds the new Azure Linux gating condition to the class.
src/libraries/System.Net.Quic/tests/FunctionalTests/MsQuicRemoteExecutorTests.cs Adds the new Azure Linux gating condition to the class.
src/libraries/System.Net.Quic/tests/FunctionalTests/MsQuicCipherSuitesPolicyTests.cs Adds the new Azure Linux gating condition to the class.

@wfurt
Copy link
Copy Markdown
Member

wfurt commented Mar 20, 2026

/backport to release/10.0

@github-actions
Copy link
Copy Markdown
Contributor

Started backporting to release/10.0 (link to workflow run)

akoeplinger pushed a commit that referenced this pull request Mar 24, 2026
Backport of #125665 to release/10.0

## Customer Impact

- [ ] Customer reported
- [ ] Found internally

[Select one or both of the boxes. Describe how this issue impacts
customers, citing the expected and actual behaviors and scope of the
issue. If customer-reported, provide the issue number.]

## Regression

- [ ] Yes
- [ ] No

[If yes, specify when the regression was introduced. Provide the PR or
commit if known.]

## Testing

[How was the fix verified? How was the issue missed previously? What
tests were added?]

## Risk

[High/Medium/Low. Justify the indication by mentioning how risks were
measured and addressed.]

**IMPORTANT**: If this backport is for a servicing release, please
verify that:

- For .NET 8 and .NET 9: The PR target branch is `release/X.0-staging`,
not `release/X.0`.
- For .NET 10+: The PR target branch is `release/X.0` (no `-staging`
suffix).

## Package authoring no longer needed in .NET 9

**IMPORTANT**: Starting with .NET 9, you no longer need to edit a NuGet
package's csproj to enable building and bump the version.
Keep in mind that we still need package authoring in .NET 8 and older
versions.

---------

Co-authored-by: ManickaP <mapichov@microsoft.com>
Co-authored-by: Marie Píchová <11718369+ManickaP@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants