Skip to content

Revert Node 24.14.1 pin in default-build.yml#66979

Merged
wtgodbe merged 1 commit into
dotnet:mainfrom
wtgodbe:wtgodbe/unpin-node-2414
Jun 8, 2026
Merged

Revert Node 24.14.1 pin in default-build.yml#66979
wtgodbe merged 1 commit into
dotnet:mainfrom
wtgodbe:wtgodbe/unpin-node-2414

Conversation

@wtgodbe

@wtgodbe wtgodbe commented Jun 2, 2026

Copy link
Copy Markdown
Member

What

Undoes the Node version pinning .yml changes from #66465, while keeping the two installNodeJs: false additions and the Cache@2 continueOnError: true.

Specifically reverts, in .azure/pipelines/jobs/default-build.yml (both job blocks):

  • Install Node 24.14.1Install Node 24.x
  • version: 24.14.1version: 24.x
  • Cache/restore keys node24.14.1node24

Not touched

Relates to #66465

Copilot AI review requested due to automatic review settings June 2, 2026 16:17
@wtgodbe wtgodbe requested a review from a team as a code owner June 2, 2026 16:17
@github-actions github-actions Bot added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Jun 2, 2026
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

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>
@wtgodbe wtgodbe force-pushed the wtgodbe/unpin-node-2414 branch from e4922c4 to 8ee1874 Compare June 2, 2026 16:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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@1 from 24.14.1 back to 24.x in both job blocks.
  • Update node_modules cache keys from node24.14.1 back to node24 for broader reuse.
  • Remove continueOnError: true from the Cache@2 node_modules task.

@wtgodbe wtgodbe changed the title Revert Node 24.14.1 pin in default-build.yml (keep installNodeJs: false) Revert Node 24.14.1 pin in default-build.yml Jun 2, 2026
@wtgodbe

wtgodbe commented Jun 2, 2026

Copy link
Copy Markdown
Member Author

Seeing if nodejs/node#62991 has been resolved

@Youssef1313 Youssef1313 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's run multiple times before merge to ensure it was indeed fixed

@wtgodbe

wtgodbe commented Jun 2, 2026

Copy link
Copy Markdown
Member Author

Let's run multiple times before merge to ensure it was indeed fixed

Yep, that's the plan

@wtgodbe

wtgodbe commented Jun 2, 2026

Copy link
Copy Markdown
Member Author

1st run all good
2nd run all good
3rd run all good
4th run all good
5th run all good
6th run all good
7th run all good
8th run all good

@wtgodbe

wtgodbe commented Jun 2, 2026

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@wtgodbe

wtgodbe commented Jun 2, 2026

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@wtgodbe

wtgodbe commented Jun 2, 2026

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@wtgodbe

wtgodbe commented Jun 3, 2026

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@wtgodbe

wtgodbe commented Jun 3, 2026

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@wtgodbe

wtgodbe commented Jun 3, 2026

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@wtgodbe

wtgodbe commented Jun 3, 2026

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@wtgodbe

wtgodbe commented Jun 3, 2026

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@wtgodbe wtgodbe merged commit 672741d into dotnet:main Jun 8, 2026
19 of 25 checks passed
@dotnet-milestone-bot dotnet-milestone-bot Bot added this to the 11.0-preview6 milestone Jun 9, 2026
wtgodbe added a commit to wtgodbe/aspnetcore that referenced this pull request Jun 24, 2026
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>
wtgodbe added a commit that referenced this pull request Jun 25, 2026
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants