Skip to content

Commit 8a67be1

Browse files
committed
fix/workaround: add value alternativeClientCa
Signed-off-by: Vanessa Gaube <dev@vanessagaube.de>
1 parent 2a15664 commit 8a67be1

5 files changed

Lines changed: 26 additions & 0 deletions

File tree

valkey/templates/deploy_valkey.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -115,14 +115,22 @@ spec:
115115
startupProbe:
116116
exec:
117117
{{- if .Values.tls.enabled }}
118+
{{- if .Values.tls.alternativeClientCa }}
119+
command: [ "sh", "-c", "valkey-cli --cacert {{ .Values.tls.alternativeClientCa }} --tls ping" ]
120+
{{- else }}
118121
command: [ "sh", "-c", "valkey-cli --cacert /tls/{{ .Values.tls.caPublicKey }} --tls ping" ]
122+
{{- end }}
119123
{{- else }}
120124
command: [ "sh", "-c", "valkey-cli ping" ]
121125
{{- end }}
122126
livenessProbe:
123127
exec:
124128
{{- if .Values.tls.enabled }}
129+
{{- if .Values.tls.alternativeClientCa }}
130+
command: [ "sh", "-c", "valkey-cli --cacert {{ .Values.tls.alternativeClientCa }} --tls ping" ]
131+
{{- else }}
125132
command: [ "sh", "-c", "valkey-cli --cacert /tls/{{ .Values.tls.caPublicKey }} --tls ping" ]
133+
{{- end }}
126134
{{- else }}
127135
command: [ "sh", "-c", "valkey-cli ping" ]
128136
{{- end }}

valkey/templates/statefulset.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -128,14 +128,22 @@ spec:
128128
startupProbe:
129129
exec:
130130
{{- if .Values.tls.enabled }}
131+
{{- if .Values.tls.alternativeClientCa }}
132+
command: [ "sh", "-c", "valkey-cli --cacert {{ .Values.tls.alternativeClientCa }} --tls ping" ]
133+
{{- else }}
131134
command: [ "sh", "-c", "valkey-cli --cacert /tls/{{ .Values.tls.caPublicKey }} --tls ping" ]
135+
{{- end }}
132136
{{- else }}
133137
command: [ "sh", "-c", "valkey-cli ping" ]
134138
{{- end }}
135139
livenessProbe:
136140
exec:
137141
{{- if .Values.tls.enabled }}
142+
{{- if .Values.tls.alternativeClientCa }}
143+
command: [ "sh", "-c", "valkey-cli --cacert {{ .Values.tls.alternativeClientCa }} --tls ping" ]
144+
{{- else }}
138145
command: [ "sh", "-c", "valkey-cli --cacert /tls/{{ .Values.tls.caPublicKey }} --tls ping" ]
146+
{{- end }}
139147
{{- else }}
140148
command: [ "sh", "-c", "valkey-cli ping" ]
141149
{{- end }}

valkey/templates/tests/auth.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,11 @@ spec:
3535
3636
{{- if .Values.tls.enabled }}
3737
# TLS flags
38+
{{- if .Values.tls.alternativeClientCa }}
39+
TLS_FLAGS="--tls --cacert {{ .Values.tls.alternativeClientCa }}"
40+
{{- else }}
3841
TLS_FLAGS="--tls --cacert /tls/{{ .Values.tls.caPublicKey }}"
42+
{{- end }}
3943
{{- else }}
4044
TLS_FLAGS=""
4145
{{- end }}

valkey/values.schema.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -499,6 +499,9 @@
499499
"caPublicKey": {
500500
"type": "string"
501501
},
502+
"alternativeClientCa": {
503+
"type": "string"
504+
},
502505
"dhParamKey": {
503506
"type": "string"
504507
},

valkey/values.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -253,6 +253,9 @@ tls:
253253
serverKey: server.key
254254
# Secret key name containing Certificate Authority public certificate
255255
caPublicKey: ca.crt
256+
# in case the caPublicKey does not work for the client (e.g. valkey-cli), you can set an alternative CA cert as an absolute path here.
257+
# Useful e.g. for trust-manager in combination with cert-manager-generated ACME certs.
258+
alternativeClientCa: ""
256259
# Secret key name containing DH parameters (optional)
257260
dhParamKey: ""
258261
# Require that clients authenticate with a certificate

0 commit comments

Comments
 (0)