-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Disable some suspicious tests which might lead to OOM crashes on Android emulators and arm64 devices #62767
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
Disable some suspicious tests which might lead to OOM crashes on Android emulators and arm64 devices #62767
Conversation
…on Android emulators and arm64 devices
|
Tagging subscribers to this area: @dotnet/area-system-security, @vcsjones, @krwq Issue DetailsJust blindly disabling several crypto test classes near which crashes have been observed for a while (based on a couple of the mobile targets rolling build runs).
|
|
/azp run runtime-manual |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run runtime-manual |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@akoeplinger Ideally, I'd like to run less iterations of tests if we can. Would it be better to skip the crypto suite outright on emulators in order to have max coverage that runs on devices? |
…WriteSubjectPublicKeyInfo()
|
Hm I think it should be possible to reduce the iteration count for the tests, but I haven't looked too deeply. I'm fine with disabling for now. |
|
/azp run runtime-manual |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
There is sill a crash on |
|
Yeah if you look a little further in that adb log you can see that essentially the whole system crashed and rebooted. That might be still because of the crypto tests e.g. if they still consume too much RAM and the system gets overloaded, but I think we can monitor that in the rolling build and react if necessary. |
Just blindly disabling several crypto tests near which crashes have been observed for a while (based on a couple of the mobile targets rolling build runs).
Disabled tests:
See also #62547.
cc @steveisok @akoeplinger