From f8d4b4ef913316498d17056a8150ad0484808c57 Mon Sep 17 00:00:00 2001 From: Emily Ahlstrand Rager Date: Wed, 25 Jan 2023 13:18:47 +0100 Subject: [PATCH] Added lint config & alert summaries Signed-off-by: Vitaly Zhuravlev --- contrib/mixin/.lint | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) create mode 100644 contrib/mixin/.lint diff --git a/contrib/mixin/.lint b/contrib/mixin/.lint new file mode 100644 index 000000000..d373b8e2f --- /dev/null +++ b/contrib/mixin/.lint @@ -0,0 +1,18 @@ +--- +exclusions: + template-instance-rule: + reason: The mixin only uses `instance` for alerts, and `cluster` for dashboard queries + template-job-rule: + reason: The dashboards use 'cluster' label as selector, rather than 'job' + target-job-rule: + reason: The mixin uses 'cluster' instead of 'job' + target-instance-rule: + reason: The mixin only uses `instance` for alerts, and `cluster` for dashboard queries + alert-name-camelcase: + reason: etcd is spelled all lowercase, meaning all alert name start with a lowercase + alert-summary-style: + reason: etcd is spelled all lowercase, meaning summaries starting with 'etcd' are still valid + panel-units-rule: + reason: Stat panels have no unit, and some panels use custom unit or text + panel-title-description-rule: + reason: Suppress noisy linting rule until we can address minor tech debt like this