Skip to content

Commit 58a1436

Browse files
committed
[helm] Optional fix secrets
1 parent 72ff122 commit 58a1436

8 files changed

+38
-21
lines changed

chart/templates/registry-facade-daemonset.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ spec:
108108
name: {{ template "gitpod.comp.configMap" $this }}
109109
- name: ws-manager-client-tls-certs
110110
secret:
111-
secretName: ws-manager-client-tls
111+
secretName: {{ .Values.components.wsManager.tls.client.secretName }}
112112
{{- if $comp.handover.enabled }}
113113
- name: handover
114114
hostPath:

chart/templates/server-deployment.yaml

Lines changed: 19 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -5,19 +5,24 @@
55
{{- $ := .root -}}
66
{{- $comp := .comp -}}
77
manager:
8-
- name: "{{ template "gitpod.installation.shortname" . }}"
9-
url: "ws-manager:8080"
10-
state: "available"
11-
maxScore: 100
12-
score: 50
13-
govern: true
14-
tls:
15-
ca: /ws-manager-client-tls-certs/ca.crt
16-
crt: /ws-manager-client-tls-certs/tls.crt
17-
key: /ws-manager-client-tls-certs/tls.key
18-
{{ if $comp.wsman -}}
19-
{{ $comp.wsman | toYaml | indent 2 }}
20-
{{- end -}}
8+
- name: "{{ template "gitpod.installation.shortname" . }}"
9+
url: "ws-manager:8080"
10+
state: "available"
11+
maxScore: 100
12+
score: 50
13+
govern: true
14+
tls:
15+
ca: /ws-manager-client-tls-certs/ca.crt
16+
crt: /ws-manager-client-tls-certs/tls.crt
17+
key: /ws-manager-client-tls-certs/tls.key
18+
{{- range $_, $wsman := $comp.wsman }}
19+
{{ "- " -}}
20+
{{ $wsman | toYaml | indent 2 | trim }}
21+
tls:
22+
ca: /ws-manager-client-tls-certs/ca.crt
23+
crt: /ws-manager-client-tls-certs/tls.crt
24+
key: /ws-manager-client-tls-certs/tls.key
25+
{{- end }}
2126
{{- end -}}
2227

2328
{{ define "rate-limiter-config" -}}
@@ -209,7 +214,7 @@ spec:
209214
volumes:
210215
- name: ws-manager-client-tls-certs
211216
secret:
212-
secretName: ws-manager-client-tls
217+
secretName: {{ .Values.components.wsManager.tls.client.secretName }}
213218
{{- if $comp.github.app }}
214219
- name: github-app-cert-secret
215220
secret:

chart/templates/ws-manager-bridge-deployment.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,6 @@ spec:
7373
name: {{ template "gitpod.comp.configMap" $this }}
7474
- name: ws-manager-client-tls-certs
7575
secret:
76-
secretName: ws-manager-client-tls
76+
secretName: {{ .Values.components.wsManager.tls.client.secretName }}
7777
{{ toYaml .Values.defaults | indent 6 }}
7878
{{ end }}

chart/templates/ws-manager-deployment.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ spec:
5353
secretName: ws-daemon-tls
5454
- name: tls-certs
5555
secret:
56-
secretName: ws-manager-tls
56+
secretName: {{ $comp.tls.server.secretName }}
5757
- name: workspace-template
5858
configMap:
5959
name: workspace-template

chart/templates/ws-manager-tlssecret.yaml

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,14 @@
66
{{- if not $comp.disabled -}}
77
{{- $altNames := list ( printf "%s.%s" (include "gitpod.fullname" .) .Release.Namespace ) ( printf "%s.%s.svc" "ws-manager" .Release.Namespace ) "ws-manager" "ws-manager-dev" -}}
88
{{- $ca := genCA "wsmanager-ca" 365 -}}
9+
10+
{{- $server := $comp.tls.server }}
11+
{{- if not $server.crtFile }}
912
{{- $cert := genSignedCert (include "gitpod.fullname" . ) nil $altNames 365 $ca -}}
1013
apiVersion: v1
1114
kind: Secret
1215
metadata:
13-
name: ws-manager-tls
16+
name: "{{ $server.secretName }}"
1417
labels:
1518
app: {{ template "gitpod.fullname" $ }}
1619
chart: "{{ .Chart.Name }}-{{ .Chart.Version }}"
@@ -22,15 +25,18 @@ data:
2225
ca.crt: {{ $ca.Cert | b64enc }}
2326
tls.crt: {{ $cert.Cert | b64enc }}
2427
tls.key: {{ $cert.Key | b64enc }}
28+
{{- end }}
2529

2630
---
2731

32+
{{- $client := $comp.tls.client }}
33+
{{- if not $client.crtFile }}
2834
{{- $altNames := list "registry-facade" "server" "ws-manager-bridge" "ws-scheduler" "ws-proxy" "ws-manager" -}}
2935
{{- $cert := genSignedCert (include "gitpod.fullname" . ) nil $altNames 365 $ca -}}
3036
apiVersion: v1
3137
kind: Secret
3238
metadata:
33-
name: ws-manager-client-tls
39+
name: "{{ $client.secretName }}"
3440
labels:
3541
app: {{ template "gitpod.fullname" $ }}
3642
chart: "{{ .Chart.Name }}-{{ .Chart.Version }}"
@@ -42,4 +48,5 @@ data:
4248
ca.crt: {{ $ca.Cert | b64enc }}
4349
tls.crt: {{ $cert.Cert | b64enc }}
4450
tls.key: {{ $cert.Key | b64enc }}
51+
{{- end }}
4552
{{- end -}}

chart/templates/ws-proxy-deployment.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ spec:
4949
name: {{ template "gitpod.comp.configMap" $this }}
5050
- name: ws-manager-client-tls-certs
5151
secret:
52-
secretName: ws-manager-client-tls
52+
secretName: {{ .Values.components.wsManager.tls.client.secretName }}
5353
{{- if $.Values.certificatesSecret.secretName }}
5454
- name: config-certificates
5555
secret:

chart/templates/ws-scheduler-deployment.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ spec:
4949
name: {{ template "gitpod.comp.configMap" $this }}
5050
- name: ws-manager-client-tls-certs
5151
secret:
52-
secretName: ws-manager-client-tls
52+
secretName: {{ .Values.components.wsManager.tls.client.secretName }}
5353
containers:
5454
- name: scheduler
5555
args: ["run", "-v", "--config", "/config/config.json"]

chart/values.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -356,6 +356,11 @@ components:
356356
resources:
357357
cpu: 100m
358358
memory: 32Mi
359+
tls:
360+
server:
361+
secretName: ws-manager-tls
362+
client:
363+
secretName: ws-manager-client-tls
359364
ports:
360365
rpc:
361366
expose: true

0 commit comments

Comments
 (0)