Removed hardwired sentinel port and masterset name. Updated changelog.

Author: arms11

CHANGELOG.md

@@ -2,6 +2,9 @@
 
 ## In Development
 
+## v0.51.0
+* Added Redis with Sentinel to replace Etcd as backend coordinator (#167)
+
 ## v0.50.0
 * Drop Helm `v2` support and fully migrate to Helm `v3` (#163)
 * Switch dependencies from deprecated `helm/charts` to new Bitnami Subcharts (#163)

Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 # StackStorm version which refers to Docker images tag
 appVersion: 3.4dev
 name: stackstorm-ha
-version: 0.50.0
+version: 0.51.0
 description: StackStorm K8s Helm Chart, optimized for running StackStorm in HA environment.
 home: https://stackstorm.com/
 icon: https://landscape.cncf.io/logos/stack-storm.svg

templates/_helpers.tpl

@@ -83,11 +83,13 @@ Create the name of the stackstorm-ha service account to use
 {{- fail "value for redis.sentinel.enabled MUST be true" }}
 {{- end }}
 {{- $replicas := (int (index .Values "redis" "cluster" "slaveCount")) }}
+{{- $master_name := (index .Values "redis" "sentinel" "masterSet") }}
+{{- $sentinel_port := (index .Values "redis" "sentinel" "port") }}
 {{- range $index0 := until $replicas -}}
   {{- if eq $index0 0 -}}
-    {{ $.Release.Name }}-redis-node-{{ $index0 }}.{{ $.Release.Name }}-redis-headless:26379?sentinel=mymaster
+    {{ $.Release.Name }}-redis-node-{{ $index0 }}.{{ $.Release.Name }}-redis-headless:{{ $sentinel_port }}?sentinel={{ $master_name }}
   {{- else -}}
-    &sentinel_fallback={{ $.Release.Name }}-redis-node-{{ $index0 }}.{{ $.Release.Name }}-redis-headless:26379
+    &sentinel_fallback={{ $.Release.Name }}-redis-node-{{ $index0 }}.{{ $.Release.Name }}-redis-headless:{{ $sentinel_port }}
   {{- end -}}
 {{- end -}}
 {{- end -}}