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

Bump Traefik to v1.7.0-rc3 #4664

Conversation

mmatur
Copy link
Contributor

@mmatur mmatur commented Aug 1, 2018

Hello,

This PR updates Traefik to v1.7.0-rc3.

And adds the windows docker image (superseeds #4650).

And adds @dduportal as maintainer

/cc @vdemeester @emilevauge @ldez

Cheers

Copy link
Contributor

@ldez ldez left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM 🐸

Copy link

@dtomcej dtomcej left a comment

Choose a reason for hiding this comment

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

LGTM
:shipit:

Copy link
Contributor

@emilevauge emilevauge left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

Copy link

@SantoDE SantoDE left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@juliens juliens left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@nmengin nmengin left a comment

Choose a reason for hiding this comment

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

LGTM

@tianon
Copy link
Member

tianon commented Aug 2, 2018

Diff:
diff --git a/_bashbrew-arches b/_bashbrew-arches
index 8fd1046..9a58598 100644
--- a/_bashbrew-arches
+++ b/_bashbrew-arches
@@ -10,3 +10,4 @@ traefik:maroilles @ arm64v8
 traefik:maroilles-alpine @ amd64
 traefik:maroilles-alpine @ arm32v6
 traefik:maroilles-alpine @ arm64v8
+traefik:maroilles-nanoserver-sac2016 @ windows-amd64
diff --git a/_bashbrew-list b/_bashbrew-list
index 86abfa5..1c9d725 100644
--- a/_bashbrew-list
+++ b/_bashbrew-list
@@ -4,12 +4,18 @@ traefik:1.6.5
 traefik:1.6.5-alpine
 traefik:1.7
 traefik:1.7-alpine
-traefik:1.7.0-rc2
-traefik:1.7.0-rc2-alpine
+traefik:1.7-nanoserver
+traefik:1.7-nanoserver-sac2016
+traefik:1.7.0-rc3
+traefik:1.7.0-rc3-alpine
+traefik:1.7.0-rc3-nanoserver
+traefik:1.7.0-rc3-nanoserver-sac2016
 traefik:alpine
 traefik:latest
 traefik:maroilles
 traefik:maroilles-alpine
+traefik:maroilles-nanoserver
+traefik:maroilles-nanoserver-sac2016
 traefik:tetedemoine
 traefik:tetedemoine-alpine
 traefik:v1.6
@@ -18,5 +24,9 @@ traefik:v1.6.5
 traefik:v1.6.5-alpine
 traefik:v1.7
 traefik:v1.7-alpine
-traefik:v1.7.0-rc2
-traefik:v1.7.0-rc2-alpine
+traefik:v1.7-nanoserver
+traefik:v1.7-nanoserver-sac2016
+traefik:v1.7.0-rc3
+traefik:v1.7.0-rc3-alpine
+traefik:v1.7.0-rc3-nanoserver
+traefik:v1.7.0-rc3-nanoserver-sac2016
diff --git a/traefik_v1.7.0-rc2-alpine/Dockerfile b/traefik_v1.7.0-rc3-alpine/Dockerfile
similarity index 88%
rename from traefik_v1.7.0-rc2-alpine/Dockerfile
rename to traefik_v1.7.0-rc3-alpine/Dockerfile
index 10a6043..b4e0beb 100644
--- a/traefik_v1.7.0-rc2-alpine/Dockerfile
+++ b/traefik_v1.7.0-rc3-alpine/Dockerfile
@@ -9,7 +9,7 @@ RUN set -ex; \
 		*) echo >&2 "error: unsupported architecture: $apkArch"; exit 1 ;; \
 	esac; \
 	apk add --no-cache --virtual .fetch-deps libressl; \
-	wget -O /usr/local/bin/traefik "https://github.com/containous/traefik/releases/download/v1.7.0-rc2/traefik_linux-$arch"; \
+	wget -O /usr/local/bin/traefik "https://github.com/containous/traefik/releases/download/v1.7.0-rc3/traefik_linux-$arch"; \
 	apk del .fetch-deps; \
 	chmod +x /usr/local/bin/traefik
 COPY entrypoint.sh /
@@ -22,5 +22,5 @@ LABEL org.label-schema.vendor="Containous" \
       org.label-schema.url="https://traefik.io" \
       org.label-schema.name="Traefik" \
       org.label-schema.description="A modern reverse-proxy" \
-      org.label-schema.version="v1.7.0-rc2" \
+      org.label-schema.version="v1.7.0-rc3" \
       org.label-schema.docker.schema-version="1.0"
diff --git a/traefik_v1.7.0-rc2-alpine/entrypoint.sh b/traefik_v1.7.0-rc3-alpine/entrypoint.sh
similarity index 75%
rename from traefik_v1.7.0-rc2-alpine/entrypoint.sh
rename to traefik_v1.7.0-rc3-alpine/entrypoint.sh
index 49b3a32..0e7fff6 100755
--- a/traefik_v1.7.0-rc2-alpine/entrypoint.sh
+++ b/traefik_v1.7.0-rc3-alpine/entrypoint.sh
@@ -8,8 +8,8 @@ fi
 
 # if our command is a valid Traefik subcommand, let's invoke it through Traefik instead
 # (this allows for "docker run traefik version", etc)
-if traefik "$1" --help 2>&1 >/dev/null | grep "help requested" > /dev/null 2>&1; then
+if traefik "$1" --help | grep -s -q "help"; then
     set -- traefik "$@"
 fi
 
-exec "$@"
\ No newline at end of file
+exec "$@"
diff --git a/traefik_v1.7.0-rc2/Dockerfile b/traefik_v1.7.0-rc3-nanoserver/Dockerfile
similarity index 66%
copy from traefik_v1.7.0-rc2/Dockerfile
copy to traefik_v1.7.0-rc3-nanoserver/Dockerfile
index e31f36b..6fd1adc 100644
--- a/traefik_v1.7.0-rc2/Dockerfile
+++ b/traefik_v1.7.0-rc3-nanoserver/Dockerfile
@@ -1,6 +1,10 @@
-FROM scratch
-COPY certs/ca-certificates.crt /etc/ssl/certs/
-COPY traefik /
+FROM microsoft/nanoserver:sac2016
+
+COPY ./traefik /traefik.exe
+
+VOLUME C:/etc/traefik
+VOLUME C:/etc/ssl
+
 EXPOSE 80
 ENTRYPOINT ["/traefik"]
 
@@ -9,5 +13,5 @@ LABEL org.label-schema.vendor="Containous" \
       org.label-schema.url="https://traefik.io" \
       org.label-schema.name="Traefik" \
       org.label-schema.description="A modern reverse-proxy" \
-      org.label-schema.version="v1.7.0-rc2" \
+      org.label-schema.version="v1.7.0-rc3" \
       org.label-schema.docker.schema-version="1.0"
diff --git a/traefik_v1.7.0-rc2/traefik b/traefik_v1.7.0-rc3-nanoserver/traefik
similarity index 79%
copy from traefik_v1.7.0-rc2/traefik
copy to traefik_v1.7.0-rc3-nanoserver/traefik
index 2175480..6bb2504 100755
Binary files a/traefik_v1.7.0-rc2/traefik and b/traefik_v1.7.0-rc3-nanoserver/traefik differ
diff --git a/traefik_v1.7.0-rc2/Dockerfile b/traefik_v1.7.0-rc3/Dockerfile
similarity index 88%
rename from traefik_v1.7.0-rc2/Dockerfile
rename to traefik_v1.7.0-rc3/Dockerfile
index e31f36b..29481e4 100644
--- a/traefik_v1.7.0-rc2/Dockerfile
+++ b/traefik_v1.7.0-rc3/Dockerfile
@@ -9,5 +9,5 @@ LABEL org.label-schema.vendor="Containous" \
       org.label-schema.url="https://traefik.io" \
       org.label-schema.name="Traefik" \
       org.label-schema.description="A modern reverse-proxy" \
-      org.label-schema.version="v1.7.0-rc2" \
+      org.label-schema.version="v1.7.0-rc3" \
       org.label-schema.docker.schema-version="1.0"
diff --git a/traefik_v1.7.0-rc2/certs/ca-certificates.crt b/traefik_v1.7.0-rc3/certs/ca-certificates.crt
similarity index 100%
rename from traefik_v1.7.0-rc2/certs/ca-certificates.crt
rename to traefik_v1.7.0-rc3/certs/ca-certificates.crt
diff --git a/traefik_v1.7.0-rc2/traefik b/traefik_v1.7.0-rc3/traefik
similarity index 82%
rename from traefik_v1.7.0-rc2/traefik
rename to traefik_v1.7.0-rc3/traefik
index 2175480..46ed316 100755
Binary files a/traefik_v1.7.0-rc2/traefik and b/traefik_v1.7.0-rc3/traefik differ

@tianon
Copy link
Member

tianon commented Aug 2, 2018

+VOLUME C:/etc/traefik
+VOLUME C:/etc/ssl

This seems a little strange given that the other non-Windows variants don't include VOLUMEs for these -- is this intentional?

+COPY ./traefik /traefik.exe

Is it also intentional that the in-repo copy of traefik.exe doesn't have the .exe suffix? (just checking)

@dduportal
Copy link
Contributor

Hello @tianon ! Thanks for the careful review!

  • About the volumes: you are right, there is an incoherency between the 2 images. I inspired my from from @StefanScherer (github.com/stefanScherer/dockerfiles-windows/tree/master/traefik), and it makes sense to have the volume for the windows image. However, I don't have all the history on the linux images for the rest: this is something that need to be tackled down for the 1.7.0-rc4 if it is ok for everyone?
  • You are totally right to point out the missing extension. This is because of https://github.com/containous/traefik-library-image/blob/master/update.sh#L34 (it can be improved): it is normal since we have the same binary download process for every platforms, and then the Dockerfile is taking care of getting again the extension.

@tianon
Copy link
Member

tianon commented Aug 2, 2018

Those two paths are typically for configuration, right? (not data storage)

So if they're defined as VOLUMEs, then users can't trivially create an image that's FROM traefik:nanoserver which pre-embed the configuration they want to deploy. We don't mind either way (it's y'all's choice what direction to go), but wanted to make sure you're aware and thinking it through with as much info/context as I can provide. 👍

@emilevauge
Copy link
Contributor

@tianon I think we can merge it as is, we will discuss on a possible enhancement for the next release candidate :)

@tianon
Copy link
Member

tianon commented Aug 2, 2018

Aight, LGTM 👍

Build test of #4664; e257380; amd64 (traefik):

$ bashbrew build traefik:v1.7.0-rc3
Building bashbrew/cache:42eda9cb7079e012f2772202633f90aef118c463192b936991f8c9ab52927651 (traefik:v1.7.0-rc3)
Tagging traefik:v1.7.0-rc3
Tagging traefik:1.7.0-rc3
Tagging traefik:v1.7
Tagging traefik:1.7
Tagging traefik:maroilles

$ test/run.sh traefik:v1.7.0-rc3
testing traefik:v1.7.0-rc3
	'cve-2014--shellshock' [1/2]...passed
	'override-cmd' [2/2]...image does not appear to contain "echo" -- assuming single-binary image
passed


$ bashbrew build traefik:v1.7.0-rc3-alpine
Building bashbrew/cache:37f293ad1644eb0b737924dcf849e4a07e48603cd867b1e2c0b8f70d87cc0cfa (traefik:v1.7.0-rc3-alpine)
Tagging traefik:v1.7.0-rc3-alpine
Tagging traefik:1.7.0-rc3-alpine
Tagging traefik:v1.7-alpine
Tagging traefik:1.7-alpine
Tagging traefik:maroilles-alpine

$ test/run.sh traefik:v1.7.0-rc3-alpine
testing traefik:v1.7.0-rc3-alpine
	'cve-2014--shellshock' [1/2]...passed
	'override-cmd' [2/2]...2018/08/02 17:31:51 command echo not found
passed


$ bashbrew build traefik:v1.7.0-rc3-nanoserver
skipping "traefik:v1.7.0-rc3-nanoserver" (due to architecture "amd64"; only "windows-amd64" supported)

$ test/run.sh traefik:v1.7.0-rc3-nanoserver
testing traefik:v1.7.0-rc3-nanoserver
	image does not exist!


$ bashbrew build traefik:v1.6.5
Using bashbrew/cache:7d77a2ae55da9b1fe1ef388a3adb2c96f936e5b8eccd20833e55103fc08ec54a (traefik:v1.6.5)
Tagging traefik:v1.6.5
Tagging traefik:1.6.5
Tagging traefik:v1.6
Tagging traefik:1.6
Tagging traefik:tetedemoine
Tagging traefik:latest

$ test/run.sh traefik:v1.6.5
testing traefik:v1.6.5
	'cve-2014--shellshock' [1/2]...passed
	'override-cmd' [2/2]...image does not appear to contain "echo" -- assuming single-binary image
passed


$ bashbrew build traefik:v1.6.5-alpine
Using bashbrew/cache:114c600e7ab4900374c8cbfdcabac1cd0177d5f4f1e12430cbf2ede37a47672a (traefik:v1.6.5-alpine)
Tagging traefik:v1.6.5-alpine
Tagging traefik:1.6.5-alpine
Tagging traefik:v1.6-alpine
Tagging traefik:1.6-alpine
Tagging traefik:tetedemoine-alpine
Tagging traefik:alpine

$ test/run.sh traefik:v1.6.5-alpine
testing traefik:v1.6.5-alpine
	'cve-2014--shellshock' [1/2]...passed
	'override-cmd' [2/2]...passed

The following images failed at least one test:

  • traefik:
    • traefik:v1.7.0-rc3-nanoserver

@tianon tianon merged commit a0b3266 into docker-library:master Aug 2, 2018
@seal-ss
Copy link

seal-ss commented Aug 3, 2018

Thanks @tianon I can't remember why I used/tried it with VOLUME on Windows 😅

@dduportal
Copy link
Contributor

@seal-ss don't worry, we got this: containous/traefik-library-image#34 (for the rc4 ).

@mmatur mmatur deleted the traefik-new-version-6dadcaf64497da64d2283f2e86302798554f1539 branch January 3, 2024 14:05
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.