Skip to content

Commit

Permalink
Mixin: Rename exclusion rule from panel-job-instance-rule to `targe…
Browse files Browse the repository at this point in the history
…t-instance-rule`

Within grafana/dashboard-linter@9a32e58, the rules have been split into two different rules:

`target-job-rule`
`target-instance-rule`

All of our queries do contain the `job` label but as per the reason, we don't need both in this particular case.

Fixes #2899

Signed-off-by: gotjosh <[email protected]>
  • Loading branch information
gotjosh committed May 2, 2022
1 parent 71d61c9 commit 35bf59f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions doc/alertmanager-mixin/.lint
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
exclusions:
panel-job-instance-rule:
target-instance-rule:
reason: no need to have every query contains two matchers within every selector - `{job=~"$job", instance=~"$instance"}`
template-job-rule:
template-job-rule:
entries:
- dashboard: Alertmanager / Overview
reason: multi-select is not always required

0 comments on commit 35bf59f

Please sign in to comment.