-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[release/7.0-rc1] perf pipeline: Switch to node 14.x #74089
[release/7.0-rc1] perf pipeline: Switch to node 14.x #74089
Conversation
- Switch to node 14.x instead of 18.x, because the latter doesn't seem to be compatible with ubuntu 18.x: `node: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.28' not found (required by node)` - Also, remove the existing `nodejs` package before installing a new one, because these machines are shared, and would get affected by installations performed by other runs.
…and set _commandExitCode
This allows cases which don't care about this, to continue working, because they won't set `$PERF_PREREQS_INSTALL_FAILED`, and default to continuing with the job.
Tagging subscribers to this area: @hoyosjs Issue DetailsBackport of #74017 to release/7.0-rc1 /cc @radical Customer ImpactTestingRiskIMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.
|
/azp run runtime-wasm-perf |
Azure Pipelines could not run because the pipeline triggers exclude this branch/path. |
/azp run runtime-wasm-perf |
Azure Pipelines successfully started running 1 pipeline(s). |
Approved and signed off. |
Backport of #74017 to release/7.0-rc1
/cc @radical
Customer Impact
Testing
Risk
IMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.