Skip to content

Commit 8e0ecd5

Browse files
authored
Add missing template definition for extraUsers (#60)
* Add missing template definition for extraUsers * fix files block in chi.yaml
1 parent 3354dc7 commit 8e0ecd5

File tree

2 files changed

+15
-5
lines changed

2 files changed

+15
-5
lines changed

charts/clickhouse/templates/_helpers.tpl

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -203,6 +203,15 @@ Extra Config
203203
{{- end -}}
204204
{{- end -}}
205205
{{/*
206+
Extra Users
207+
*/}}
208+
{{- define "clickhouse.extraUsers" -}}
209+
{{- if not (empty .Values.clickhouse.extraUsers) -}}
210+
{{ .Values.clickhouse.extraUsers }}
211+
{{- else -}}
212+
{{- end -}}
213+
{{- end -}}
214+
{{/*
206215
Common labels
207216
*/}}
208217
{{- define "clickhouse.labels" -}}

charts/clickhouse/templates/chi.yaml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -190,16 +190,17 @@ spec:
190190
port: {{ .Values.clickhouse.keeper.port }}
191191
{{- end }}
192192
{{- $extraConfig := tpl (include "clickhouse.extraConfig" . ) . -}}
193-
{{- if not (empty $extraConfig) }}
193+
{{- $extraUsers := tpl (include "clickhouse.extraUsers" . ) . -}}
194+
{{- if not (and (empty $extraConfig) (empty $extraUsers)) }}
194195
files:
196+
{{- if not (empty $extraConfig) }}
195197
config.d/extra_config.xml: |
196198
{{- tpl $extraConfig . | nindent 10 }}
197-
{{- end }}
198-
{{- $extraUsers := tpl (include "clickhouse.extraUsers" . ) . -}}
199-
{{- if not (empty $extraUsers) }}
200-
files:
199+
{{- end }}
200+
{{- if not (empty $extraUsers) }}
201201
users.d/extra_users.xml: |
202202
{{- tpl $extraUsers . | nindent 10 }}
203+
{{- end }}
203204
{{- end }}
204205

205206
{{ include "validate.clickhouse.keeper" . }}

0 commit comments

Comments
 (0)