From 493e3f511eb37cca936618d4900b17330c7daf1c Mon Sep 17 00:00:00 2001
From: Roman Pertl <533172+roock@users.noreply.github.com>
Date: Wed, 7 Jun 2023 16:34:50 +0000
Subject: [PATCH] feat: improve rule for used connection on redis

use max allowed connections value instead of a fixed value
---
 dist/rules/redis/oliver006-redis-exporter.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dist/rules/redis/oliver006-redis-exporter.yml b/dist/rules/redis/oliver006-redis-exporter.yml
index 7109972..6afd17b 100644
--- a/dist/rules/redis/oliver006-redis-exporter.yml
+++ b/dist/rules/redis/oliver006-redis-exporter.yml
@@ -86,13 +86,13 @@ groups:
         description: "Redis is running out of configured maxmemory (> 90%)\n  VALUE = {{ $value }}\n  LABELS = {{ $labels }}"
 
     - alert: RedisTooManyConnections
-      expr: 'redis_connected_clients > 100'
+      expr: 'redis_connected_clients / redis_config_maxclients * 100 > 90'
       for: 2m
       labels:
         severity: warning
       annotations:
         summary: Redis too many connections (instance {{ $labels.instance }})
-        description: "Redis instance has too many connections\n  VALUE = {{ $value }}\n  LABELS = {{ $labels }}"
+        description: "Redis is running out of connections (> 90% used)\n  VALUE = {{ $value }}\n  LABELS = {{ $labels }}"
 
     - alert: RedisNotEnoughConnections
       expr: 'redis_connected_clients < 5'