Revert Node 24.14.1 pin in default-build.yml#66979
Conversation
|
Hey @dotnet/aspnet-build, looks like this PR is something you want to take a look at. |
Undoes the Node version pinning (24.x -> 24.14.1 and the node24 -> node24.14.1 cache key changes) introduced in dotnet#66465, while keeping the installNodeJs: false additions and the Cache@2 continueOnError: true. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
e4922c4 to
8ee1874
Compare
There was a problem hiding this comment.
Pull request overview
This PR reverts the Node.js patch-version pinning in the shared AzDO default-build.yml job template (back to 24.x), while preserving the existing approach of conditionally installing Node based on the installNodeJs parameter.
Changes:
- Revert
UseNode@1from24.14.1back to24.xin both job blocks. - Update
node_modulescache keys fromnode24.14.1back tonode24for broader reuse. - Remove
continueOnError: truefrom theCache@2node_modulestask.
|
Seeing if nodejs/node#62991 has been resolved |
Youssef1313
left a comment
There was a problem hiding this comment.
Let's run multiple times before merge to ensure it was indeed fixed
Yep, that's the plan |
|
1st run all good |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
Node.js v24.15.0 and every later 24.x release (24.16.0, 24.17.0, 24.18.0) cause intermittent native crashes in node.exe on Windows CI agents. The current signature is a Windows access violation (0xC0000005, npm error code 3221225477 / MSBuild exit code -1073741819) inside node.exe while running 'rollup -c' for the Components JS workspaces. Crash frequency rises with each release; over the past week the rollup crash was observed on 24.16.0 (2x), 24.17.0 (3x) and 24.18.0 (4x) on the Build: Windows leg, while 24.14.1 had zero crashes. This matches the earlier v24.15.0 incident (0xDEAD during npm-ci extraction) that led to the original 24.14.1 pin (dotnet#66465), which was reverted in dotnet#66979. Re-pin to 24.14.1 (last release with zero observed crashes) until the upstream Node issue is fixed. Also update cache keys to avoid stale node_modules from 24.x builds. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Node.js v24.18.0 (released Jun 23) causes intermittent native crashes in node.exe during the rollup bundling step of the Components JS build on Windows CI agents. The crash is a Windows access violation (0xC0000005, npm error code 3221225477 / MSBuild exit code -1073741819) inside node.exe while running 'rollup -c' for the JS workspaces. CI floats 'version: 24.x' and began pulling v24.18.0 at ~2026-06-24 04:00 UTC (within hours of its release); the crash storm started the same day. Builds on v24.17.0 and earlier are clean. This mirrors the earlier v24.15.0 incident that led to the 24.14.1 pin (#66465), which was later reverted (#66979). Pin to 24.17.0 (last known-good) until the upstream Node issue is fixed. Also update cache keys to avoid stale node_modules from 24.x builds. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
What
Undoes the Node version pinning
.ymlchanges from #66465, while keeping the twoinstallNodeJs: falseadditions and theCache@2continueOnError: true.Specifically reverts, in
.azure/pipelines/jobs/default-build.yml(both job blocks):Install Node 24.14.1→Install Node 24.xversion: 24.14.1→version: 24.xnode24.14.1→node24Not touched
installNodeJs: falseadditions inci-public.ymlandci.ymlare kept.continueOnError: trueon theCache@2node_modules task is kept.ci.yml's Node-install block from Disable npm audit to prevent ETIMEDOUT build failures #66465 was already removed onmainby a later change, so nothing to revert there..ymlchanges from Disable npm audit to prevent ETIMEDOUT build failures #66465 (.npmrc,eng/Npm.Workspace.nodeproj) are out of scope and untouched.Relates to #66465