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

Flaky Kestrel Test: ClientCanReceiveFullConnectionCloseResponseWithoutErrorAtALowDataRate #14671

Closed
analogrelay opened this issue Oct 2, 2019 · 3 comments
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions blocked The work on this issue is blocked due to some dependency
Milestone

Comments

@analogrelay
Copy link
Contributor

The Kestrel functional test: ClientCanReceiveFullConnectionCloseResponseWithoutErrorAtALowDataRate is only passing 66% of the time in master and less than 50% of the builds most days. We should deflake this test.

It's already quarantined but we need to actually fix it now.

The reported failure is:

System.IO.IOException : Unable to read data from the transport connection: Connection reset by peer.
---- System.Net.Sockets.SocketException : Connection reset by peer
   at Microsoft.AspNetCore.Server.Kestrel.FunctionalTests.ResponseTests.AssertStreamCompletedAtTargetRate(Stream stream, Int64 expectedBytes, Int32 targetBytesPerSecond) in /_/src/Servers/Kestrel/test/FunctionalTests/ResponseTests.cs:line 1039
   at Microsoft.AspNetCore.Server.Kestrel.FunctionalTests.ResponseTests.ClientCanReceiveFullConnectionCloseResponseWithoutErrorAtALowDataRate() in /_/src/Servers/Kestrel/test/FunctionalTests/ResponseTests.cs:line 970
--- End of stack trace from previous location where exception was thrown ---
----- Inner Stack Trace -----

Logs:

| [0.001s] TestLifetime Information: Starting test ClientCanReceiveFullConnectionCloseResponseWithoutErrorAtALowDataRate at 2019-09-27T18:00:17
| [0.050s] Microsoft.AspNetCore.Hosting.Diagnostics Debug: Hosting starting
| [0.051s] Microsoft.AspNetCore.Hosting.Diagnostics Debug: Hosting started
| [0.051s] Microsoft.AspNetCore.Hosting.Diagnostics Debug: Loaded hosting startup assembly Sockets.FunctionalTests, Version=42.42.42.42, Culture=neutral, PublicKeyToken=adb9793829ddae60
| [0.051s] Microsoft.AspNetCore.Server.Kestrel Debug: TestServer is listening on port 46003
| [0.056s] Microsoft.AspNetCore.Server.Kestrel Debug: Connection id "0HLQ3EI5B6SU5" accepted.
| [0.057s] Microsoft.AspNetCore.Server.Kestrel Debug: Connection id "0HLQ3EI5B6SU5" started.
| [0.058s] Microsoft.AspNetCore.Hosting.Diagnostics Information: Request starting HTTP/1.1 GET http:/// - -
| [11.133s] Microsoft.AspNetCore.Hosting.Diagnostics Information: Request finished HTTP/1.1 GET http:/// - - - 200 - - 11074.5875ms
| [11.134s] Microsoft.AspNetCore.Server.Kestrel Debug: Connection id "0HLQ3EI5B6SU5" disconnecting.
| [11.134s] Microsoft.AspNetCore.Server.Kestrel Debug: Connection id "0HLQ3EI5B6SU5" stopped.
| [11.135s] Microsoft.AspNetCore.Server.Kestrel.Transport.Sockets Debug: Connection id "0HLQ3EI5B6SU5" sending FIN because: "The Socket transport's send loop completed gracefully."
| [14.138s] Microsoft.AspNetCore.Hosting.Diagnostics Debug: Hosting shutdown
| [16.006s] TestLifetime Information: Finished test ClientCanReceiveFullConnectionCloseResponseWithoutErrorAtALowDataRate in 16.0048934s
@halter73
Copy link
Member

halter73 commented Oct 8, 2019

This test is already marked flaky. It was added to master post-3.0 to track a regression in the .NET Core 5.0 Linux socket behavior: #13532

This should be fixed once dotnet/corefx#41250 is merged into corefx.

@halter73 halter73 added this to the 5.0.0 milestone Oct 8, 2019
@halter73 halter73 added the blocked The work on this issue is blocked due to some dependency label Oct 8, 2019
@ghost
Copy link

ghost commented Jul 27, 2020

Thanks for contacting us.
We're moving this issue to the Next sprint planning milestone for future evaluation / consideration. We will evaluate the request when we are planning the work for the next milestone. To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@BrennanConroy
Copy link
Member

This has been unquarantined since Feb 28 and the PR that was blocking this was merged last year.

@ghost ghost locked as resolved and limited conversation to collaborators Oct 17, 2020
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Aug 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions blocked The work on this issue is blocked due to some dependency
Projects
None yet
Development

No branches or pull requests

4 participants