Skip to content

Commit 58902ea

Browse files
author
gscho
committed
Fix bug with pvc and volume mounts.
Signed-off-by: gscho <greg@geldata.com>
1 parent 5bc9cfa commit 58902ea

File tree

3 files changed

+12
-14
lines changed

3 files changed

+12
-14
lines changed

charts/gel-server/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,5 @@ apiVersion: v2
22
name: gel-server
33
description: A Helm chart for running gel-server on Kubernetes
44
type: application
5-
version: 0.2.0
5+
version: 0.2.1
66
appVersion: "7.0"

charts/gel-server/templates/deployment.yaml

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -95,11 +95,13 @@ spec:
9595
- name: {{ $key }}
9696
value: {{ $value | quote }}
9797
{{- end }}
98-
{{- range .Values.extraEnvFromSecrets }}
98+
{{- if .Values.extraEnvFromSecrets }}
9999
envFrom:
100+
{{- range .Values.extraEnvFromSecrets }}
100101
- secretRef:
101102
name: {{ .name }}
102103
{{- end }}
104+
{{- end }}
103105
{{- with .Values.livenessProbe }}
104106
livenessProbe:
105107
{{- toYaml . | nindent 12 }}
@@ -112,7 +114,6 @@ spec:
112114
resources:
113115
{{- toYaml . | nindent 12 }}
114116
{{- end }}
115-
{{- if or .Values.config.security.tls.existingSecret .Values.volumeMounts }}
116117
volumeMounts:
117118
- name: gel-data
118119
mountPath: /var/lib/gel/data
@@ -123,20 +124,18 @@ spec:
123124
{{- end }}
124125

125126
{{- with .Values.volumeMounts }}
126-
{{- toYaml . | nindent 2 }}
127+
{{- toYaml . | nindent 12 }}
127128
{{- end }}
128-
{{- end }}
129-
{{- if or .Values.config.security.tls.existingSecret .Values.volumeMounts }}
130129
volumes:
131130
- name: gel-data
132131
{{- if .Values.persistence.existingClaim }}
133-
persistentVolumeClaim:
134-
claimName: {{ .Values.persistence.existingClaim }}
132+
persistentVolumeClaim:
133+
claimName: {{ .Values.persistence.existingClaim }}
135134
{{- else if .Values.persistence.enabled }}
136-
persistentVolumeClaim:
137-
claimName: {{ include "gel-server.fullname" . }}-data
135+
persistentVolumeClaim:
136+
claimName: {{ include "gel-server.fullname" . }}-data
138137
{{- else }}
139-
emptyDir: {}
138+
emptyDir: {}
140139
{{- end }}
141140
{{- if .Values.config.security.tls.existingSecret }}
142141
- name: tls
@@ -146,7 +145,6 @@ spec:
146145
{{- with .Values.volumes }}
147146
{{- toYaml . | nindent 8 }}
148147
{{- end }}
149-
{{- end }}
150148
{{- with .Values.nodeSelector }}
151149
nodeSelector:
152150
{{- toYaml . | nindent 8 }}

charts/gel-server/values.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,8 @@ config:
3939
# existingSecret: "gel-tls-cert" # name of secret containing tls.crt / tls.key
4040

4141
extraEnv: {}
42-
# GEL_DOCKER_LOG_LEVEL: "enabled"
43-
# GEL_DEBUG_HTTP_INJECT_CORS: 1
42+
# Full list of environment variables: https://docs.geldata.com/reference/running/configuration
43+
# GEL_DOCKER_LOG_LEVEL: "info"
4444

4545
# extraEnvFromSecrets:
4646
# - name: gel-db-creds

0 commit comments

Comments
 (0)