Skip to content

[QUIC] Investigate failing test#128645

Closed
ManickaP wants to merge 2 commits into
dotnet:mainfrom
ManickaP:quic-test-failure
Closed

[QUIC] Investigate failing test#128645
ManickaP wants to merge 2 commits into
dotnet:mainfrom
ManickaP:quic-test-failure

Conversation

@ManickaP
Copy link
Copy Markdown
Member

Related to #121567

Copilot AI review requested due to automatic review settings May 27, 2026 13:52
@ManickaP ManickaP added NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it labels May 27, 2026
@ManickaP
Copy link
Copy Markdown
Member Author

/azp run runtime-extra-platforms

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines failed to run 1 pipeline(s).

@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.

@ManickaP
Copy link
Copy Markdown
Member Author

/azp list

@azure-pipelines
Copy link
Copy Markdown

CI/CD Pipelines for this repository:

@ManickaP
Copy link
Copy Markdown
Member Author

/azp run runtime-coreclr libraries-jitstress

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

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 instruments System.Net.Quic functional testing to investigate the intermittent MultipleReadsAndWrites failure linked to issue #121567.

Changes:

  • Adds networking event-source logging to MultipleReadsAndWrites.
  • Increases the test loop from 100 to 10,000 iterations with a much larger per-iteration timeout.
  • Adds timing/disposal changes around the stream write/read flow.

@ManickaP
Copy link
Copy Markdown
Member Author

/azp run runtime-coreclr libraries-jitstress

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@ManickaP
Copy link
Copy Markdown
Member Author

Problem found, there's a race between RECEIVE event and ReadAsync that manifests with unblocking a 0-byte read when it shouldn't. In this test, it leads to the consuming stream wrongfully thinking it already received FIN and thus Dispose aborting the reading side.
I have a stable repro with Thread.Sleep in HandleEventReceive, closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-System.Net.Quic NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants