[redis] fix: better IPv6 compatibility (CloudPirates-io#296)

* [redis] fix: better IPv6 compatibility

* [redis] bump version of the chart

* [redis] bump version of the chart

Author: dcoppa

charts/redis/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: redis
 description: An open source, in-memory data structure store used as a database, cache, and message broker.
 type: application
-version: 0.6.1
+version: 0.6.2
 appVersion: "8.2.1"
 keywords:
   - redis

charts/redis/templates/prestop-configmap.yaml

@@ -27,9 +27,16 @@ data:
     export REDISCLI_AUTH="${REDIS_PASSWORD}"
     {{- end }}
 
+    # Set loopback address based on ipFamily configuration
+    {{- if eq .Values.ipFamily "ipv6" }}
+    REDIS_LOOPBACK="::1"
+    {{- else }}
+    REDIS_LOOPBACK="127.0.0.1"
+    {{- end }}
+
     # Function to run Redis commands
     run_redis_command() {
-        local args=("-h" "127.0.0.1" "-p" "$REDIS_PORT")
+        local args=("-h" "$REDIS_LOOPBACK" "-p" "$REDIS_PORT")
         redis-cli "${args[@]}" "$@"
     }
 

charts/redis/templates/statefulset.yaml

@@ -60,7 +60,7 @@ spec:
               else
                 # Create minimal config if no config exists
                 cat > /tmp/redis.conf << EOF
-              bind 0.0.0.0
+              bind * -::*
               port 6379
               EOF
               fi
@@ -209,9 +209,17 @@ spec:
                 - /bin/sh
                 - -c
                 {{- if .Values.auth.enabled }}
-                - redis-cli -a ${REDIS_PASSWORD} ping
+                {{- if eq .Values.ipFamily "ipv6" }}
+                - redis-cli -h "::1" -a ${REDIS_PASSWORD} ping
                 {{- else }}
-                - redis-cli ping
+                - redis-cli -h "127.0.0.1" -a ${REDIS_PASSWORD} ping
+                {{- end }}
+                {{- else }}
+                {{- if eq .Values.ipFamily "ipv6" }}
+                - redis-cli -h "::1" ping
+                {{- else }}
+                - redis-cli -h "127.0.0.1" ping
+                {{- end }}
                 {{- end }}
             initialDelaySeconds: {{ .Values.livenessProbe.initialDelaySeconds }}
             periodSeconds: {{ .Values.livenessProbe.periodSeconds }}
@@ -225,20 +233,35 @@ spec:
               command:
                 - /bin/sh
                 - -c
-                - |
-                  {{- if and .Values.sentinel.enabled (eq .Values.architecture "replication") }}
-                  {{- if .Values.auth.enabled }}
-                  redis-cli -a ${REDIS_PASSWORD} ping | grep -q PONG
-                  {{- else }}
-                  redis-cli ping | grep -q PONG
-                  {{- end }}
-                  {{- else }}
-                  {{- if .Values.auth.enabled }}
-                  redis-cli -a ${REDIS_PASSWORD} ping
-                  {{- else }}
-                  redis-cli ping
-                  {{- end }}
-                  {{- end }}
+                {{- if and .Values.sentinel.enabled (eq .Values.architecture "replication") }}
+                {{- if .Values.auth.enabled }}
+                {{- if eq .Values.ipFamily "ipv6" }}
+                - redis-cli -h "::1" -a ${REDIS_PASSWORD} ping | grep -q PONG
+                {{- else }}
+                - redis-cli -h "127.0.0.1" -a ${REDIS_PASSWORD} ping | grep -q PONG
+                {{- end }}
+                {{- else }}
+                {{- if eq .Values.ipFamily "ipv6" }}
+                - redis-cli -h "::1" ping | grep -q PONG
+                {{- else }}
+                - redis-cli -h "127.0.0.1" ping | grep -q PONG
+                {{- end }}
+                {{- end }}
+                {{- else }}
+                {{- if .Values.auth.enabled }}
+                {{- if eq .Values.ipFamily "ipv6" }}
+                - redis-cli -h "::1" -a ${REDIS_PASSWORD} ping
+                {{- else }}
+                - redis-cli -h "127.0.0.1" -a ${REDIS_PASSWORD} ping
+                {{- end }}
+                {{- else }}
+                {{- if eq .Values.ipFamily "ipv6" }}
+                - redis-cli -h "::1" ping
+                {{- else }}
+                - redis-cli -h "127.0.0.1" ping
+                {{- end }}
+                {{- end }}
+                {{- end }}
             initialDelaySeconds: {{ .Values.readinessProbe.initialDelaySeconds }}
             periodSeconds: {{ .Values.readinessProbe.periodSeconds }}
             timeoutSeconds: {{ .Values.readinessProbe.timeoutSeconds }}
@@ -288,7 +311,12 @@ spec:
 
               # Wait for Redis to be ready
               echo "Waiting for Redis to start..."
-              while ! redis-cli {{- if .Values.auth.enabled }} -a "${REDIS_PASSWORD}"{{- end }} -h 127.0.0.1 -p {{ .Values.service.port }} ping >/dev/null 2>&1; do
+              {{- if eq .Values.ipFamily "ipv6" }}
+              REDIS_HOST="::1"
+              {{- else }}
+              REDIS_HOST="127.0.0.1"
+              {{- end }}
+              while ! redis-cli {{- if .Values.auth.enabled }} -a "${REDIS_PASSWORD}"{{- end }} -h "${REDIS_HOST}" -p {{ .Values.service.port }} ping >/dev/null 2>&1; do
                 sleep 1
               done
               echo "Redis is ready"
@@ -355,7 +383,7 @@ spec:
               # Create Sentinel config
               cat > /tmp/sentinel.conf << EOF
               port {{ .Values.sentinel.port }}
-              bind 0.0.0.0
+              bind * -::*
               # Enable hostname resolution for Redis Sentinel
               sentinel resolve-hostnames yes
               sentinel announce-hostnames yes
@@ -407,9 +435,11 @@ spec:
               command:
                 - /bin/sh
                 - -c
-                - |
-                  # Check if sentinel is responding
-                  redis-cli -h 127.0.0.1 -p {{ .Values.sentinel.port }} {{- if .Values.auth.enabled }} -a "${REDIS_PASSWORD}"{{- end }} ping | grep -q PONG
+                {{- if eq .Values.ipFamily "ipv6" }}
+                - redis-cli -h "::1" -p {{ .Values.sentinel.port }} {{- if .Values.auth.enabled }} -a "${REDIS_PASSWORD}"{{- end }} ping | grep -q PONG
+                {{- else }}
+                - redis-cli -h "127.0.0.1" -p {{ .Values.sentinel.port }} {{- if .Values.auth.enabled }} -a "${REDIS_PASSWORD}"{{- end }} ping | grep -q PONG
+                {{- end }}
             initialDelaySeconds: 5
             periodSeconds: 10
             timeoutSeconds: 5

charts/redis/test-production-values.yaml

@@ -97,7 +97,7 @@ readinessProbe:
 config:
   content: |
     # Redis production configuration
-    bind 0.0.0.0
+    bind * -::*
     port 6379
     
     # Memory management

charts/redis/tests/common-parameters_test.yaml

@@ -7,7 +7,7 @@ set:
   config:
     content: |
       # Redis configuration
-      bind 0.0.0.0
+      bind * -::*
       port 6379
 tests:
   - it: should use default values when nothing is overridden

charts/redis/values.yaml

@@ -70,7 +70,7 @@ config:
   ## @param config.content Include your custom Redis configurations here as string
   content: |
     # Redis configuration
-    bind 0.0.0.0
+    bind * -::*
     port 6379
   ## param config.existingConfigmap Name of an existing Configmap to use instead of creating one
   existingConfigmap: ""