Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add service account for agent #3788

Merged
merged 1 commit into from
Jun 16, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 17 additions & 0 deletions charts/flyte-core/templates/agent/rbac.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
{{- if .Values.flyteagent.enabled }}
---
{{- if .Values.flyteagent.serviceAccount.create }}
apiVersion: v1
kind: ServiceAccount
metadata:
name: {{ template "flyteagent.name" . }}
namespace: {{ template "flyte.namespace" . }}
labels: {{ include "flyteagent.labels" . | nindent 4 }}
{{- with .Values.flyteagent.serviceAccount.annotations }}
annotations: {{ tpl (toYaml .) $ | nindent 4 }}
{{- end}}
{{- with .Values.flyteagent.serviceAccount.imagePullSecrets }}
imagePullSecrets: {{ tpl (toYaml .) $ | nindent 2 }}
pmahindrakar-oss marked this conversation as resolved.
Show resolved Hide resolved
{{- end }}
{{- end }}
{{- end }}
12 changes: 12 additions & 0 deletions deployment/sandbox/flyte_helm_generated.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,18 @@ metadata:
helm.sh/chart: flyte-v0.1.10
app.kubernetes.io/managed-by: Helm
---
# Source: flyte/charts/flyte/templates/agent/rbac.yaml
apiVersion: v1
kind: ServiceAccount
metadata:
name: flyteagent
namespace: flyte
labels:
app.kubernetes.io/name: flyteagent
app.kubernetes.io/instance: flyte
helm.sh/chart: flyte-v0.1.10
app.kubernetes.io/managed-by: Helm
---
# Source: flyte/charts/flyte/templates/datacatalog/rbac.yaml
apiVersion: v1
kind: ServiceAccount
Expand Down
4 changes: 2 additions & 2 deletions docker/sandbox-bundled/manifests/complete.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -784,7 +784,7 @@ type: Opaque
---
apiVersion: v1
data:
haSharedSecret: ZHdWUkRjZ0c3eG96Q2pmdw==
haSharedSecret: SzlHZ0doUHFQaEswZTBEWQ==
proxyPassword: ""
proxyUsername: ""
kind: Secret
Expand Down Expand Up @@ -1299,7 +1299,7 @@ spec:
metadata:
annotations:
checksum/config: 8f50e768255a87f078ba8b9879a0c174c3e045ffb46ac8723d2eedbe293c8d81
checksum/secret: 38bb3051ec8a9a1187a0b4bce306dd83e331e5ba23a719e905996a1db214d770
checksum/secret: a9517655e17e8f68a20248fbb76c1d65f57c7d2e30d8dd58e86b4c859017f270
labels:
app: docker-registry
release: flyte-sandbox
Expand Down
4 changes: 2 additions & 2 deletions docker/sandbox-bundled/manifests/dev.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -499,7 +499,7 @@ metadata:
---
apiVersion: v1
data:
haSharedSecret: RVlCdFpwQTI4SlVZYllzSg==
haSharedSecret: TmJ6QldYSHBnaFBJaEQ3NQ==
proxyPassword: ""
proxyUsername: ""
kind: Secret
Expand Down Expand Up @@ -875,7 +875,7 @@ spec:
metadata:
annotations:
checksum/config: 8f50e768255a87f078ba8b9879a0c174c3e045ffb46ac8723d2eedbe293c8d81
checksum/secret: 06215b00158e7a4db172bc01358fdc3846e476f94d0e653acc9aefedd4a765fc
checksum/secret: 0ea5a0f5db0680e8f00da24466ac135454104a626fbc465af8b2a069830ecdb0
labels:
app: docker-registry
release: flyte-sandbox
Expand Down