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

Changing timezone #92

Closed
benhartwich opened this issue Oct 18, 2019 · 8 comments · Fixed by #94
Closed

Changing timezone #92

benhartwich opened this issue Oct 18, 2019 · 8 comments · Fixed by #94

Comments

@benhartwich
Copy link

Hi,

how can I change the date / timezone of the dashlet? It seems that UTC is used by default although my server and icinga has the right timezone set.

@dnsmichi
Copy link
Collaborator

@benhartwich
Copy link
Author

Thanks, what´s the best practise when using docker environment?

@dnsmichi
Copy link
Collaborator

Rebuild the image and run it. Likewise, push it to your own registry. That applies for any changes made to the source code btw, also re-arranging dashlets.

@benhartwich
Copy link
Author

Thanks. Hmm, I think an environment variable would be perfect - if it´s somehow possible. Or using the system´s date. It would be a little bit complicated staying updateable when I rebuild this image for my own, wouldn´t it?

@dnsmichi
Copy link
Collaborator

Sure, I just did not have time to look into a solution here. Likely you can create a similar one like #54 - feel free to create PR.

@dnsmichi
Copy link
Collaborator

You can test the linked PR.

michi@mbpmif ~/dev/github/dashing-icinga2 (feature/clock-timezone-config=) $ DASHBOARD_TIMEZONE="Europe/Berlin" binpaths/ruby/2.6.0/bin/smashing start -p 8005

Screen Shot 2019-10-23 at 10 17 56

@benhartwich
Copy link
Author

Thank you very much - so the new environment variable will be DASHBOARD_TIMEZONE? Have I to build the docker manually or can I use docker pull?

@dnsmichi
Copy link
Collaborator

That's not in Docker atm, that's just a patch for the source code to try.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants