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

[RFE] Add support for InfluxDB `tz` clause #10322

Closed
cykl opened this Issue Dec 21, 2017 · 6 comments

Comments

Projects
None yet
6 participants
@cykl
Copy link
Contributor

cykl commented Dec 21, 2017

  • What Grafana version are you using? 4.6.3
  • What datasource are you using? InfluxDB

InfluxDB supports timezone for queries since 1.3:

Grafana currently does not allow to configure it (unless you switch to the edit mode) and does not automatically set a tz clause. It means that InfluxDB uses UTC to calculate time buckets which is annoying when you are trying to visualize metrics that should be bucketed according to a given timezone rather than UTC.

It would be great to be able to explicitly set a tz. I am also wondering if Grafana shouldn't automatically set tz to the dashboard time zone. Compatibility with InfluxDB < 1.3 should also be taken into account.

@tecmatia-dp

This comment has been minimized.

Copy link

tecmatia-dp commented May 11, 2018

+1
In Table plugin with GROUP BY (1d) is very important the tz() clause.
Although written manually, it is lost when changing edit mode.
The workaround GROUP BY(1d,-2h) is best aproach, but it fails with daylight saving time change.

@pmouawad

This comment has been minimized.

Copy link

pmouawad commented Sep 24, 2018

+1

@tecmatia-dp

This comment has been minimized.

Copy link

tecmatia-dp commented Nov 29, 2018

Any news about this?

@cykl

This comment has been minimized.

Copy link
Contributor Author

cykl commented Dec 3, 2018

@tecmatia-dp Have you tried @jkreileder patch? Maybe a positive feedback could help it getting merged?

@overallduka

This comment has been minimized.

Copy link

overallduka commented Dec 20, 2018

+1

@torkelo

This comment has been minimized.

Copy link
Member

torkelo commented Dec 23, 2018

this is now in nightly master , please test and give feedback!

@marefr marefr modified the milestones: 6.0, 6.0-beta1 Jan 28, 2019

marefr added a commit that referenced this issue Jan 28, 2019

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