Skip to content

Commit 77598e9

Browse files
committed
fix(charts): restore spiffeHelper.image values block with latest as default
Hardcoding :latest in the template was worse — it removed the ability to override the image via Helm values without forking the chart. Restore the templated approach with tag: latest as the default, which is the only reliably-published tag. Signed-off-by: Alan Cha <alan.cha@ibm.com> Assisted-By: Claude (Anthropic AI) <noreply@anthropic.com> Signed-off-by: Alan Cha <Alan.cha1@ibm.com>
1 parent 8cac96c commit 77598e9

2 files changed

Lines changed: 7 additions & 2 deletions

File tree

charts/kagenti-operator/templates/manager/manager.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -193,8 +193,8 @@ spec:
193193
{{- end }}
194194
{{- if and .Values.spiffe .Values.spiffe.enabled .Values.spiffe.operatorAuth .Values.spiffe.operatorAuth.enabled }}
195195
- name: spiffe-helper
196-
image: ghcr.io/kagenti/kagenti-extensions/spiffe-helper:latest
197-
imagePullPolicy: IfNotPresent
196+
image: {{ .Values.spiffe.operatorAuth.spiffeHelper.image.repository | default "ghcr.io/kagenti/kagenti-extensions/spiffe-helper" }}:{{ .Values.spiffe.operatorAuth.spiffeHelper.image.tag | default "latest" }}
197+
imagePullPolicy: {{ .Values.spiffe.operatorAuth.spiffeHelper.image.pullPolicy | default "IfNotPresent" }}
198198
args:
199199
- "-config"
200200
- "/etc/spiffe-helper/config.hcl"

charts/kagenti-operator/values.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -207,6 +207,11 @@ spiffe:
207207
enabled: false
208208
# Path to JWT-SVID file written by spiffe-helper sidecar
209209
jwtSVIDPath: "/opt/jwt_svid.token"
210+
spiffeHelper:
211+
image:
212+
repository: ghcr.io/kagenti/kagenti-extensions/spiffe-helper
213+
tag: latest
214+
pullPolicy: IfNotPresent
210215

211216
# Feature gates — highest-priority layer in the injection precedence chain.
212217
# Set globalEnabled to false to disable ALL sidecar injection (kill switch).

0 commit comments

Comments
 (0)