Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

* Changed the plugin to take an interval for a sampling aggregator ra… #78

Open
wants to merge 47 commits into
base: master
Choose a base branch
from

Conversation

jsabin
Copy link

@jsabin jsabin commented Nov 13, 2018

  • Alphabetized the aggregators list.
  • Fixed the legacy target converted to use the interval for a sampling aggregator.
  • Fixed the legacy target converter to not set the unit for the percentile aggregator.
  • All variables are now resolved, not just the metric name.
  • Fixed a bug where options was not available sometimes when metricFindQuery() is called.
  • Fixed bug where a metric only has one tag value for a given tag but there are data points without the tag value. The UI would not let you select the tag. This was also confusing for a single-valued tag because it looked like the tag value was selected but it was not.
  • If you add 2 or more aggregators to a Grafana panel, delete the first, the other will show it as having the interval and alignment from the first aggregator rather than the one you just entered. The fix is to directly call the getVisibleParameters() method from the controller. I believe this was not working because the UI was not refreshing the controller state ($scope.apply() not being called).

Jeff Sabin and others added 30 commits November 13, 2018 11:11
…ther thanusing a separate drop-down for the value and the unit.

* Alphabetized the aggregators list.
* Fixed the legacy target converted to use the interval for a sampling aggregator.
* Fixed the legacy target converter to not set the unit for the percentile aggregator.
* All variables are now resolved, not just the metric name.
* Fixed a bug where options was not available sometimes when metricFindQuery() is called.
* Fixed bug where a metric only has one tag value for a given tag but there are data points without the tag value. The UI would not let you select the tag. This was also confusing for a single valued tag because it looked like the tag value was selected but it was not.
* If you add 2 or more aggregators to a Grafana panel, delete the first, the other will show it as having the interval and alignment from the first aggregator rather than the one you just entered. The fix is to directly call the getVisibleParameters() method from the controller. I believe this was not working because the UI was not refreshing the controller state ($scope.apply() not being called).
@zsabin zsabin mentioned this pull request Dec 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants