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

Traefik: Introduce the first windows image #4650

Closed
wants to merge 1 commit into from
Closed

Traefik: Introduce the first windows image #4650

wants to merge 1 commit into from

Conversation

dduportal
Copy link
Contributor

Following the issue traefik/traefik#3603 on the Træfik's issue tracker,
I am happy to introduce a first declination of a Windows Container image for Træfik.

This image is based on the nanoserver-sac2016 variant: this is not the most recent, but it is a first start. Although we are planning to improve to other declination, some work have to be done on Golang and Træfik's sides as explained in traefik/traefik#3603 (comment) by @StefanScherer.

The related work can be found here: containous/traefik-library-image#31 .

Let me know if anything else is needed !

PS: a big THANK YOU to @StefanScherer and @tianon for their help, guidance and explanations!

Signed-off-by: Damien DUPORTAL damien.duportal@gmail.com

…16) + Adding myself as maintainer

Signed-off-by: Damien DUPORTAL <damien.duportal@gmail.com>
@dduportal
Copy link
Contributor Author

Following practise in #4602 :

/cc @vdemeester @emilevauge @ldez

Cheers !

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
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.

@dduportal we have few concerns, we should discuss a bit more on this one before merging :)

@dduportal
Copy link
Contributor Author

@emilevauge ack. I'm closing this issue time to fix the internals concerns and we'll go again as soon as possible.

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