-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[android][infra] Runtime tests on Android are not getting sent to Helix #107551
Comments
Tagging subscribers to 'arch-android': @vitek-karas, @simonrozsival, @steveisok, @akoeplinger |
Tagging subscribers to this area: @directhex, @matouskozak |
@jkoritzinsky do you think that #107200 could be the cause? I thought that the changes shouldn't affect Mono/Mobile but only this and #107167 from the range looks to be infra related. |
Unless Android was only "running" those tests (which are skipped on Android) and not running any other runtime tests, it couldn't be those. Can you check which tests were running beforehand? |
Well that's really not good.... I'll see if I can get these jobs back to actually doing work. If you want to get these pipelines green in the meantime, you can disable these jobs as you aren't actually losing any test coverage... |
I agree and I'm unsure why we only used to run Thank you for looking into it. |
I found the issue. There was a change to fix an issue in running the runtime tests on desktop platforms about a year and a half ago which broke all mobile platforms. A later fix re-enabled WASM. I'm using that same fix (as it will work for our use case) for Android. |
Some recent changes likely from this range 516e81e...57a4b04 caused that Android runtime tests are not properly bundled and sent Helix (
SendHelixJob given no WorkItems to send.
)Affected CI:
android-x64 Release AllSubsets_Mono_RuntimeTests minijit
android-x64 Release AllSubsets_Mono_RuntimeTests_Interp monointerpreter
The text was updated successfully, but these errors were encountered: