Environment
- Platform: Ubuntu 24.04.4 LTS
- Docker Version: N/A
- Node.js Version: N/A
- Image Tag: N/A
Expected Behavior
Executing
should "Update all images, skip updating Alpine and Yarn" and the help text displayed by
should be accurate.
Current Behavior
Executing
replaces
in each of the target directories, such as 25/trixie/Dockerfile with
Attempting to build a Docker image with this setting fails.
Possible Solution
- Preserve the original YARN_VERSION or
- Remove the
-s option
In any case, ensure that the help text is accurate.
Although the help text in update.sh refers to Alpine, this part of the logic was been removed without changing the help text.
Considering that Yarn v1 Classic is frozen at 1.22.22 and bundling is scheduled for removal in images for Node.js >=26, there seems to be no need to retain this option.
Steps to Reproduce
git clone https://github.com/nodejs/docker-node
cd docker-node
./update.sh -s 25
git diff
shows every Dockerfile updated with
-ENV YARN_VERSION=1.22.22
+ENV YARN_VERSION=0.0.0
where no change should have been made.
Additional Information
61380fa (PR #2258)
- removed code linking the
-s option to alpine
- removed code that preserved the existing Yarn version
git checkout cc626c1 # previous commit
./update.sh -s 24 # causes only NODE_VERSION to be updated
git checkout 61380fa
./update.sh -s 24 # causes NODE_VERSION to be updated and YARN_VERSION to be set to illegal 0.0.0 version
Environment
Expected Behavior
Executing
should "Update all images, skip updating Alpine and Yarn" and the help text displayed by
should be accurate.
Current Behavior
Executing
replaces
ENV YARN_VERSION=1.22.22in each of the target directories, such as 25/trixie/Dockerfile with
ENV YARN_VERSION=0.0.0Attempting to build a Docker image with this setting fails.
Possible Solution
-soptionIn any case, ensure that the help text is accurate.
Although the help text in update.sh refers to Alpine, this part of the logic was been removed without changing the help text.
Considering that Yarn v1 Classic is frozen at 1.22.22 and bundling is scheduled for removal in images for Node.js >=26, there seems to be no need to retain this option.
Steps to Reproduce
git clone https://github.com/nodejs/docker-node cd docker-node ./update.sh -s 25 git diffshows every Dockerfile updated with
where no change should have been made.
Additional Information
61380fa (PR #2258)
-soption to alpinegit checkout cc626c1 # previous commit
./update.sh -s 24 # causes only NODE_VERSION to be updated
git checkout 61380fa
./update.sh -s 24 # causes NODE_VERSION to be updated and YARN_VERSION to be set to illegal 0.0.0 version