-
Notifications
You must be signed in to change notification settings - Fork 1.2k
alpine variant missing /usr/share/zoneinfo #220
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
Comments
This was actually excluded intentionally ( postgres/Dockerfile-alpine.template Lines 101 to 102 in 03f4064
# tzdata loading functionality is optional, and fails gracefully
# tzdata \ Once it's added, it cannot be removed again (for users who don't need it), and it does add ~1.288 MB to the image size. Something like the following should work to add it after the fact (which will then be imported during FROM postgres:9.6-alpine
RUN apk add --no-cache tzdata |
Thanks for the explanation. |
For anyone interested: there's an unofficial repo of postgres in alpine with tzdata enabled out of the box. |
Is the timezone data used commonly enough that we should reconsider including it by default? Another really simple solution would be to mount in zoneinfo from the host, ie |
It sorta is. The Django documentation suggests that |
It seems to add 1.288 MB to the image |
@ptman it is around 1/50 of image size, which is not that terribly much. |
Yeah, seems sane enough -- someone want to make a PR uncommenting the existing line and removing the comment it has? (probably adding a new one pointing to that Django documentation as a real-world example) |
which results in:
which can be fixed with
The text was updated successfully, but these errors were encountered: