Skip to content

Commit ab4948b

Browse files
Fix missing envs key (#26)
Co-authored-by: Azat Safin <azat@leetcode.com> Co-authored-by: Azat Safin <azat.safin@gmail.com>
1 parent 212a3c1 commit ab4948b

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

charts/kafka-ui/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,6 @@ apiVersion: v2
22
name: kafka-ui
33
description: A Helm chart for kafka-UI
44
type: application
5-
version: 1.4.7
5+
version: 1.4.8
66
appVersion: v1.0.0
77
icon: https://raw.githubusercontent.com/kafbat/kafka-ui/main/documentation/images/logo_new.png

charts/kafka-ui/templates/deployment.yaml

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,10 @@ spec:
5454
{{- toYaml .Values.securityContext | nindent 12 }}
5555
image: {{ include "kafka-ui.imageName" . }}
5656
imagePullPolicy: {{ .Values.image.pullPolicy }}
57-
{{- if or .Values.env .Values.yamlApplicationConfig .Values.yamlApplicationConfigConfigMap .Values.yamlApplicationConfigSecret }}
57+
{{- if or .Values.env
58+
.Values.yamlApplicationConfig .Values.yamlApplicationConfigConfigMap .Values.yamlApplicationConfigSecret
59+
(.Values.envs).secretMappings (.Values.envs).configMappings
60+
}}
5861
env:
5962
{{- with .Values.env }}
6063
{{- toYaml . | nindent 12 }}
@@ -69,20 +72,20 @@ spec:
6972
value: /kafka-ui/{{ .Values.yamlApplicationConfigSecret.keyName | default "config.yml" }}
7073
{{- end }}
7174
{{- end }}
72-
{{- end }}
73-
{{- range $key, $value := .Values.envs.secretMappings }}
75+
{{- range $key, $value := .Values.envs.secretMappings }}
7476
- name: {{ $key }}
7577
valueFrom:
7678
secretKeyRef:
7779
name: {{ required "Missing required value envs.secretMappings.[].name" $value.name }}
7880
key: {{ required "Missing required value envs.secretMappings.[].keyName" $value.keyName }}
79-
{{- end }}
80-
{{- range $key, $value := .Values.envs.configMappings }}
81+
{{- end }}
82+
{{- range $key, $value := .Values.envs.configMappings }}
8183
- name: {{ $key }}
8284
valueFrom:
8385
configMapKeyRef:
8486
name: {{ required "Missing required value envs.configMappings.[].name" $value.name }}
8587
key: {{ required "Missing required value envs.configMappings.[].keyName" $value.keyName }}
88+
{{- end }}
8689
{{- end }}
8790
{{- if or .Values.existingConfigMap .Values.envs.config .Values.existingSecret .Values.envs.secret }}
8891
envFrom:

0 commit comments

Comments
 (0)