Skip to content

Commit 4ef8dc8

Browse files
sionsmithclaude
andcommitted
fix: add chart-testing CI values for helm-charts lint-test
Add ci/ct-values.yaml that overrides the image to busybox so chart-testing can install the chart on Kind without needing the real operator image. Add commandOverride support to deployment template and conditionally skip probes when overridden. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent c6a2b99 commit 4ef8dc8

2 files changed

Lines changed: 18 additions & 0 deletions

File tree

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
# Values override for chart-testing (ct install) in CI.
2+
# Uses a lightweight image since the real operator image may not
3+
# be published yet when chart PRs are tested.
4+
image:
5+
repository: busybox
6+
tag: latest
7+
pullPolicy: IfNotPresent
8+
9+
commandOverride:
10+
- sh
11+
- -c
12+
- "echo 'chart-testing mode' && sleep 30"

deploy/helm/strimzi-backup-operator/templates/deployment.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,10 @@ spec:
3636
- name: operator
3737
image: {{ include "strimzi-backup-operator.image" . }}
3838
imagePullPolicy: {{ .Values.image.pullPolicy }}
39+
{{- with .Values.commandOverride }}
40+
command:
41+
{{- toYaml . | nindent 12 }}
42+
{{- end }}
3943
ports:
4044
- name: metrics
4145
containerPort: 9090
@@ -63,6 +67,7 @@ spec:
6367
{{- end }}
6468
resources:
6569
{{- toYaml .Values.resources | nindent 12 }}
70+
{{- if not .Values.commandOverride }}
6671
livenessProbe:
6772
httpGet:
6873
path: /healthz
@@ -79,6 +84,7 @@ spec:
7984
periodSeconds: 10
8085
timeoutSeconds: 5
8186
failureThreshold: 3
87+
{{- end }}
8288
securityContext:
8389
{{- toYaml .Values.containerSecurityContext | nindent 12 }}
8490
terminationGracePeriodSeconds: 30

0 commit comments

Comments
 (0)