Skip to content

Commit

Permalink
do not include CA configmap in helm chart, use it by reference (#59)
Browse files Browse the repository at this point in the history
* do not include CA configmap in helm chart, use it by reference

* allow the configmap not to be defined without error
  • Loading branch information
laverya authored Sep 11, 2024
1 parent 85c52dc commit f91e3c4
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 14 deletions.
12 changes: 0 additions & 12 deletions templates/configmap-private-cas.yaml

This file was deleted.

9 changes: 8 additions & 1 deletion templates/kotsadm-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -89,8 +89,10 @@ spec:
{{- end }}
- name: DISABLE_OUTBOUND_CONNECTIONS
value: {{ .Values.isAirgap | quote }}
{{- if .Values.privateCAs.enabled }}
- name: SSL_CERT_DIR
value: /certs
{{- end }}
image: {{ .Values.images.kotsadm }}
imagePullPolicy: IfNotPresent
name: kotsadm
Expand All @@ -117,8 +119,10 @@ spec:
name: backup
- mountPath: /tmp
name: tmp
{{- if .Values.privateCAs.enabled }}
- mountPath: /certs
name: kotsadm-private-cas
{{- end }}
initContainers:
- args:
- plan
Expand Down Expand Up @@ -208,6 +212,9 @@ spec:
name: backup
- emptyDir: {}
name: tmp
{{- if .Values.privateCAs.enabled }}
- configmap:
name: kotsadm-private-cas
name: {{- .Values.privateCAs.configmapName }}
optional: true
name: kotsadm-private-cas
{{- end }}
4 changes: 3 additions & 1 deletion values.yaml.tmpl
Original file line number Diff line number Diff line change
Expand Up @@ -111,4 +111,6 @@ kurlProxy:
enabled: false
targetPort: 8800

privateCAs: {}
privateCAs:
enabled: false
configmapName: ""

0 comments on commit f91e3c4

Please sign in to comment.