Skip to content
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

Update node to 5.0.0 #1151

Merged
merged 1 commit into from
Oct 31, 2015
Merged

Update node to 5.0.0 #1151

merged 1 commit into from
Oct 31, 2015

Conversation

Starefossen
Copy link
Contributor

This PR updates the node image to v5.0.0 and adds the long term support version argon tag.

Changeset: nodejs/docker-node@04df868...2445743

Related: nodejs/docker-node#59 nodejs/node#3466
Signed-off-by: Hans Kristian Flaatten hans.kristian.flaatten@turistforeningen.no

PR-URL: docker-library#1151

Related: nodejs/docker-node#59 nodejs/node#3466

Signed-off-by: Hans Kristian Flaatten <hans.kristian.flaatten@turistforeningen.no>
@chorrell
Copy link
Contributor

Ha, you beat me to it by like 5 seconds.

@chorrell chorrell mentioned this pull request Oct 30, 2015
@Starefossen
Copy link
Contributor Author

Ah, sorry for the lack of communication @chorrell. I just saw you comment on nodejs/docker-node#60 and that there were no PR to the official repository yet 😄

@chorrell
Copy link
Contributor

No worries!

Looking forward to having that argon tag available! 🎉

@tianon
Copy link
Member

tianon commented Oct 31, 2015

LGTM

$ git diff -C 04df8682a438b0ced8f530ab562f5197595e0cbb...2445743c1453941f787b0aa22cca51c62a1a3f09 -- 4.2 5.0
diff --git a/4.2/Dockerfile b/5.0/Dockerfile
similarity index 97%
copy from 4.2/Dockerfile
copy to 5.0/Dockerfile
index 90e75f7..22311f8 100644
--- a/4.2/Dockerfile
+++ b/5.0/Dockerfile
@@ -14,7 +14,7 @@ RUN set -ex \
   done

 ENV NPM_CONFIG_LOGLEVEL info
-ENV NODE_VERSION 4.2.1
+ENV NODE_VERSION 5.0.0

 RUN curl -SLO "https://nodejs.org/dist/v$NODE_VERSION/node-v$NODE_VERSION-linux-x64.tar.gz" \
   && curl -SLO "https://nodejs.org/dist/v$NODE_VERSION/SHASUMS256.txt.asc" \
diff --git a/4.2/onbuild/Dockerfile b/5.0/onbuild/Dockerfile
similarity index 91%
copy from 4.2/onbuild/Dockerfile
copy to 5.0/onbuild/Dockerfile
index 80ef67c..50d44b1 100644
--- a/4.2/onbuild/Dockerfile
+++ b/5.0/onbuild/Dockerfile
@@ -1,4 +1,4 @@
-FROM node:4.2.1
+FROM node:5.0.0

 RUN mkdir -p /usr/src/app
 WORKDIR /usr/src/app
diff --git a/4.2/slim/Dockerfile b/5.0/slim/Dockerfile
similarity index 97%
copy from 4.2/slim/Dockerfile
copy to 5.0/slim/Dockerfile
index 4ce1c19..4010ee6 100644
--- a/4.2/slim/Dockerfile
+++ b/5.0/slim/Dockerfile
@@ -14,7 +14,7 @@ RUN set -ex \
   done

 ENV NPM_CONFIG_LOGLEVEL info
-ENV NODE_VERSION 4.2.1
+ENV NODE_VERSION 5.0.0

 RUN curl -SLO "https://nodejs.org/dist/v$NODE_VERSION/node-v$NODE_VERSION-linux-x64.tar.gz" \
   && curl -SLO "https://nodejs.org/dist/v$NODE_VERSION/SHASUMS256.txt.asc" \
diff --git a/4.2/wheezy/Dockerfile b/5.0/wheezy/Dockerfile
similarity index 97%
copy from 4.2/wheezy/Dockerfile
copy to 5.0/wheezy/Dockerfile
index 631c67a..75fe0b7 100644
--- a/4.2/wheezy/Dockerfile
+++ b/5.0/wheezy/Dockerfile
@@ -14,7 +14,7 @@ RUN set -ex \
   done

 ENV NPM_CONFIG_LOGLEVEL info
-ENV NODE_VERSION 4.2.1
+ENV NODE_VERSION 5.0.0

 RUN curl -SLO "https://nodejs.org/dist/v$NODE_VERSION/node-v$NODE_VERSION-linux-x64.tar.gz" \
   && curl -SLO "https://nodejs.org/dist/v$NODE_VERSION/SHASUMS256.txt.asc" \

Build test of #1151; e023098 (node):

$ url="https://raw.githubusercontent.com/docker-library/official-images/e023098827c500f1e222ca1187f4b5eb8e00223f/library/node"
$ bashbrew build "$url"
Processing node:0.10.40 ...
Processing node:0.10 ...
Processing node:0.10.40-onbuild ...
Processing node:0.10-onbuild ...
Processing node:0.10.40-slim ...
Processing node:0.10-slim ...
Processing node:0.10.40-wheezy ...
Processing node:0.10-wheezy ...
Processing node:0.12.7 ...
Processing node:0.12 ...
Processing node:0 ...
Processing node:0.12.7-onbuild ...
Processing node:0.12-onbuild ...
Processing node:0-onbuild ...
Processing node:0.12.7-slim ...
Processing node:0.12-slim ...
Processing node:0-slim ...
Processing node:0.12.7-wheezy ...
Processing node:0.12-wheezy ...
Processing node:0-wheezy ...
Processing node:4.2.1 ...
Processing node:4.2 ...
Processing node:4 ...
Processing node:argon ...
Processing node:4.2.1-onbuild ...
Processing node:4.2-onbuild ...
Processing node:4-onbuild ...
Processing node:argon-onbuild ...
Processing node:4.2.1-slim ...
Processing node:4.2-slim ...
Processing node:4-slim ...
Processing node:argon-slim ...
Processing node:4.2.1-wheezy ...
Processing node:4.2-wheezy ...
Processing node:4-wheezy ...
Processing node:argon-wheezy ...
Processing node:5.0.0 ...
Processing node:5.0 ...
Processing node:5 ...
Processing node:latest ...
Processing node:5.0.0-onbuild ...
Processing node:5.0-onbuild ...
Processing node:5-onbuild ...
Processing node:onbuild ...
Processing node:5.0.0-slim ...
Processing node:5.0-slim ...
Processing node:5-slim ...
Processing node:slim ...
Processing node:5.0.0-wheezy ...
Processing node:5.0-wheezy ...
Processing node:5-wheezy ...
Processing node:wheezy ...
$ bashbrew list --uniq "$url" | xargs test/run.sh
testing node:0.10.40
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:0.10.40-onbuild
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:0.10.40-slim
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:0.10.40-wheezy
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:0.12.7
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:0.12.7-onbuild
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:0.12.7-slim
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:0.12.7-wheezy
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:4.2.1
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:4.2.1-onbuild
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:4.2.1-slim
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:4.2.1-wheezy
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:5.0.0
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:5.0.0-onbuild
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:5.0.0-slim
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed
testing node:5.0.0-wheezy
    'utc' [1/5]...passed
    'cve-2014--shellshock' [2/5]...passed
    'no-hard-coded-passwords' [3/5]...passed
    'override-cmd' [4/5]...passed
    'node-hello-world' [5/5]...passed

tianon added a commit that referenced this pull request Oct 31, 2015
@tianon tianon merged commit 7e04656 into docker-library:master Oct 31, 2015
pierreozoux pushed a commit to pierreozoux/official-images that referenced this pull request Jan 7, 2016
PR-URL: docker-library#1151

Related: nodejs/docker-node#59 nodejs/node#3466

Signed-off-by: Hans Kristian Flaatten <hans.kristian.flaatten@turistforeningen.no>
RichardScothern pushed a commit to RichardScothern/official-images that referenced this pull request Jun 14, 2016
PR-URL: docker-library#1151

Related: nodejs/docker-node#59 nodejs/node#3466

Signed-off-by: Hans Kristian Flaatten <hans.kristian.flaatten@turistforeningen.no>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants