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

feat(docker): enable TZ env variable for setting timezone #1733

Merged
merged 1 commit into from
Feb 2, 2024

Conversation

tboerger
Copy link
Collaborator

@tboerger tboerger commented Feb 2, 2024

Supersedes #1730 and fixes #1042.

With the installation of the tzdata package it's possible to provide the TZ environment variable which changes the timezone within the container image.

@fiftin
Copy link
Collaborator

fiftin commented Feb 2, 2024

@tboerger did you test it?

@tboerger
Copy link
Collaborator Author

tboerger commented Feb 2, 2024

Not for the runner itself, but the container is already set to the defined timezone, and since the runner/semaphore is just using the "system" time it should work totally fine:

❯ podman run -ti --rm docker.io/semaphoreui/semaphore:latest date
Fri Feb  2 07:49:51 UTC 2024

❯ podman run -ti --rm -e TZ=Europe/Berlin localhost/semaphore:latest date
Fri Feb  2 08:49:42 CET 202

@fiftin fiftin merged commit 9e4f640 into semaphoreui:develop Feb 2, 2024
4 checks passed
@tboerger tboerger deleted the timezone-fix branch February 2, 2024 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cron at UTC time instead of local time
2 participants