Misthios/grafana-zabbix
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix target backward compatibility

Browse Source
This commit is contained in:
Alexander Zobnin
2017-07-26 12:11:12 +03:00
parent 4020710d00
commit 39f4db5182
10 changed files with 32 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
dist/datasource-zabbix/datasource.js vendored
View File

@@ -315,7 +315,7 @@ System.register(['lodash', 'app/core/utils/datemath', './utils', './migrations',
var useTrends = _this.isUseTrends(timeRange);
// Metrics or Text query mode
if (target.mode === c.MODE_METRICS || target.mode === c.MODE_TEXT || target.mode === c.MODE_ITEMID) {
if (!target.mode || target.mode === c.MODE_METRICS || target.mode === c.MODE_TEXT || target.mode === c.MODE_ITEMID) {
// Migrate old targets
target = migrations.migrate(target);

2
dist/datasource-zabbix/datasource.js.map vendored
View File

File diff suppressed because one or more lines are too long

11
dist/datasource-zabbix/metricFunctions.js vendored
View File

@@ -80,7 +80,8 @@ System.register(['lodash', 'jquery'], function (_export, _context) {
Filter: [],
Trends: [],
Time: [],
Alias: []
Alias: [],
Special: []
};
addFuncDef({
name: 'groupBy',
@@ -239,6 +240,14 @@ System.register(['lodash', 'jquery'], function (_export, _context) {
defaultParams: ['/(.*)/', '$1']
});
// Special
addFuncDef({
name: 'consolidateBy',
category: 'Special',
params: [{ name: 'type', type: 'string', options: ['avg', 'min', 'max', 'sum', 'count'] }],
defaultParams: ['avg']
});
_.each(categories, function (funcList, catName) {
categories[catName] = _.sortBy(funcList, 'name');
});

2
dist/datasource-zabbix/metricFunctions.js.map vendored
View File

File diff suppressed because one or more lines are too long

4
dist/datasource-zabbix/specs/datasource_specs.js vendored
View File

@@ -69,7 +69,7 @@ describe('ZabbixDatasource', () => {
// Check that useTrends options is true
expect(ctx.ds.queryNumericData)
.to.have.been.calledWith(defined, defined, true);
.to.have.been.calledWith(defined, defined, true, sinon.match.any);
});
done();
@@ -85,7 +85,7 @@ describe('ZabbixDatasource', () => {
// Check that useTrends options is false
expect(ctx.ds.queryNumericData)
.to.have.been.calledWith(defined, defined, false);
.to.have.been.calledWith(defined, defined, false, sinon.match.any);
});
done();
});

2
dist/test/datasource-zabbix/datasource.js vendored
View File

@@ -168,7 +168,7 @@ var ZabbixAPIDatasource = function () {
var useTrends = _this.isUseTrends(timeRange);
// Metrics or Text query mode
if (target.mode === c.MODE_METRICS || target.mode === c.MODE_TEXT || target.mode === c.MODE_ITEMID) {
if (!target.mode || target.mode === c.MODE_METRICS || target.mode === c.MODE_TEXT || target.mode === c.MODE_ITEMID) {
// Migrate old targets
target = migrations.migrate(target);

11
dist/test/datasource-zabbix/metricFunctions.js vendored
View File

@@ -29,7 +29,8 @@ var categories = {
Filter: [],
Trends: [],
Time: [],
Alias: []
Alias: [],
Special: []
};
function addFuncDef(funcDef) {
@@ -202,6 +203,14 @@ addFuncDef({
defaultParams: ['/(.*)/', '$1']
});
// Special
addFuncDef({
name: 'consolidateBy',
category: 'Special',
params: [{ name: 'type', type: 'string', options: ['avg', 'min', 'max', 'sum', 'count'] }],
defaultParams: ['avg']
});
_lodash2.default.each(categories, function (funcList, catName) {
categories[catName] = _lodash2.default.sortBy(funcList, 'name');
});

4
dist/test/datasource-zabbix/specs/datasource_specs.js vendored
View File

@@ -84,7 +84,7 @@ describe('ZabbixDatasource', function () {
ctx.ds.query(ctx.options);
// Check that useTrends options is true
expect(ctx.ds.queryNumericData).to.have.been.calledWith(defined, defined, true);
expect(ctx.ds.queryNumericData).to.have.been.calledWith(defined, defined, true, _sinon2.default.match.any);
});
done();
@@ -99,7 +99,7 @@ describe('ZabbixDatasource', function () {
ctx.ds.query(ctx.options);
// Check that useTrends options is false
expect(ctx.ds.queryNumericData).to.have.been.calledWith(defined, defined, false);
expect(ctx.ds.queryNumericData).to.have.been.calledWith(defined, defined, false, _sinon2.default.match.any);
});
done();
});

3
src/datasource-zabbix/datasource.js
View File

@@ -113,7 +113,8 @@ class ZabbixAPIDatasource {
let useTrends = this.isUseTrends(timeRange);
// Metrics or Text query mode
if (target.mode === c.MODE_METRICS || target.mode === c.MODE_TEXT || target.mode === c.MODE_ITEMID) {
if (!target.mode || target.mode === c.MODE_METRICS ||
target.mode === c.MODE_TEXT || target.mode === c.MODE_ITEMID) {
// Migrate old targets
target = migrations.migrate(target);

4
src/datasource-zabbix/specs/datasource_specs.js
View File

@@ -69,7 +69,7 @@ describe('ZabbixDatasource', () => {
// Check that useTrends options is true
expect(ctx.ds.queryNumericData)
.to.have.been.calledWith(defined, defined, true);
.to.have.been.calledWith(defined, defined, true, sinon.match.any);
});
done();
@@ -85,7 +85,7 @@ describe('ZabbixDatasource', () => {
// Check that useTrends options is false
expect(ctx.ds.queryNumericData)
.to.have.been.calledWith(defined, defined, false);
.to.have.been.calledWith(defined, defined, false, sinon.match.any);
});
done();
});