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

Support timeFilter in query templating for InfluxDB #12598

Merged
merged 3 commits into from
Aug 14, 2018

Conversation

kichristensen
Copy link
Contributor

After support for queries in template variables was added to InfluxDB, it can be necessary to added dymanic time constraints. This can now be done changing the variable refresh to "On Time Range Changed" for InfluxDB

Copy link
Member

@torkelo torkelo left a comment

Choose a reason for hiding this comment

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

a unit test for this would be good.

@kichristensen
Copy link
Contributor Author

kichristensen commented Jul 18, 2018 via email

After support for queries in template variables was added to InfluxDB,
it can be necessary to added dymanic time constraints. This can now be
done changing the variable refresh to "On Time Range Changed" for
InfluxDB
@kichristensen
Copy link
Contributor Author

@torkelo I've now added a unit test for this

@kichristensen
Copy link
Contributor Author

Ahh timezone issues :( I'll fix it

@kichristensen
Copy link
Contributor Author

@torkelo The test should have been fixed now

@torkelo torkelo added this to the 5.3 milestone Aug 2, 2018
@marefr
Copy link
Member

marefr commented Aug 14, 2018

@kichristensen you'll need to sign our Contributor License Agreement (CLA) in order for us to merge this.

@kichristensen
Copy link
Contributor Author

@marefr Done

@marefr marefr merged commit 03aed21 into grafana:master Aug 14, 2018
@marefr
Copy link
Member

marefr commented Aug 14, 2018

Thank you for contributing to Grafana!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/datasource datasource/InfluxDB pr/external This PR is from external contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants