Update: monitoring mixins set to v1.12.0 for cilium-mixin (#929)

Signed-off-by: Nicolas Lamirault <[email protected]> Co-authored-by: Nicolas Lamirault <[email protected]>
portefaix · Nov 23, 2024 · 6cb7f05 · 6cb7f05
1 parent 5d8801a
commit 6cb7f05
Show file tree

Hide file tree

Showing 3 changed files with 50 additions and 6 deletions.
diff --git a/charts/cilium-mixin/Chart.yaml b/charts/cilium-mixin/Chart.yaml
@@ -27,7 +27,7 @@ keywords:
   - cilium
   - monitoring-mixin
   - portefaix
-version: 0.3.2
+version: 0.4.0
 appVersion: master
 
 maintainers:
@@ -52,5 +52,5 @@ annotations:
     fingerprint: C39918B3EBDE35C23B8D0B8E5F99269A6FCA437C
     url: https://keybase.io/nlamirault/pgp_keys.asc
   artifacthub.io/changes: |
-    - kind: fixed
-      description: key for dashboard json file in lowercase
+    - kind: changed
+      description: cilium-mixin vmaster
diff --git a/charts/cilium-mixin/templates/alerts.yaml b/charts/cilium-mixin/templates/alerts.yaml
@@ -208,4 +208,48 @@ spec:
             for: 15m
             labels:
               severity: info
+      - name: Cilium Clustermesh
+        rules:
+          - alert: CiliumAgentRemoteClusterNotReady
+            annotations:
+              description: Agent can't mesh with {{`{{`}}$labels.target_cluster{{`}}`}}
+              summary: Agent can't mesh with remote cluster.
+            expr: count(cilium_clustermesh_remote_cluster_readiness_status < 1) by (source_cluster, target_cluster) > 0
+            for: 5m
+            labels:
+              severity: critical
+          - alert: CiliumAgentRemoteClusterFailing
+            annotations:
+              description: Agent fails to mesh with {{`{{`}}$labels.target_cluster{{`}}`}}
+              summary: Agent fails to mesh with remote cluster.
+            expr: sum(rate(cilium_clustermesh_remote_cluster_failures[5m])) by (source_cluster, target_cluster) > 0
+            for: 5m
+            labels:
+              severity: critical
+      - name: Cilium Kvstoremesh
+        rules:
+          - alert: CiliumKvstoremeshRemoteClusterNotReady
+            annotations:
+              description: Kvstoremesh can't mesh with {{`{{`}}$labels.target_cluster{{`}}`}}
+              summary: Kvstoremesh can't mesh with remote cluster.
+            expr: count(cilium_kvstoremesh_remote_cluster_readiness_status < 1) by (source_cluster, target_cluster) > 0
+            for: 5m
+            labels:
+              severity: critical
+          - alert: CiliumKvstoremeshRemoteClusterFailing
+            annotations:
+              description: Kvstoremesh fails to mesh with {{`{{`}}$labels.target_cluster{{`}}`}}
+              summary: Kvstoremesh fails to mesh with remote cluster.
+            expr: sum(rate(cilium_kvstoremesh_remote_cluster_failures[5m])) by (source_cluster, target_cluster) > 0
+            for: 5m
+            labels:
+              severity: critical
+          - alert: CiliumKvstoremeshErrors
+            annotations:
+              description: Kvstoremesh fails to mesh with {{`{{`}}$labels.target_cluster{{`}}`}}
+              summary: Kvstoremesh fails to mesh with remote cluster.
+            expr: sum(rate(cilium_kvstoremesh_kvstore_sync_errors_total[5m])) by (source_cluster) > 0
+            for: 5m
+            labels:
+              severity: critical
 {{- end }}
diff --git a/charts/cilium-mixin/templates/configmap-dashboards.yaml b/charts/cilium-mixin/templates/configmap-dashboards.yaml
@@ -17,23 +17,23 @@ items:
       grafana-dashboard: {{ $dashboardName }}
       {{- include "cilium-mixin.labels" $ | indent 6 }}
       app.kubernetes.io/component: dashboard
-    name: {{ printf "dashboard-%s" $dashboardName | trunc 63 | trimSuffix "-"  }}
+    name: {{ printf "dashboard-%s" $dashboardName | trunc 63 | trimSuffix "-" }}
     namespace: {{ include "cilium-mixin.namespace" $ }}
   data:
     {{ $dashboardName }}.json: |-
 {{ $.Files.Get $path | indent 6}}
 {{- end }}
 {{ if $.Values.grafanaDashboard.grafanaOperator.enabled -}}
 {{- range $path, $fileContents := $files }}
-{{- $dashboardName := regexReplaceAll "(^.*/)(.*)\\.json$" $path "${2}" | lower }}
+{{- $dashboardName := regexReplaceAll "(^.*/)(.*)\\.json$" $path "${2}" }}
 ---
 apiVersion: grafana.integreatly.org/v1beta1
 kind: GrafanaDashboard
 metadata:
   labels:
     {{- include "cilium-mixin.labels" $ | indent 4 }}
     app.kubernetes.io/component: dashboard
-  name: {{ printf "%s" $dashboardName | trunc 63 | trimSuffix "-" }}
+  name: {{ printf "%s" $dashboardName | trunc 63 | trimSuffix "-" | lower }}
   namespace: {{ $.Release.Namespace }}
 spec:
   allowCrossNamespaceImport: {{ $.Values.grafanaDashboard.grafanaOperator.allowCrossNamespaceImport }}