Skip to content

Commit b187fbb

Browse files
authoredNov 3, 2024··
fix(redis): CPU stuck at 100% after rolling update (#20645)
Signed-off-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>
1 parent d761c94 commit b187fbb

File tree

4 files changed

+30
-6
lines changed

4 files changed

+30
-6
lines changed
 

‎manifests/ha/base/redis-ha/chart/upstream.yaml

+6-1
Original file line numberDiff line numberDiff line change
@@ -1375,7 +1375,12 @@ spec:
13751375
- mountPath: /health
13761376
name: health
13771377
lifecycle:
1378-
{}
1378+
postStart:
1379+
exec:
1380+
command:
1381+
- /bin/sh
1382+
- -c
1383+
- sleep 30; redis-cli -p 26379 sentinel reset argocd
13791384

13801385
- name: split-brain-fix
13811386
image: public.ecr.aws/docker/library/redis:7.0.15-alpine

‎manifests/ha/base/redis-ha/chart/values.yaml

+10-3
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@ redis-ha:
77
redis:
88
masterGroupName: argocd
99
config:
10-
save: "\"\""
11-
bind: "0.0.0.0"
10+
save: '""'
11+
bind: '0.0.0.0'
1212
haproxy:
1313
enabled: true
1414
IPv6:
@@ -26,4 +26,11 @@ redis-ha:
2626
tag: 7.0.15-alpine
2727
containerSecurityContext: null
2828
sentinel:
29-
bind: "0.0.0.0"
29+
bind: '0.0.0.0'
30+
lifecycle:
31+
postStart:
32+
exec:
33+
command:
34+
- '/bin/sh'
35+
- '-c'
36+
- 'sleep 30; redis-cli -p 26379 sentinel reset argocd'

‎manifests/ha/install.yaml

+7-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎manifests/ha/namespace-install.yaml

+7-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)
Please sign in to comment.