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.

Sign up for GitHub

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

Jump to bottom

Nginx alpine image #35

Closed
tudddorrr opened this issue May 11, 2021 · 6 comments
Closed

Nginx alpine image #35

tudddorrr opened this issue May 11, 2021 · 6 comments

Comments

@tudddorrr
Copy link

Hey, just wanted to check, is there any reason why the Nginx base image doesn't use the alpine version?
@JonasAlfredsson
Copy link
Owner

Hi tudddorrr,

There is no strong reason for why Apline is not used but I will list some of the reasons:

  • Staticfloat's image was based on the mainline/Debian release of Nginx, and my repository was originally a fork of it so it has remained that way :)
  • My default working environment is Debian, so that is what I am most comfortable working with.
  • You are also the first one to ask about an Alpine version, so I have not really had to give it much thought up until now. :)
  • Nginx seem to indicate that Debian is their primary platform since anything "Alpine" needs "-alpine" attached to the tag.

But if there is a strong desire to provide an Alipne version as well, then I'll see what can be done.

@tudddorrr
Copy link
Author

That all makes sense thanks for letting me know. It's not a big deal really, I was just a bit curious. Cheers.

@JonasAlfredsson JonasAlfredsson mentioned this issue May 13, 2021
Closed
JonasAlfredsson added a commit that referenced this issue May 13, 2021
@JonasAlfredsson
Add Dockerfile that use the Alpine image
8513f41 
A user [wondered][1] why there was no Alpine version of this image,
so I took some time to see if this could be solved.

As of now it works, but I found a very interesting [bug][2] in Bash
(present in both Alpine and Debian) that might even be worth reporting
to those maintainers. However, for now we work around this one by using
an older version of Bash instead (<5.1.0).

[1]: #35
[2]: https://github.com/JonasAlfredsson/bash_fail-to-wait
@JonasAlfredsson
Copy link
Owner

Hey, I just wanted to ping you and say that an Alpine version should now be available.
Check out issue 42 for more details :)

@tudddorrr
Copy link
Author

That was quick, thanks Jonas!

@JonasAlfredsson
Copy link
Owner

No problem, it was a fun challenge :)

I have only had time to do some basic testing of the new image, so if you start using this one would you be so kind and comment in #42 if it works for your use case or if something needs to be fixed?
Thanks in advance :)

@tudddorrr
Copy link
Author

Will do!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tudddorrr @JonasAlfredsson