Merge branch 'main' into feat/proxyVarsFromSecret

oauth2-proxy · May 7, 2024 · 274f6a4 · 274f6a4
2 parents 73adb59 + b814da4
commit 274f6a4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/helm/oauth2-proxy/scripts/check-redis.sh b/helm/oauth2-proxy/scripts/check-redis.sh
@@ -31,12 +31,12 @@ parse_and_check() {
 }
 
 # Main
-if [ "$OAUTH2_PROXY_REDIS_USE_CLUSTER" = "true" ]; then
+if [ -n "$OAUTH2_PROXY_REDIS_CLUSTER_CONNECTION_URLS" ]; then
     echo "Checking Redis in cluster mode..."
     echo "$OAUTH2_PROXY_REDIS_CLUSTER_CONNECTION_URLS" | tr ',' '\n' | while read -r addr; do
         parse_and_check $addr || exit 1
     done
-elif [ "$OAUTH2_PROXY_REDIS_USE_SENTINEL" = "true" ]; then
+elif [ -n "$OAUTH2_PROXY_REDIS_SENTINEL_CONNECTION_URLS" ]; then
     echo "Checking Redis in sentinel mode..."
     echo "$OAUTH2_PROXY_REDIS_SENTINEL_CONNECTION_URLS" | tr ',' '\n' | while read -r addr; do
         parse_and_check $addr || exit 1

diff --git a/helm/oauth2-proxy/templates/deployment.yaml b/helm/oauth2-proxy/templates/deployment.yaml
@@ -72,9 +72,13 @@ spec:
         - name: OAUTH2_PROXY_REDIS_CONNECTION_URL
           value: {{ include "oauth2-proxy.redis.StandaloneUrl" . }}
         {{- else if eq (default "" .Values.sessionStorage.redis.clientType) "cluster" }}
+        - name: OAUTH2_PROXY_REDIS_USE_CLUSTER
+          value: "true"
         - name: OAUTH2_PROXY_REDIS_CLUSTER_CONNECTION_URLS
           value: {{ .Values.sessionStorage.redis.cluster.connectionUrls }}
         {{- else if eq (default "" .Values.sessionStorage.redis.clientType) "sentinel" }}
+        - name: OAUTH2_PROXY_REDIS_USE_SENTINEL
+          value: "true"
         - name: OAUTH2_PROXY_REDIS_SENTINEL_CONNECTION_URLS
           value: {{ .Values.sessionStorage.redis.sentinel.connectionUrls }}
         {{- end }}