Make the redis connection threshold a variable for alerting (#604)

* Make the redis connection threshold a variable for alerting * Specifiy redisConnectionsThreshold as string for interpolation Co-authored-by: Louis Taylor <louis@kragniz.eu> Co-authored-by: ioanav <ioanav@graphcore.ai> Co-authored-by: Louis Taylor <louis@kragniz.eu>
oliver006 · Jan 31, 2022 · 1f480f5 · 1f480f5
1 parent caa2061
commit 1f480f5
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/contrib/redis-mixin/alerts/redis.libsonnet b/contrib/redis-mixin/alerts/redis.libsonnet
@@ -30,7 +30,7 @@
           },
           {
             alert: 'RedisTooManyConnections',
-            expr: 'redis_connected_clients > 100',
+            expr: 'redis_connected_clients > %(redisConnectionsThreshold)s' % $._config,
             'for': '5m',
             labels: {
               severity: 'warning',

diff --git a/contrib/redis-mixin/config.libsonnet b/contrib/redis-mixin/config.libsonnet
@@ -0,0 +1,5 @@
+{
+  _config+:: {
+    redisConnectionsThreshold: '100',
+  },
+}
diff --git a/contrib/redis-mixin/mixin.libsonnet b/contrib/redis-mixin/mixin.libsonnet
@@ -1,3 +1,4 @@
 (import 'alerts/redis.libsonnet') +
 (import 'rules/redis.libsonnet') +
-(import 'dashboards/redis.libsonnet')
+(import 'dashboards/redis.libsonnet') +
+(import 'config.libsonnet')