Filter triggers by tags, #487

This commit is contained in:
Alexander Zobnin
2017-12-16 14:21:59 +03:00
parent ea0c5a29b5
commit 401ad69687
10 changed files with 123 additions and 20 deletions

View File

@@ -1,5 +1,8 @@
import _ from 'lodash';
import {DEFAULT_TARGET} from './triggers_panel_ctrl';
// Actual schema version
const CURRENT_SCHEMA_VERSION = 3;
export const CURRENT_SCHEMA_VERSION = 4;
export function migratePanelSchema(panel) {
if (isEmptyPanel(panel)) {
@@ -28,6 +31,14 @@ export function migratePanelSchema(panel) {
delete panel.hideHostsInMaintenance;
}
if (schemaVersion < 4) {
if (panel.targets && !_.isEmpty(panel.targets)) {
_.each(panel.targets, (target) => {
_.defaultsDeep(target, DEFAULT_TARGET);
});
}
}
return panel;
}

View File

@@ -78,6 +78,13 @@
}"
empty-to-null>
</div>
<div class="gf-form">
<label class="gf-form-label query-keyword width-7">Tags</label>
<input type="text" class="gf-form-input"
ng-model="editor.panel.targets[ds].tags.filter"
ng-blur="editor.parseTarget()"
placeholder="tag1:value1, tag2:value2">
</div>
</div>
</div>
</div>

View File

@@ -44,7 +44,7 @@ describe('Triggers Panel schema migration', () => {
let updatedPanelCtrl = new TriggerPanelCtrl(ctx.scope, {}, timeoutMock, datasourceSrvMock, {}, {}, {});
let expected = {
schemaVersion: 3,
schemaVersion: 4,
datasources: ['zabbix'],
targets: {
'zabbix': DEFAULT_TARGET
@@ -80,7 +80,7 @@ describe('Triggers Panel schema migration', () => {
let updatedPanelCtrl = new TriggerPanelCtrl(ctx.scope, {}, {}, datasourceSrvMock, {}, {}, {});
let expected = {
schemaVersion: 3,
schemaVersion: 4,
datasources: ['zabbix_default'],
targets: {
'zabbix_default': DEFAULT_TARGET

View File

@@ -5,7 +5,7 @@ import * as utils from '../datasource-zabbix/utils';
import {PanelCtrl} from 'app/plugins/sdk';
import {triggerPanelOptionsTab} from './options_tab';
import {triggerPanelTriggersTab} from './triggers_tab';
import {migratePanelSchema} from './migrations';
import {migratePanelSchema, CURRENT_SCHEMA_VERSION} from './migrations';
const ZABBIX_DS_ID = 'alexanderzobnin-zabbix-datasource';
@@ -13,7 +13,8 @@ export const DEFAULT_TARGET = {
group: {filter: ""},
host: {filter: ""},
application: {filter: ""},
trigger: {filter: ""}
trigger: {filter: ""},
tags: {filter: ""},
};
export const DEFAULT_SEVERITY = [
@@ -28,7 +29,7 @@ export const DEFAULT_SEVERITY = [
const DEFAULT_TIME_FORMAT = "DD MMM YYYY HH:mm:ss";
export const PANEL_DEFAULTS = {
schemaVersion: 3,
schemaVersion: CURRENT_SCHEMA_VERSION,
datasources: [],
targets: {},
// Fields
@@ -263,12 +264,26 @@ export class TriggerPanelCtrl extends PanelCtrl {
filterTriggersPre(triggerList, ds) {
// Filter triggers by description
var triggerFilter = this.panel.targets[ds].trigger.filter;
let triggerFilter = this.panel.targets[ds].trigger.filter;
triggerFilter = this.datasources[ds].replaceTemplateVars(triggerFilter);
if (triggerFilter) {
triggerList = filterTriggers(triggerList, triggerFilter);
}
// Filter by tags
const target = this.panel.targets[ds];
if (target.tags.filter) {
let tagsFilter = this.datasources[ds].replaceTemplateVars(target.tags.filter);
// replaceTemplateVars() builds regex-like string, so we should trim it.
tagsFilter = tagsFilter.replace('/^', '').replace('$/', '');
const tags = this.parseTags(tagsFilter);
triggerList = _.filter(triggerList, trigger => {
return _.every(tags, (tag) => {
return _.find(trigger.tags, {tag: tag.tag, value: tag.value});
});
});
}
return triggerList;
}
@@ -392,6 +407,19 @@ export class TriggerPanelCtrl extends PanelCtrl {
return trigger;
}
parseTags(tagStr) {
let tags = _.map(tagStr.split(','), (tag) => tag.trim());
tags = _.map(tags, (tag) => {
const tagParts = tag.split(':');
return {tag: tagParts[0].trim(), value: tagParts[1].trim()};
});
return tags;
}
tagsToString(tags) {
return _.map(tags, (tag) => `${tag.tag}:${tag.value}`).join(', ');
}
switchComment(trigger) {
trigger.showComment = !trigger.showComment;
}