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

Bump xharness to fix TCP issue in maccatalyst #82903

Closed
wants to merge 2 commits into from

Conversation

premun
Copy link
Member

@premun premun commented Mar 2, 2023

We started seeing a high number of APP_CRASH results caused by TCP problems so reverting the latest change in XHarness.

The problem means the OSX queues get clogged with work so we need to rush this in to unblock them. That's why this update is being done in isolation and will conflict with #82600

@ghost
Copy link

ghost commented Mar 2, 2023

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

Issue Details

We started seeing a high number of APP_CRASH results caused by TCP problems so reverting the latest change in XHarness

Author: premun
Assignees: premun
Labels:

area-Infrastructure-libraries

Milestone: -

@premun
Copy link
Member Author

premun commented Mar 2, 2023

@steveisok could I ask you to watch over this and merge?

@marek-safar marek-safar added the os-maccatalyst MacCatalyst OS label Mar 2, 2023
@ghost
Copy link

ghost commented Mar 2, 2023

Tagging subscribers to 'os-maccatalyst': @steveisok, @akoeplinger
See info in area-owners.md if you want to be subscribed.

Issue Details

We started seeing a high number of APP_CRASH results caused by TCP problems so reverting the latest change in XHarness.

The problem means the OSX queues get clogged with work so we need to rush this in to unblock them. That's why this update is being done in isolation and will conflict with #82600

Author: premun
Assignees: premun
Labels:

area-Infrastructure-libraries, os-maccatalyst

Milestone: -

@akoeplinger
Copy link
Member

akoeplinger commented Mar 2, 2023

@premun it was using 8d789cbeecb6c89bf470fdc7727a8f501724fc8a before which was before your change so this can't be related I think.

The xharness bump was still pending in #82600.

@steveisok steveisok self-requested a review March 2, 2023 19:23
@premun
Copy link
Member Author

premun commented Mar 3, 2023

@akoeplinger then we need to look in runtime for a regression. There was a considerable amount of maccatalyst crashes coming from all kinds of PRs since March 1st. The retries are clogging the 12.00 queue then.

@premun premun closed this Mar 3, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Apr 2, 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.

3 participants