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.

Sign up for GitHub

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

Jump to bottom

[MacCatalyst][libraries] Test suites flake with App Launch Failure on CI #53813

Closed
mdh1418 opened this issue Jun 7, 2021 · 1 comment · Fixed by #56625
Closed

[MacCatalyst][libraries] Test suites flake with App Launch Failure on CI #53813

mdh1418 opened this issue Jun 7, 2021 · 1 comment · Fixed by #56625
Assignees
@mdh1418
Labels
os-maccatalyst MacCatalyst OS
Milestone
7.0.0

Comments

@mdh1418
Copy link
Member

mdh1418 commented Jun 7, 2021 •
edited
Loading

In #53197, a few test suites were flaking on CI with App Crash

All of the crashes are of the same form as https://gist.github.com/mdh1418/300435dda920074f3b410df7d3448cd2

Architecture Arm64
System.Composition.AttributeModel.Tests
System.Composition.TypedParts.Tests
System.Diagnostics.DiagnosticSource.Switches.Tests
System.Globalization.Calendars.Tests
System.IO.UnmanagedMemoryStream.Tests
System.Web.HttpUtility.Tests
@mdh1418 mdh1418 added the os-maccatalyst MacCatalyst OS label Jun 7, 2021
@mdh1418 mdh1418 added this to the 6.0.0 milestone Jun 7, 2021
@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged New issue has not been triaged by the area owner label Jun 7, 2021
@dotnet-issue-labeler
Copy link

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

@steveisok steveisok removed the untriaged New issue has not been triaged by the area owner label Jun 28, 2021
@steveisok steveisok modified the milestones: 6.0.0, 7.0.0 Jul 29, 2021
@mdh1418 mdh1418 self-assigned this Jul 29, 2021
@mdh1418 mdh1418 mentioned this issue Jul 30, 2021
Merged
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Aug 2, 2021
steveisok pushed a commit that referenced this issue Aug 6, 2021
@mdh1418
Reenable MacCatalyst Arm64 crashing test suites (#56625)
ce3447c 
Evaluating the state of flakey test suite crashes #53813

1: no failures
2: no failures
3: no failures
4: no failures (but noticed that System.Diagnostics.DiagnosticSource.Switches.Tests had 0 tests due to ActiveIssue at class level)
5: no failures (Remove MacCatalyst Skip instead of Android skip for System.Diagnostics.DiagnosticSource.Switches.Tests)
6: no failures

From several test runs of MacCatalyst arm64 of the crashing test suites in #53813, none were found to be crashing anymore, so its suspected that a fix had made it in.

This PR looks to re-enable those test suites to be ran on MacCatalyst arm64 and to combine the Active Issues for Android and iOS/tvOS into one as they have the same failure.

Fixes #53813

System.Diagnostic.DiagnosticSource.Switches.Tests is re-enabled in #56737
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Aug 6, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Sep 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mdh1418 mdh1418

Labels
os-maccatalyst MacCatalyst OS
Projects
None yet
Milestone
7.0.0
Development

Successfully merging a pull request may close this issue.

Reenable MacCatalyst Arm64 crashing test suites mdh1418/runtime
3 participants
@steveisok @jeffschwMSFT @mdh1418