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

Variable query editor #856

Merged
merged 12 commits into from
Jan 13, 2020
Merged

Variable query editor #856

merged 12 commits into from
Jan 13, 2020

Conversation

alexanderzobnin
Copy link
Collaborator

Initial variable query editor implementation. This PR closes #705.
This PR just recreates functionality of legacy query editor (only Group, Host, App and Item queries available) with improved UX, but allows adding new query types: tags, item values (#417). It's also possible to add some filters to query, like enabled/disabled hosts (#362).

This is a basic view of the new component. Notice that regex and template variables highlighted.

Screenshot from 2019-12-31 13-07-40

This is input validation example:

Screenshot from 2019-12-31 13-08-05

@alexanderzobnin alexanderzobnin self-assigned this Dec 31, 2019
@alexanderzobnin alexanderzobnin merged commit 82cfda6 into master Jan 13, 2020
@alexanderzobnin alexanderzobnin deleted the variable-query-editor branch January 13, 2020 08:31
alexanderzobnin added a commit that referenced this pull request Jan 15, 2020
* CI: fix shellcheck issues (#789)

Signed-off-by: Mario Trangoni <[email protected]>

* annotations: fix options in grafana 6.x, fix #813

* fix function editor in Grafana 6.4, closes #810

* add typings for grafana packages

* Add $__range_series variable for calculating function over the whole series, #531

* fix tests

* Don't set alert styles for react panels, fix #823

* docs: add range variables

* docs: percentile reference

* fix codespell

* update packages (build with node 12)

* update circleci node image to 12

* fix test configuration (babel)

* Fix 817 (#851)

* problems: update panel schema

* update packages (build with node 12)

* problems: use datasource from target

* problems: fix query editor after schema update

* problems: fix list layout

* update circleci node image to 12

* fix tests

* build(deps-dev): bump lodash from 4.17.10 to 4.17.13 (#852)

Bumps [lodash](https://github.com/lodash/lodash) from 4.17.10 to 4.17.13.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.10...4.17.13)

Signed-off-by: dependabot[bot] <[email protected]>

* fix packages security alerts

* problems: fix tags adding and removal

* fix adding func from typeahead, closes #468

* update change log

* bump plugin version to 3.10.5

* problems: fix tag removal (list layout)

* Fix percentile() function, closes #862 (#863)

Like the other aggregation functions, the datapoints need to be sorted in
time before calling groupBy_perf().

* Update copyright, happy New Year!

* fix not acknowledged problem color with a message (#858)

* fix not acknowledged problem color with a message

* fix not acknowledged problem color with a message, closes #857

* Variable query editor (#856)

* refactor: convert module to typescript

* refactor: covert utils to typescript

* variable query editor WIP

* variable editor: fix type error after grafana/ui update

* variable editor: use FormLabel from grafana/ui

* variable editor: refactor

* variable editor: input validation and highlights

* variable editor: fix tests

* variable query: fix backward compatibility with empty queries

* fix linter errors

* variable editor: fix variable replacement in queries

* Fixes for backend

Co-authored-by: Mario Trangoni <[email protected]>
Co-authored-by: Alexander Zobnin <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mark Reibert <[email protected]>
Co-authored-by: memfiz <[email protected]>
@konstantin-kornienko
Copy link

@alexanderzobnin , thanks for the update!
Could please describe how disabled hosts can be filtered out?

Btw, there's futher improvements possible.
For example variables like:

  • Filter hosts - show only those hosts where some particular item is exists. For now it's not possible.

This feature will allow build really dynamic dashbords!

@alexanderzobnin
Copy link
Collaborator Author

@konstantin-kornienko that's a good idea! Could you open a feature request?

@polak785
Copy link

Could please describe how disabled hosts can be filtered out?

I'm also interest for that, can you explain please. Or this is not possible for now?

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

Successfully merging this pull request may close these issues.

Improve variable query editor
3 participants