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

[MacCatalyst][libraries] Test suites flake with App Crash on CI #53624

Open
mdh1418 opened this issue Jun 2, 2021 · 13 comments
Open

[MacCatalyst][libraries] Test suites flake with App Crash on CI #53624

mdh1418 opened this issue Jun 2, 2021 · 13 comments
Assignees
Milestone

Comments

@mdh1418
Copy link
Member

mdh1418 commented Jun 2, 2021

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

All of the crashes are of the same form

Architecture x64
Microsoft.CSharp.Tests - Passes locally but not ran on CI
System.Diagnostics.Tracing.Tests - Still crashed
System.Dynamic.Runtime.Tests - Still crashed
System.Linq.Expressions.Tests - Still crashed
System.Linq.Parallel.Tests - Passed on CI
System.Text.Json.Tests - Still crashed

Architecture Arm64
System.IO.FileSystem.Watcher.Tests - Didn't crash on run 1
System.Net.WebSockets.Client.Tests - Didn't crash on run 1

@mdh1418 mdh1418 added the os-maccatalyst MacCatalyst OS label Jun 2, 2021
@mdh1418 mdh1418 added this to the 6.0.0 milestone Jun 2, 2021
@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged New issue has not been triaged by the area owner label Jun 2, 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
Copy link
Member

@mdh1418 the logs don't tell us much. This isn't high priority, but when you have a moment, try to run this locally through xcode and see what it says there.

@steveisok
Copy link
Member

@premun @mandel-macaque The apps reliably crash on CI really early on. It doesn't seem like they even execute and mlaunch core dumps. If you pull the same app file used on helix and run it local, there is no problem.

@mdh1418 The same crashes seem to happen for iOS and tvOS. Can you please disable for all?

@steveisok steveisok modified the milestones: 6.0.0, 7.0.0 Aug 16, 2021
@premun
Copy link
Member

premun commented Aug 17, 2021

@steveisok I will have a look at the app (probably tomorrow). Do you know - did you try locally on the same Xcode/iOS version?

I have already seen one BCL app before that would:

  • Crash on iOS 13.5
  • Time out on iOS 14.2
  • Succeed on iOS 14.4
    (which is kind of surprising, I would have thought we won't be iOS version sensitive much)

@steveisok
Copy link
Member

I didn't match the xcode version w/ whatever is on CI.

On the app you saw crash, did it actually execute or did mlaunch bail earlier?

@premun
Copy link
Member

premun commented Aug 18, 2021

I am tracking the investigation here - dotnet/xharness#692 - as it was already referenced in couple of places

steveisok pushed a commit that referenced this issue Sep 13, 2021
…t and test fixes for Android (#58841)

Manual backport of #57208, #58519, #58562, #58210, #57732, #58428, #58586, #58745, #57687 to release/6.0

Numerous test suites have been failing for iOS/tvOS/MacCatalyst consistently on CI without useful logs as to why. Moreover, some of these suites pass locally.

This PR looks to reduce the failures on CI by skipping the problematic suites
Skips test suites logged in #53624

ActiveIssues
#58440
#58418
#58367
#58584

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>
Co-authored-by: Jo Shields <directhex@apebox.org>
@lambdageek
Copy link
Member

#59182 may help with System.Text.Json tests

@steveisok
Copy link
Member

@mkhamoyan Can you check on CI to see if this still happens?

@mdh1418
Copy link
Member Author

mdh1418 commented Jul 18, 2022

The test suites under x64 still pass locally, didn't check CI

@mkhamoyan
Copy link
Contributor

@steveisok these ones are still failing on CI

  • System.Dynamic.Runtime
  • System.Net.Requests
  • System.Net.Security

rest of them are passing on CI #72445.

mkhamoyan added a commit that referenced this issue Jul 20, 2022
mkhamoyan added a commit that referenced this issue Jul 25, 2022
mkhamoyan added a commit that referenced this issue Jul 25, 2022
mkhamoyan added a commit that referenced this issue Jul 26, 2022
mkhamoyan added a commit that referenced this issue Jul 26, 2022
mkhamoyan added a commit that referenced this issue Jul 26, 2022
mkhamoyan added a commit that referenced this issue Jul 26, 2022
mkhamoyan added a commit that referenced this issue Jul 28, 2022
* #53624 enable test suites in maccatalyst and update ntrampolines
@mkhamoyan mkhamoyan modified the milestones: 7.0.0, 8.0.0 Jul 28, 2022
@mkhamoyan
Copy link
Contributor

Only failing tests that are still disabled are System.Net.Security and System.Net.Requests.
They are failing with following exception.
certificate
.

@steveisok steveisok modified the milestones: 8.0.0, Future Aug 8, 2023
@mkhamoyan mkhamoyan removed their assignment Jun 25, 2024
@mkhamoyan
Copy link
Contributor

cc @vitek-karas

@vitek-karas
Copy link
Member

/cc @simonrozsival

@simonrozsival simonrozsival self-assigned this Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants