Skip to content

Redisable timouting MsQuic tests - #69843

Merged
rzikm merged 1 commit into
dotnet:mainfrom
rzikm:64944-redisable-tests
May 26, 2022
Merged

Redisable timouting MsQuic tests#69843
rzikm merged 1 commit into
dotnet:mainfrom
rzikm:64944-redisable-tests

Conversation

@rzikm

@rzikm rzikm commented May 26, 2022

Copy link
Copy Markdown
Member

Contributes to #64944.

The test is still flaky and sometimes causes timeout for the entire test suite.

cc: @wfurt

@ghost ghost assigned rzikm May 26, 2022
@ghost ghost added the area-System.Net.Quic label May 26, 2022
@ghost

ghost commented May 26, 2022

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

Contributes to #64944.

The test is still flaky and sometimes causes timeout for the entire test suite.

cc: @wfurt

Author: rzikm
Assignees: rzikm
Labels:

area-System.Net.Quic

Milestone: -
@rzikm

rzikm commented May 26, 2022

Copy link
Copy Markdown
Member Author

The timeout occured in #69798

@wfurt

wfurt commented May 26, 2022

Copy link
Copy Markdown
Member

This seems to fail predictably for me as result of #69603 because we did not pull in new msquic for Windows.
With updated msquic (build locally from main) the test is passing fine.

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

I think this is ok for now. We should update msquic and give it another try.

@rzikm

rzikm commented May 26, 2022

Copy link
Copy Markdown
Member Author

/azp run runtime

@azure-pipelines

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

Copy link
Copy Markdown
Member

I don't think this made it to the preview 5 branch which is seeing build failures on PR builds (for example here #69872). Should this change get cherry-picked to preview 5 branch?

@wfurt

wfurt commented May 26, 2022

Copy link
Copy Markdown
Member

I think that would be good @joperezr if #69603 did.

@ghost ghost locked as resolved and limited conversation to collaborators Jun 26, 2022
@karelz karelz added this to the 7.0.0 milestone Jul 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

4 participants