chore: remove deprecated value coder.tls.secretName

deansheather · deansheather · commit 502a4a45a12c · 2022-10-19T20:59:53.000Z
diff --git a/helm/templates/NOTES.txt b/helm/templates/NOTES.txt
@@ -1,8 +1,6 @@
-{{- if .Values.coder.tls.secretName }}
-
-WARN: coder.tls.secretName is deprecated and will be removed in a future
-      release. Please use coder.tls.secretNames instead.
-{{- end }}
+{{/*
+Deprecation notices:
+*/}}
 
 Enjoy Coder! Please create an issue at https://github.com/coder/coder if you run
 into any problems! :)
diff --git a/helm/templates/_helpers.tpl b/helm/templates/_helpers.tpl
@@ -46,7 +46,7 @@ Coder Docker image URI
 Coder listen port (must be > 1024)
 */}}
 {{- define "coder.port" }}
-{{- if or .Values.coder.tls.secretNames .Values.coder.tls.secretName -}}
+{{- if .Values.coder.tls.secretNames -}}
 8443
 {{- else -}}
 8080
@@ -57,7 +57,7 @@ Coder listen port (must be > 1024)
 Coder service port
 */}}
 {{- define "coder.servicePort" }}
-{{- if or .Values.coder.tls.secretNames .Values.coder.tls.secretName -}}
+{{- if .Values.coder.tls.secretNames -}}
 443
 {{- else -}}
 80
@@ -68,7 +68,7 @@ Coder service port
 Port name
 */}}
 {{- define "coder.portName" }}
-{{- if or .Values.coder.tls.secretNames .Values.coder.tls.secretName -}}
+{{- if .Values.coder.tls.secretNames -}}
 https
 {{- else -}}
 http
@@ -86,49 +86,29 @@ Scheme
 Coder volume definitions.
 */}}
 {{- define "coder.volumes" }}
-{{- if or .Values.coder.tls.secretNames .Values.coder.tls.secretName .Values.coder.certs.secretNames }}
+{{- if .Values.coder.tls.secretNames }}
 volumes:
 {{ range $secretName := .Values.coder.tls.secretNames -}}
 - name: "tls-{{ $secretName }}"
   secret:
     secretName: {{ $secretName | quote }}
 {{ end -}}
-{{- if .Values.coder.tls.secretName -}}
-- name: "tls-{{ .Values.coder.tls.secretName }}"
-  secret:
-    secretName: {{ .Values.coder.tls.secretName | quote }}
-{{- end }}
-{{ range $certSecretNames := .Values.coder.certs.secretNames -}}
-- name: {{ $certSecretNames | quote }}
-  secret:
-    secretName: {{ $certSecretNames | quote }}
-{{ end -}}
 {{- else }}
-volumes: {{ if and (not .Values.coder.tls.secretNames) (not .Values.coder.tls.secretName) (not .Values.coder.certs.secretNames) }}[]{{ end }}
+volumes: []
 {{- end }}
 {{- end }}
 
 {{/*
 Coder volume mounts.
 */}}
 {{- define "coder.volumeMounts" }}
-{{- if or .Values.coder.tls.secretNames .Values.coder.tls.secretName .Values.coder.certs.secretNames }}
+{{- if .Values.coder.tls.secretNames }}
 volumeMounts:
 {{ range $secretName := .Values.coder.tls.secretNames -}}
 - name: "tls-{{ $secretName }}"
   mountPath: "/etc/ssl/certs/coder/{{ $secretName }}"
   readOnly: true
 {{ end }}
-{{- if .Values.coder.tls.secretName -}}
-- name: "tls-{{ .Values.coder.tls.secretName }}"
-  mountPath: "/etc/ssl/certs/coder/{{ .Values.coder.tls.secretName }}"
-  readOnly: true
-{{- end }}
-{{ range $certSecretNames := .Values.coder.certs.secretNames -}}
-- name: {{ $certSecretNames | quote }}
-  mountPath: "/etc/ssl/certs/{{ $certSecretNames }}"
-	readOnly: true
-{{ end }}
 {{- else }}
 volumeMounts: []
 {{- end }}
@@ -138,13 +118,13 @@ volumeMounts: []
 Coder TLS environment variables.
 */}}
 {{- define "coder.tlsEnv" }}
-{{- if or .Values.coder.tls.secretNames .Values.coder.tls.secretName }}
+{{- if .Values.coder.tls.secretNames }}
 - name: CODER_TLS_ENABLE
   value: "true"
 - name: CODER_TLS_CERT_FILE
-  value: "{{ range $idx, $secretName := .Values.coder.tls.secretNames -}}{{ if $idx }},{{ end }}/etc/ssl/certs/coder/{{ $secretName }}/tls.crt{{- end }}{{ if .Values.coder.tls.secretName -}}/etc/ssl/certs/coder/{{ .Values.coder.tls.secretName }}/tls.crt{{- end }}"
+  value: "{{ range $idx, $secretName := .Values.coder.tls.secretNames -}}{{ if $idx }},{{ end }}/etc/ssl/certs/coder/{{ $secretName }}/tls.crt{{- end }}"
 - name: CODER_TLS_KEY_FILE
-  value: "{{ range $idx, $secretName := .Values.coder.tls.secretNames -}}{{ if $idx }},{{ end }}/etc/ssl/certs/coder/{{ $secretName }}/tls.key{{- end }}{{ if .Values.coder.tls.secretName -}}/etc/ssl/certs/coder/{{ .Values.coder.tls.secretName }}/tls.key{{- end }}"
+  value: "{{ range $idx, $secretName := .Values.coder.tls.secretNames -}}{{ if $idx }},{{ end }}/etc/ssl/certs/coder/{{ $secretName }}/tls.key{{- end }}"
 {{- end }}
 {{- end }}
 
@@ -172,10 +152,9 @@ included at the top of coder.yaml.
 */}}
 {{- define "coder.verifyDeprecated" }}
 {{/*
-Deprecated value coder.tls.secretName should not be used alongside new value
-coder.tls.secretName.
+Deprecated value coder.tls.secretName must not be used.
 */}}
-{{- if and .Values.coder.tls.secretName .Values.coder.tls.secretNames }}
-{{ fail "You must specify either coder.tls.secretName or coder.tls.secretNames, not both." }}
+{{- if .Values.coder.tls.secretName }}
+{{ fail "coder.tls.secretName is deprecated, use coder.tls.secretNames instead." }}
 {{- end }}
 {{- end }}
diff --git a/helm/values.yaml b/helm/values.yaml
@@ -53,9 +53,6 @@ coder:
     # will be automatically mounted into the pod if specified, and the correct
     # "CODER_TLS_*" environment variables will be set for you.
     secretNames: []
-    # coder.tls.secretName -- Deprecated. Use `coder.tls.secretNames` instead.
-    # This will be removed in a future release.
-    # secretName: ""
 
   # coder.resources -- The resources to request for Coder. These are optional
   # and are not set by default.
