Protei/PromucFlow_constructor
2
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Fix redis sub chart name in helm chart (#17866)

Browse Source 
Currently, the helm chart has a minor bug where the chart name is
hard-coded. This fix addresses the issue
This commit is contained in:
Goutham Pratapa 2022-10-28 10:25:17 +05:30 committed by GitHub
parent 3f32308252
commit 5c5fbaa9f0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
deploy/helm/templates/configMap.yaml
View File

@ -1,4 +1,5 @@
{{- $nameSpace := include "appsmith.namespace" . -}}
{{- $name := include "appsmith.fullname" . -}}
{{- $mongoUser := .Values.mongodb.auth.rootUser -}}
{{- $mongoPassword := .Values.mongodb.auth.rootPassword -}}
{{- $mongoServicename := .Values.mongodb.service.nameOverride -}}
@ -18,7 +19,7 @@ data:
{{- end }}
{{- if and (eq "APPSMITH_REDIS_URL" $key) ( not $value) }}
{{- if $.Values.redis.enabled }}
{{ $key }}: redis://appsmith-redis-master.{{ $nameSpace }}.svc.cluster.local:6379
{{ $key }}: redis://{{ $name }}-redis-master.{{ $nameSpace }}.svc.cluster.local:6379
{{- end }}
{{- end }}
{{- if $value }}

2
deploy/helm/templates/statefulset.yaml
View File

@ -47,7 +47,7 @@ spec:
{{- if .Values.redis.enabled }}
- name: redis-init-container
image: alpine
command: ['sh', '-c', "apk add redis; until redis-cli -h appsmith-redis-master.{{.Release.Namespace}}.svc.cluster.local ping; do echo waiting for redis; sleep 2; done"]
command: ['sh', '-c', "apk add redis; until redis-cli -h {{ include "appsmith.fullname" . }}-redis-master.{{.Release.Namespace}}.svc.cluster.local ping; do echo waiting for redis; sleep 2; done"]
{{- end }}
{{- if .Values.mongodb.enabled }}
- name: mongo-init-container