Add switch to enable/disable alert-rules

evry-ace · Mar 5, 2021 · 2e53277 · 2e53277
1 parent 5332d16
commit 2e53277
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 5 deletions.
diff --git a/node-dns-exporter/templates/alert-rules.yaml b/node-dns-exporter/templates/alert-rules.yaml
@@ -1,13 +1,15 @@
+{{- if .Values.metrics.enabled }}
 apiVersion: monitoring.coreos.com/v1
 kind: PrometheusRule
 metadata:
   name: {{ template "node-dns-exporter.name" . }}
   namespace: {{ .Release.Namespace }}
   labels:
+    {{- include "node-dns-exporter.labels" . | nindent 4 }}
     alertmanager: {{ .Values.alertManager | default "default" }}
 spec:
   groups:
-  - name: nodeDNS
+  - name: NodeDNS
     rules:
     - alert: NodeDNSerror
       annotations:
@@ -17,6 +19,7 @@ spec:
       for: {{ .Values.alertFor | default "3m" }}
       labels:
         serverity: warning
-{{- if .Values.alertRules }}
-{{ toYaml .Values.alertRules | indent 4 }}
+{{- if .Values.customAlertRules }}
+{{ toYaml .Values.customAlertRules | indent 4 }}
+{{- end }}
 {{- end }}
diff --git a/node-dns-exporter/values.yaml b/node-dns-exporter/values.yaml
@@ -75,12 +75,17 @@ nodeDns:
 
 alertFor: #3m  
 
-alertRules: []
+# Sets the alertmanager instance as a label
+# in case the chart is deployed in an environment with
+# multiple alertmanager instances
+alertManager: #default
+
+customAlertRules: []
 #  - alert: something
 #    annotations:
 #      message: |-
 #        {{` something is not working `}}
 #    expr: ''
 #    for: 10m
 #    labels:
-#      severity: warning
+#      severity: warning