Commit c4048f8f by John McFarlane

[prometheus] Fix step calibration

The step interval is (correctly) being converted to seconds, but the
unit of measure suffixed onto the end is preventing the subsequent
step calibration. Because the query upstream defaults to seconds as
the unit of measure, the suffix can simply be removed and everything
works as intended.

patchset 01: Also fix the spec.
parent 924ecce2
...@@ -212,7 +212,7 @@ function (angular, _, moment, dateMath) { ...@@ -212,7 +212,7 @@ function (angular, _, moment, dateMath) {
sec = 1; sec = 1;
} }
return Math.ceil(sec * intervalFactor) + 's'; return Math.ceil(sec * intervalFactor);
}; };
function transformMetricData(md, options) { function transformMetricData(md, options) {
......
...@@ -19,7 +19,7 @@ describe('PrometheusDatasource', function() { ...@@ -19,7 +19,7 @@ describe('PrometheusDatasource', function() {
var results; var results;
var urlExpected = 'proxied/api/v1/query_range?query=' + var urlExpected = 'proxied/api/v1/query_range?query=' +
encodeURIComponent('test{job="testjob"}') + encodeURIComponent('test{job="testjob"}') +
'&start=1443438675&end=1443460275&step=60s'; '&start=1443438675&end=1443460275&step=60';
var query = { var query = {
range: { from: moment(1443438674760), to: moment(1443460274760) }, range: { from: moment(1443438674760), to: moment(1443460274760) },
targets: [{ expr: 'test{job="testjob"}' }], targets: [{ expr: 'test{job="testjob"}' }],
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment