-
Notifications
You must be signed in to change notification settings - Fork 5k
[wasm] CI build improvements #69519
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] CI build improvements #69519
Conversation
Tagging subscribers to 'arch-wasm': @lewing Issue Detailsnull
|
8f84e1d
to
140b157
Compare
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
53f5d5c
to
eaf977b
Compare
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
This replaces the job that used `WasmApp.LocalBuild*` stuff to build with a EMSDK checkout. That case already gets tested with library tests. Since we have very few tests to run for the noworkloads case, they are all run in a single helix work item.
Tagging subscribers to this area: @thaystg Issue Details
|
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
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
/azp run runtime-wasm |
Azure Pipelines failed to run 1 pipeline(s). |
/azp run runtime-wasm |
Azure Pipelines failed to run 1 pipeline(s). |
/azp run runtime-wasm |
Azure Pipelines failed to run 1 pipeline(s). |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
Run all the debugger tests in one helix job, because most of them take a couple of minutes to run. And this would avoid the overhead of individual helix jobs
Add a no-workloads variant for
Wasm.Build.Tests
, instead ofEMSDK
WasmApp.LocalBuild*
stuff to buildwith a
EMSDK
checkout. That case already gets tested with library tests.EMSDK
case had been disabled recently due to an Arcade issueall run in a single helix work item.
Fixes #58159
Fixes #67887 (by completely removing the
EMSDK
run)