feat: add cluster to alerts description (when possible) (#988)

* feat: add cluster to alerts description (when possible) * feat: use showMultiCluster instead of cluster label
kubernetes-monitoring · Nov 25, 2024 · 72a1a23 · 72a1a23
1 parent bdbf7f4
commit 72a1a23
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/alerts/kube_apiserver.libsonnet b/alerts/kube_apiserver.libsonnet
@@ -59,7 +59,10 @@ local utils = import '../lib/utils.libsonnet';
               severity: 'warning',
             },
             annotations: {
-              description: 'A client certificate used to authenticate to kubernetes apiserver is expiring in less than %s.' % (utils.humanizeSeconds($._config.certExpirationWarningSeconds)),
+              description: 'A client certificate used to authenticate to kubernetes apiserver is expiring in less than %s%s.' % [
+                (utils.humanizeSeconds($._config.certExpirationWarningSeconds)),
+                utils.ifShowMultiCluster($._config, ' on cluster {{ $labels.%(clusterLabel)s }}' % $._config),
+              ],
               summary: 'Client certificate is about to expire.',
             },
           },
@@ -75,7 +78,10 @@ local utils = import '../lib/utils.libsonnet';
               severity: 'critical',
             },
             annotations: {
-              description: 'A client certificate used to authenticate to kubernetes apiserver is expiring in less than %s.' % (utils.humanizeSeconds($._config.certExpirationCriticalSeconds)),
+              description: 'A client certificate used to authenticate to kubernetes apiserver is expiring in less than %s%s.' % [
+                (utils.humanizeSeconds($._config.certExpirationCriticalSeconds)),
+                utils.ifShowMultiCluster($._config, ' on cluster {{ $labels.%(clusterLabel)s }}' % $._config),
+              ],
               summary: 'Client certificate is about to expire.',
             },
           },

diff --git a/lib/utils.libsonnet b/lib/utils.libsonnet
@@ -56,4 +56,8 @@
           metric: labels.metric,
         },
       },
+
+  // if showMultiCluster is true in config, return the string, otherwise return an empty string
+  ifShowMultiCluster(config, string)::
+    if config.showMultiCluster then string else '',
 }