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

DataSourceAPI: Add adhoc filters to DataQueryRequest and make it not depend on global templateSrv #75552

Merged
merged 8 commits into from Sep 28, 2023

Conversation

torkelo
Copy link
Member

@torkelo torkelo commented Sep 27, 2023

The Problem PR is trying to address

Goal is to stop needing to rely on a global adhoc filter set, to support local filters (for split views) and to have more control over which requests the filters should be applied to (instead of always ALL for the same data source).

Changes

  • Explores adding adhocFilters to DataQueryRequest (DataSourceAPI.query method arg)
  • Add adhocFilters arg to interpolateVariablesInQueries
  • Add adhocFilters arg to applyTemplateVariables

… to make it not depend on global templateSrv
@github-actions
Copy link
Contributor

github-actions bot commented Sep 27, 2023

⚠️   Possible breaking changes

(Open the links below in a new tab to go to the correct steps)

grafana-data has possible breaking changes (more info)
grafana-runtime has possible breaking changes (more info)

Console output
Read our guideline

@github-actions github-actions bot added the levitate breaking change A label indicating a breaking change and assigned by Levitate. label Sep 27, 2023
@torkelo torkelo added the no-backport Skip backport of PR label Sep 28, 2023
@torkelo torkelo removed the levitate breaking change A label indicating a breaking change and assigned by Levitate. label Sep 28, 2023
@github-actions github-actions bot added the levitate breaking change A label indicating a breaking change and assigned by Levitate. label Sep 28, 2023
@torkelo torkelo merged commit 2fe4ecd into main Sep 28, 2023
17 of 18 checks passed
@torkelo torkelo deleted the adhoc-filters-to-request branch September 28, 2023 14:29
rwwiv pushed a commit that referenced this pull request Oct 2, 2023
…depend on global templateSrv (#75552)

* DataSourceAPI: Add adhoc filters to DataQueryRequest and some methods to make it not depend on global templateSrv

* Minor tweaks/fixes

* Renamed to filters

* Fix test

* Log deprecation warning

* I give up
mildwonkey pushed a commit that referenced this pull request Oct 4, 2023
…depend on global templateSrv (#75552)

* DataSourceAPI: Add adhoc filters to DataQueryRequest and some methods to make it not depend on global templateSrv

* Minor tweaks/fixes

* Renamed to filters

* Fix test

* Log deprecation warning

* I give up
@zerok zerok modified the milestones: 10.2.x, 10.2.0 Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants