-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[wasm] Fix runtime tests build on CI #93646
Conversation
Runtime tests build for wasm started timing out recently, with the job getting cancelled, likely due to the build hitting the container limits. This changes the inner test builds to not run in parallel at all, with `/m:1`, which unblocks the CI at least. Issue: dotnet#93134
Tagging subscribers to 'arch-wasm': @lewing Issue DetailsRuntime tests build for wasm started timing out recently, with the job This changes the inner test builds to not run in parallel at all, with Fixes issue: #93134
|
Tagging subscribers to this area: @hoyosjs Issue DetailsRuntime tests build for wasm started timing out recently, with the job This changes the inner test builds to not run in parallel at all, with Fixes issue: #93134
|
Tagging subscribers to this area: @directhex Issue DetailsRuntime tests build for wasm started timing out recently, with the job This changes the inner test builds to not run in parallel at all, with Fixes issue: #93134
|
|
…rrently built for wasm
I'm not sure why the failing test didn't get skipped. Any objections to merging this with the failed tests to unblock CI for runtime tests? |
Opened #93669 for the failing tests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
Runtime tests build for wasm started timing out recently, with the job
getting cancelled, likely due to the build hitting the container limits.
This changes the inner test builds to not run in parallel at all, with
/m:1
, which unblocks the CI at least.Fixes issue: #93134