Skip to content

Use timeouts in 2 socket test cases - #84030

Merged
antonfirsov merged 4 commits into
dotnet:mainfrom
antonfirsov:SendRecv_Stream_TCP_LargeMultiBufferSends-timeouts
Apr 3, 2023
Merged

Use timeouts in 2 socket test cases#84030
antonfirsov merged 4 commits into
dotnet:mainfrom
antonfirsov:SendRecv_Stream_TCP_LargeMultiBufferSends-timeouts

Conversation

@antonfirsov

Copy link
Copy Markdown
Contributor

We noticed several test process hangs in these tests recently. This should help collecting more specific failure data, and avoid failing the entire test run. /cc @rzikm

@ghost ghost assigned antonfirsov Mar 28, 2023
@ghost

ghost commented Mar 28, 2023

Copy link
Copy Markdown

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

Issue Details

We noticed several test process hangs in these tests recently. This should help collecting more specific failure data, and avoid failing the entire test run. /cc @rzikm

Author: antonfirsov
Assignees: antonfirsov
Labels:

area-System.Net.Sockets

Milestone: -

@antonfirsov

Copy link
Copy Markdown
Contributor Author

/azp run runtime-libraries-coreclr outerloop-linux

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@antonfirsov

Copy link
Copy Markdown
Contributor Author

/azp run runtime-libraries-coreclr outerloop-linux

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@wfurt wfurt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@antonfirsov

Copy link
Copy Markdown
Contributor Author

/azp run runtime-libraries-coreclr outerloop-linux

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@antonfirsov

Copy link
Copy Markdown
Contributor Author

/azp run runtime-libraries-coreclr outerloop-linux

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@antonfirsov

Copy link
Copy Markdown
Contributor Author

Test failures are unrelated, eg. #84188

@antonfirsov
antonfirsov merged commit 4b5bb17 into dotnet:main Apr 3, 2023
@ghost ghost locked as resolved and limited conversation to collaborators May 3, 2023
@karelz karelz added this to the 8.0.0 milestone May 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants