Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[QUIC] SendMoreThanStreamLimitRequestsConcurrently_LastWaits on Mocks hangs in CI #53688

Closed
ManickaP opened this issue Jun 3, 2021 · 4 comments
Labels
area-System.Net.Quic disabled-test The test is disabled in source code against the issue
Projects
Milestone

Comments

@ManickaP
Copy link
Member

ManickaP commented Jun 3, 2021

SocketsHttpHandlerTest_Http3_Mock.SendMoreThanStreamLimitRequestsConcurrently_LastWaits(10) will regularly hang in CI.
Discovered in #52704.

I'm unable to reproduce locally, even with the test running in a loop for more than an hour.

Test is disable via a condition in code:

// This combination leads to a hang manifesting in CI only. Disabling it until there's more time to investigate.
if (this.UseQuicImplementationProvider == QuicImplementationProviders.Mock)
{
    return;
}

Update: Seems like the first run of this tends to timeout (now it's the 100), disabling all mock runs for now.

@ManickaP ManickaP added disabled-test The test is disabled in source code against the issue area-System.Net.Quic labels Jun 3, 2021
@ghost
Copy link

ghost commented Jun 3, 2021

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

Issue Details

SocketsHttpHandlerTest_Http3_Mock.SendMoreThanStreamLimitRequestsConcurrently_LastWaits(10) will regularly hang in CI.
Discovered in #52704.

I'm unable to reproduce locally, even with the test running in a loop for more than an hour.

Test is disable via a condition in code:

// This combination leads to a hang manifesting in CI only. Disabling it until there's more time to investigate.
if (streamLimit == 10 && this.UseQuicImplementationProvider == QuicImplementationProviders.Mock)
{
    return;
}
Author: ManickaP
Assignees: -
Labels:

area-System.Net.Quic, disabled-test

Milestone: -

@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged New issue has not been triaged by the area owner label Jun 3, 2021
@ManickaP ManickaP added this to To Do (Low Priority) in HTTP/3 via automation Jun 3, 2021
@ManickaP ManickaP added this to the Future milestone Jun 3, 2021
@ManickaP ManickaP moved this from To Do (Low Priority) to Future in HTTP/3 Jun 3, 2021
@ManickaP ManickaP removed the untriaged New issue has not been triaged by the area owner label Jun 3, 2021
@radical

This comment has been minimized.

@ManickaP
Copy link
Member Author

ManickaP commented Jul 19, 2021

I spun up a new issue.

@ManickaP
Copy link
Member Author

#70421

HTTP/3 automation moved this from Future to Done Jun 22, 2022
@karelz karelz modified the milestones: Future, 7.0.0 Jul 19, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Aug 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Net.Quic disabled-test The test is disabled in source code against the issue
Projects
No open projects
HTTP/3
  
Done
Development

No branches or pull requests

3 participants