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

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