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.

Sign up for GitHub

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

Jump to bottom

Custom all value does not replace variable interpolation #12965

Closed
franciscocpg opened this issue Aug 17, 2018 · 0 comments · Fixed by #12967
Closed

Custom all value does not replace variable interpolation #12965

franciscocpg opened this issue Aug 17, 2018 · 0 comments · Fixed by #12967
Labels
area/dashboard/templating type/feature-request
Milestone
5.3.0-beta1

Comments

@franciscocpg
Copy link
Contributor

franciscocpg commented Aug 17, 2018 •
edited
Loading

What Grafana version are you using?

Grafana v5.2.1 (2040f61)

What datasource are you using?

Cloudwatch

What OS are you running grafana on?

Linux

What did you do?

I'm using the value of a variable (A) as Custom all value in another query variable (B) with Include All Option enabled. So the Custom all value field of variable B is: $A.

What was the expected result?

I expected the value of All in query variable B to be replaced by the value of the variable A.

What happened instead?

The value of All in query variable B is not replaced by the value of the variable A and its raw value ($A) is used instead.

If related to metric query / data viz:

n/a

Include raw network request & response: get by opening Chrome Dev Tools (F12, Ctrl+Shift+I on windows, Cmd+Opt+I on Mac), go the network tab.

n/a
@franciscocpg franciscocpg mentioned this issue Aug 17, 2018
Merged
marefr added a commit that referenced this issue Aug 24, 2018
@marefr
changelog: add notes about closing #12952 #12965
e906d4b 
[skip ci]
@marefr marefr added this to the 5.3 milestone Aug 24, 2018
ryantxu added a commit to ryantxu/grafana that referenced this issue Aug 29, 2018
@ryantxu
Merge remote-tracking branch 'grafana/master'
8cfad74 
* grafana/master: (52 commits)
  Moved tooltip icon from input to label grafana#12945 (grafana#13059)
  added empty cta to playlist page + hid playlist table when empty (grafana#12841)
  Update provisioning.md
  changelog: add notes about closing grafana#12865
  fixed so validation of empty fields works again
  tslint: tslint to const fixes part3 (grafana#13036)
  tslint: more const fixes (grafana#13035)
  tslint: changing vars -> const (grafana#13034)
  tslint: autofix of let -> const (grafana#13033)
  WIP Update tslint (grafana#12922)
  changelog: add notes about closing grafana#12952 grafana#12965
  build: fixes rpm build when using defaults.
  changelog: add notes about closing grafana#12486
  docs: changes
  fixed so animation starts as soon as one pushes the button and animation stops if login failed
  added link to getting started to all, changed wording
  tests: fix missing tests (with .jest suffix)
  heatmap: fix tooltip bug in firefox
  Update notifications.md
  sql: added code migration type
  ...
ryantxu added a commit to ryantxu/grafana that referenced this issue Aug 29, 2018
@ryantxu
Merge remote-tracking branch 'grafana/master' into panel-visibility-flag
df8b9fc 
* grafana/master: (52 commits)
  Moved tooltip icon from input to label grafana#12945 (grafana#13059)
  added empty cta to playlist page + hid playlist table when empty (grafana#12841)
  Update provisioning.md
  changelog: add notes about closing grafana#12865
  fixed so validation of empty fields works again
  tslint: tslint to const fixes part3 (grafana#13036)
  tslint: more const fixes (grafana#13035)
  tslint: changing vars -> const (grafana#13034)
  tslint: autofix of let -> const (grafana#13033)
  WIP Update tslint (grafana#12922)
  changelog: add notes about closing grafana#12952 grafana#12965
  build: fixes rpm build when using defaults.
  changelog: add notes about closing grafana#12486
  docs: changes
  fixed so animation starts as soon as one pushes the button and animation stops if login failed
  added link to getting started to all, changed wording
  tests: fix missing tests (with .jest suffix)
  heatmap: fix tooltip bug in firefox
  Update notifications.md
  sql: added code migration type
  ...
ryantxu added a commit to ryantxu/grafana that referenced this issue Aug 29, 2018
@ryantxu
Merge remote-tracking branch 'grafana/master' into lazy-load-panels
4b7d870 
* grafana/master: (52 commits)
  Moved tooltip icon from input to label grafana#12945 (grafana#13059)
  added empty cta to playlist page + hid playlist table when empty (grafana#12841)
  Update provisioning.md
  changelog: add notes about closing grafana#12865
  fixed so validation of empty fields works again
  tslint: tslint to const fixes part3 (grafana#13036)
  tslint: more const fixes (grafana#13035)
  tslint: changing vars -> const (grafana#13034)
  tslint: autofix of let -> const (grafana#13033)
  WIP Update tslint (grafana#12922)
  changelog: add notes about closing grafana#12952 grafana#12965
  build: fixes rpm build when using defaults.
  changelog: add notes about closing grafana#12486
  docs: changes
  fixed so animation starts as soon as one pushes the button and animation stops if login failed
  added link to getting started to all, changed wording
  tests: fix missing tests (with .jest suffix)
  heatmap: fix tooltip bug in firefox
  Update notifications.md
  sql: added code migration type
  ...
ryantxu added a commit to ryantxu/grafana that referenced this issue Aug 29, 2018
@ryantxu
Merge remote-tracking branch 'grafana/master' into timepicker-tooltip
01acba4 
* grafana/master: (52 commits)
  Moved tooltip icon from input to label grafana#12945 (grafana#13059)
  added empty cta to playlist page + hid playlist table when empty (grafana#12841)
  Update provisioning.md
  changelog: add notes about closing grafana#12865
  fixed so validation of empty fields works again
  tslint: tslint to const fixes part3 (grafana#13036)
  tslint: more const fixes (grafana#13035)
  tslint: changing vars -> const (grafana#13034)
  tslint: autofix of let -> const (grafana#13033)
  WIP Update tslint (grafana#12922)
  changelog: add notes about closing grafana#12952 grafana#12965
  build: fixes rpm build when using defaults.
  changelog: add notes about closing grafana#12486
  docs: changes
  fixed so animation starts as soon as one pushes the button and animation stops if login failed
  added link to getting started to all, changed wording
  tests: fix missing tests (with .jest suffix)
  heatmap: fix tooltip bug in firefox
  Update notifications.md
  sql: added code migration type
  ...
@marefr marefr modified the milestones: 5.3, 5.3.0-beta1 Sep 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/dashboard/templating type/feature-request
Projects
None yet
Milestone
5.3.0-beta1
Development

Successfully merging a pull request may close this issue.

Replacing variable interpolation in "All value" value franciscocpg/grafana
3 participants
@xlson @marefr @franciscocpg