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

Settings: Add setting for global default timezone #18539

Closed
wants to merge 1 commit into from

Conversation

samertm
Copy link

@samertm samertm commented Aug 13, 2019

What this PR does / why we need it:

Add the setting "default_timezone" under "[users]" for setting the
default timezone globally.

Most of our monitoring tooling uses UTC, and this will let us set the
global default timezone to UTC instead of having to set the timezone
for every user or dashboard individually.

Tested that the default timezone is still "browser" without any
custom.ini, and that the default timezone changes to "utc" after
adding the following to custom.ini:

[users]
default_timezone = utc

Special notes for your reviewer:

Not sure what docs I should update.

Add the setting "default_timezone" under "[users]" for setting the
default timezone globally.

Most of our monitoring tooling uses UTC, and this will let us set the
global default timezone to UTC instead of having to set the timezone
for every user or dashboard individually.

Tested that the default timezone is still "browser" without any
custom.ini, and that the default timezone changes to "utc" after
adding the following to custom.ini:

```
[users]
default_timezone = utc
```
@torkelo
Copy link
Member

torkelo commented Aug 14, 2019

Not sure we need this, You can set this on the organization level already:

image

@samertm
Copy link
Author

samertm commented Aug 14, 2019

Oh, my bad! That works too :P

@samertm samertm closed this Aug 14, 2019
@samertm
Copy link
Author

samertm commented Aug 14, 2019

It would be cool if someone added this to the grafana docs, since I searched for this kind of setting a bunch and couldn't find it :P

@ying-jeanne ying-jeanne added the pr/external This PR is from external contributor label Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/external This PR is from external contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants