Skip to content

Commit

Permalink
Fix cluster secrets access for flyte-core helm charts when used in mu…
Browse files Browse the repository at this point in the history
…lti-cluster setup

Signed-off-by: gdabisias <[email protected]>
  • Loading branch information
gdabisias committed Aug 28, 2023
1 parent 516dd39 commit 1462a8a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
2 changes: 2 additions & 0 deletions charts/flyte-core/templates/admin/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,8 @@ spec:
name: clusters-config-volume
- mountPath: /etc/secrets/
name: admin-secrets
- mountPath: /var/run/credentials
name: cluster-secrets
{{- end }}
- name: generate-secrets
image: "{{ .Values.flyteadmin.image.repository }}:{{ .Values.flyteadmin.image.tag }}"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ spec:
name: config-volume
{{- if gt (len .Values.configmap.clusters.labelClusterMap) 0 }}
- mountPath: /var/run/credentials
name: flyte-admin-secrets
name: cluster-secrets
{{- end }}
serviceAccountName: {{ .Values.cluster_resource_manager.service_account_name }}
{{- if .Values.cluster_resource_manager.resources }}
Expand All @@ -55,9 +55,9 @@ spec:
name: flyte-clusterresourcesync-config
name: config-volume
{{- if gt (len .Values.configmap.clusters.labelClusterMap) 0 }}
- name: flyte-admin-secrets
- name: cluster-secrets
secret:
secretName: flyte-admin-secrets
secretName: cluster-secrets
{{- end }}
{{- if .Values.cluster_resource_manager.config.cluster_resources.standaloneDeployment }}
- name: auth
Expand Down

0 comments on commit 1462a8a

Please sign in to comment.