From d88673daa2ca62acf44203cc37345b5d719fe825 Mon Sep 17 00:00:00 2001
From: zhenyus <zhenyus@mathmast.com>
Date: Sat, 8 Mar 2025 15:12:39 +0800
Subject: [PATCH] fix: update Redis alert rules for improved clarity and
 severity levels

Signed-off-by: zhenyus <zhenyus@mathmast.com>
---
 .../3rd/redis-cluster/values.prod.yaml        | 32 ++++---------------
 1 file changed, 6 insertions(+), 26 deletions(-)

diff --git a/freeleaps/helm-pkg/3rd/redis-cluster/values.prod.yaml b/freeleaps/helm-pkg/3rd/redis-cluster/values.prod.yaml
index f3f63002..6b192328 100644
--- a/freeleaps/helm-pkg/3rd/redis-cluster/values.prod.yaml
+++ b/freeleaps/helm-pkg/3rd/redis-cluster/values.prod.yaml
@@ -1110,35 +1110,15 @@ metrics:
     ##      summary: Redis&reg; instance {{ "{{ $instance }}" }} has evicted keys
     ##      description: Redis&reg; instance {{ "{{ $instance }}" }} has evicted {{ "{{ $value }}" }} keys in the last 5 minutes.
     ##
-    rules: 
+    rules:
       - alert: RedisDown
-        expr: redis_up{service="{{ template "common.names.fullname" . }}-metrics"} == 0
-        for: 2m
+        expr: redis_up == 0{service="{{ template "common.names.fullname" . }}-metrics"} == 0
+        for: 0m
         labels:
-          severity: error
+          severity: critical
         annotations:
-          summary: Redis&reg; instance {{ "{{ $instance }}" }} down
-          description: Redis&reg; instance {{ "{{ $instance }}" }} is down.
-      - alert: RedisMemoryHigh
-        expr: >
-            redis_memory_used_bytes{service="{{ template "common.names.fullname" . }}-metrics"} * 100
-            /
-            redis_memory_max_bytes{service="{{ template "common.names.fullname" . }}-metrics"}
-            > 90
-        for: 2m
-        labels:
-          severity: error
-        annotations:
-          summary: Redis&reg; instance {{ "{{ $instance }}" }} is using too much memory
-          description: Redis&reg; instance {{ "{{ $instance }}" }} is using {{ "{{ $value }}" }}% of its available memory.
-      - alert: RedisKeyEviction
-        expr: increase(redis_evicted_keys_total{service="{{ template "common.names.fullname" . }}-metrics"}[5m]) > 0
-        for: 1s
-        labels:
-          severity: error
-        annotations:
-          summary: Redis&reg; instance {{ "{{ $instance }}" }} has evicted keys
-          description: Redis&reg; instance {{ "{{ $instance }}" }} has evicted {{ "{{ $value }}" }} keys in the last 5 minutes.
+          summary: Redis down (instance {{ $labels.instance }})
+          description: "Redis instance is down\n  VALUE = {{ $value }}\n  LABELS = {{ $labels }}"
   ## @param metrics.priorityClassName Metrics exporter pod priorityClassName
   ##
   priorityClassName: ""