feat: Node.js 24.19.0 - #2587
Conversation
|
There are two updates current at the moment: 24.19.0 & 26.6.0 Only 24.19.0 has a musl build ready at the moment, and I guess that 26.6.0 will take several hours before https://unofficial-builds.nodejs.org provides the musl for that version. The PR was created without a version in the title and so I edited it in. Perhaps that happens when there are two versions active at the same time? The changes to variants have kicked it. They looked correct to me. |
nschonni
left a comment
There was a problem hiding this comment.
The extra diff is just from the architecture cleaning now in the script. Should be less again in the next version
This comment was marked as outdated.
This comment was marked as outdated.
There was a problem hiding this comment.
Pull request overview
This PR updates the Node.js v24 image definitions to Node.js 24.19.0 across Debian (bullseye/bookworm/trixie) and Alpine variants, as part of the repository’s versioned Docker image matrix.
Changes:
- Bump
NODE_VERSIONfrom 24.18.1 to 24.19.0 across all Node 24 Dockerfiles. - Adjust architecture handling logic in multiple Dockerfiles (notably removing some previously-supported arches in Debian variants and simplifying Alpine arch cases).
- Update the Alpine
x86_64musl tarball checksum to match Node.js 24.19.0.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| 24/trixie/Dockerfile | Bumps Node version; modifies Debian arch mapping (armhf removed). |
| 24/trixie-slim/Dockerfile | Bumps Node version; modifies Debian slim arch mapping (armhf removed). |
| 24/bullseye/Dockerfile | Bumps Node version; removes multiple Debian arch mappings (ppc64el/s390x/armhf). |
| 24/bullseye-slim/Dockerfile | Bumps Node version; removes multiple Debian slim arch mappings (ppc64el/s390x/armhf). |
| 24/bookworm/Dockerfile | Bumps Node version; removes Debian arch mappings (s390x/armhf). |
| 24/bookworm-slim/Dockerfile | Bumps Node version; removes Debian slim arch mappings (s390x/armhf). |
| 24/alpine3.24/Dockerfile | Bumps Node version; updates checksum and simplifies Alpine arch/OpenSSL handling. |
| 24/alpine3.23/Dockerfile | Bumps Node version; updates checksum and simplifies Alpine arch/OpenSSL handling. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Copilot's feedback is based on a lack of understanding of the repo's functionality. I would hold back on asking it to review in future. |
|
I guess we could temporarily disable https://github.com/nodejs/docker-node/actions/workflows/automatic-updates.yml , edit the title of this PR. merge it and then wait for the target PR to be merged before enabling the workflow again? Once again, it seems like babysitting the automation is more effort than doing it by hand. |
|
Created PR on the official-images repo (docker-library/official-images#21970). See https://github.com/docker-library/faq#an-images-source-changed-in-git-now-what if you are wondering when it will be available on the Docker Hub. |
Automated changes by create-pull-request GitHub action