containous / traefik-library-image

ARCHIVED
https://github.com/traefik/traefik-library-image
Apache License 2.0
218 stars 60 forks source link

Windows build 1809 compatibility #49

Closed jwlogemann closed 5 years ago

jwlogemann commented 5 years ago

I've changed the Dockerfile to be Windows build 1809 compatible. This is needed when using Traefik as a Kubernetes ingress controller in Windows Server 2019

Fixes #44

zlepper commented 5 years ago

I can confirm that it seems to be working. It is at least both building and running.

For anyone currious why the "netapi32" file is there, it's to fix: https://github.com/golang/go/issues/21867

Barsonax commented 5 years ago

Any ETA when this is getting merged?

dduportal commented 5 years ago

Hi @jwlogemann, thanks for this contribution, it's cool!

This PR is currently being blocked for the following reasons:

Alas, by committing directly the binary as this PR, neither of these 3 elements are fulfilled, which would (at least) make the image refused by Docker when trying to publish it.

What should we do then?

Well, TL;DR; we have to wait for https://github.com/docker-library/official-images/pull/5929 to be ready.

Details:

Proposal: blocking this PR until Multi-stage build are supported in Docker's official library.

jorisscheppers commented 5 years ago

Proposal: blocking this PR until Multi-stage build are supported in Docker's official library.

As docker-library/official-images#5929 has recently been merged, can work on this PR continue? I would really like to see Traefik work on Windows Server 2019!

dduportal commented 5 years ago

Yes totally, let's update this PR following the new bashbrew!