-
-
Notifications
You must be signed in to change notification settings - Fork 154
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
Add tzdata package for Docker TZ environment variable usage #333
Conversation
…fect Add tzdata package to allow Docker TZ environment variable to take effect
@goofball222 thanks for the PR! Have you actually tested whether this makes a difference for the game server? Last I checked, the game server didn't respond to timezone adjustments, making them pointless |
It doesn't impact the internal gameserver logging timestamps, however the gameserver does appear to respect the timezone/internal clock setting for its 24h restart period which occurs at 00:00/midnight. The host running this container's timezone is set to UTC. Log section from the overnight self-restart in the container running with a test build for this tzdata change + Ubuntu 24 configured in UTC -7
|
Also note the time difference in the "continue" autosaves on the folder mapped in from the host:
|
Thanks @goofball222!! |
@goofball222 (and everyone reading this) it's not a "24hr interval". I've read on Reddit and Discord, and confirmed by experimenting that this is worded wrong. It is in fact a "time of day". But yeah, thus far it was essentially UTC. I was about to make an issue this week, but I assumed that the server just used UTC, since passing the TZ via "the usual means" (mapping /etc/zoneinfo, /etc/timezone, TZ env var) did not work. I did not know there are things needed in docker. Edit: oh and to confirm, is the |
@GsakuL yes, this has been covered multiple times regarding it not being an interval. The environment variable isn't strictly required, in the same way it realistically never is. |
Add tzdata package for Docker TZ environment variable usage.
Without tzdata installed the container doesn't know how to apply timezone changes / offsets.