Skip to content

Commit d27fb14

Browse files
authored
Merge branch 'main' into bump-external-secrets-api-version
2 parents 9080ad9 + c061610 commit d27fb14

11 files changed

+64
-18
lines changed

charts/retool/ci/test-eso-secretref-option.yaml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,5 +10,17 @@ externalSecrets:
1010
name: retool
1111
kind: SecretStore
1212
secretRef:
13+
# secret with just name and path
1314
- name: retool-config
1415
path: mysecret
16+
# secrets with lifecycle options
17+
- name: retool-config-creation-policy
18+
path: mysecret-creation-policy
19+
creationPolicy: Merge
20+
- name: retool-config-deletion-policy
21+
path: mysecret-deletion-policy
22+
deletionPolicy: Merge
23+
# secrets that don't have to exist
24+
- name: retool-config-maybe
25+
path: mysecret-maybe
26+
optional: true

charts/retool/ci/test-install-values.yaml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,9 @@ image:
5252
commandline:
5353
args: []
5454

55-
env: {}
55+
env:
56+
# required env var for backend to start up, but not actually registered
57+
BASE_DOMAIN: "https://helm-ci.retool.dev"
5658

5759
# Optionally specify additional environment variables to be populated from Kubernetes secrets.
5860
# Useful for passing in SCIM_AUTH_TOKEN or other secret environment variables from Kubernetes secrets.

charts/retool/templates/deployment_backend.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -267,6 +267,7 @@ spec:
267267
{{- range .Values.externalSecrets.externalSecretsOperator.secretRef }}
268268
- secretRef:
269269
name: {{ .name }}
270+
optional: {{ .optional | default false }}
270271
{{- end }}
271272
{{- end }}
272273
ports:

charts/retool/templates/deployment_dbconnector.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -189,6 +189,7 @@ spec:
189189
{{- range .Values.externalSecrets.externalSecretsOperator.secretRef }}
190190
- secretRef:
191191
name: {{ .name }}
192+
optional: {{ .optional | default false }}
192193
{{- end }}
193194
{{- end }}
194195
volumeMounts:

charts/retool/templates/deployment_jobs.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -176,6 +176,7 @@ spec:
176176
{{- range .Values.externalSecrets.externalSecretsOperator.secretRef }}
177177
- secretRef:
178178
name: {{ .name }}
179+
optional: {{ .optional | default false }}
179180
{{- end }}
180181
{{- end }}
181182
resources:

charts/retool/templates/deployment_telemetry.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,7 @@ spec:
8787
{{- range .Values.externalSecrets.externalSecretsOperator.secretRef }}
8888
- secretRef:
8989
name: {{ .name }}
90+
optional: {{ .optional | default false }}
9091
{{- end }}
9192
{{- end }}
9293
volumeMounts:

charts/retool/templates/deployment_workflows.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -245,6 +245,7 @@ spec:
245245
{{- range .Values.externalSecrets.externalSecretsOperator.secretRef }}
246246
- secretRef:
247247
name: {{ .name }}
248+
optional: {{ .optional | default false }}
248249
{{- end }}
249250
{{- end }}
250251
ports:

charts/retool/templates/deployment_workflows_worker.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -250,6 +250,7 @@ spec:
250250
{{- range .Values.externalSecrets.externalSecretsOperator.secretRef }}
251251
- secretRef:
252252
name: {{ .name }}
253+
optional: {{ .optional | default false }}
253254
{{- end }}
254255
{{- end }}
255256
ports:

charts/retool/templates/externalsecret.yaml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,13 @@ spec:
3333
name: {{ $.Values.externalSecrets.externalSecretsOperator.secretStoreRef.name }}
3434
kind: {{ $.Values.externalSecrets.externalSecretsOperator.secretStoreRef.kind }}
3535
target:
36-
name: {{ .name }}
37-
creationPolicy: Owner
36+
name: {{ .name | quote }}
37+
# reference: https://external-secrets.io/main/guides/ownership-deletion-policy/
38+
creationPolicy: {{ .creationPolicy | default "Owner" | quote }}
39+
deletionPolicy: {{ .deletionPolicy | default "Retain" | quote }}
3840
dataFrom:
3941
- extract:
40-
key: {{ .path }}
42+
key: {{ .path | quote }}
4143
---
4244
{{- end }}
4345
{{- end }}

charts/retool/values.yaml

Lines changed: 19 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -117,13 +117,25 @@ externalSecrets:
117117
name: aws-secretsmanager
118118
kind: SecretStore # or ClusterSecretStore
119119

120-
# Array of name/path key/value pairs to use for the External Secrets Objects.
121-
secretRef:
122-
[]
123-
# - name: retool-config
124-
# path: global-retool-config
125-
# - name: retool-db
126-
# path: global-retool-db-config
120+
# Array of options to use for the ExternalSecret objects and their
121+
# corresponding secretRef in pod envFrom.
122+
#
123+
# Example with only required fields:
124+
#
125+
# secretRef:
126+
# - name: retool-config
127+
# path: global-retool-config
128+
#
129+
# Example with all optional fields:
130+
#
131+
# secretRef
132+
# - name: extra-secrets
133+
# path: global-extra-secrets
134+
# creationPolicy: Owner
135+
# deletionPolicy: Retain
136+
# optional: true
137+
#
138+
secretRef: []
127139

128140
# When true, uses kubernetes-client CRDs and not external-secrets CRDs
129141
# Defaults to true

0 commit comments

Comments
 (0)