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

fix flaky test #40565

Merged
merged 1 commit into from
Aug 8, 2020
Merged

fix flaky test #40565

merged 1 commit into from
Aug 8, 2020

Conversation

geoffkizer
Copy link
Contributor

Fixes #40564

I added this check as part of my recent PR, but apparently the error code is not consistent across environments/configurations.

@Dotnet-GitSync-Bot
Copy link
Collaborator

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@geoffkizer geoffkizer self-assigned this Aug 8, 2020
@geoffkizer geoffkizer added this to the 5.0.0 milestone Aug 8, 2020
@ghost
Copy link

ghost commented Aug 8, 2020

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

Copy link
Member

@wfurt wfurt left a comment

Choose a reason for hiding this comment

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

LGTM.
It would be interesting IMHO o know why it is not consistent.
The "Unknown" looks suspicious.

@geoffkizer
Copy link
Contributor Author

It would be interesting IMHO o know why it is not consistent.
The "Unknown" looks suspicious.

I agree. FWIW I could not repro the Unknown on my box, either Windows or Linux. Best guess is some weird config issue on the CI machines, but it's hard to say for sure.

@geoffkizer geoffkizer merged commit 46acbaa into dotnet:master Aug 8, 2020
Jacksondr5 pushed a commit to Jacksondr5/runtime that referenced this pull request Aug 10, 2020
Co-authored-by: Geoffrey Kizer <[email protected]>
antonfirsov added a commit that referenced this pull request Aug 10, 2020
@geoffkizer geoffkizer deleted the fixflakytest branch November 7, 2020 23:40
@ghost ghost locked as resolved and limited conversation to collaborators Dec 8, 2020
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.

Flaky test CustomConnectionFactory_SyncRequest_Fails is causing CI failures
3 participants