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

docker-compose localtime #1730

Closed
wants to merge 1 commit into from
Closed

docker-compose localtime #1730

wants to merge 1 commit into from

Conversation

cbrookins
Copy link

@cbrookins cbrookins commented Jan 31, 2024

Updates docker-compose example to use the local time of the host

Resolves #1042

Updates docker-compsoe example to use the local time of the host
@linsomniac
Copy link

Did this work for you? I made these changes back in Aug in our environment and it did not resolve my timezone issue.

Copy link
Collaborator

@tboerger tboerger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it should be enouh to mount the localtime:

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

❯ podman run -ti --rm -e TZ=Europe/Berlin docker.io/semaphoreui/semaphore:latest date
Fri Feb  2 07:17:04 UTC 2024

❯ podman run -ti --rm -v /etc/localtime:/etc/localtime docker.io/semaphoreui/semaphore:latest date
Fri Feb  2 08:17:44 CET 2024

@tboerger
Copy link
Collaborator

tboerger commented Feb 2, 2024

@cbrookins I think adding the tzdata package and proividing the TZ environment variables is a much cleaner solution than mounting timezone information from the host system, I have created a PR at #1733.

In the end it's up to @fiftin which solution he prefers :)

@tboerger
Copy link
Collaborator

tboerger commented Feb 2, 2024

I think this PR can get closed now :)

@cbrookins cbrookins closed this Feb 3, 2024
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
3 participants