Skip to content

Commit

Permalink
helm: Add service account annotations for all charts
Browse files Browse the repository at this point in the history
In order to use AWS EKS IAM roles for service account, we must annotate
service account with IAM role. More details can be found here
https://docs.aws.amazon.com/eks/latest/userguide/iam-roles-for-service-accounts-technical-overview.html

Fixes: #11303

Signed-off-by: Tam Mach <sayboras@yahoo.com>
  • Loading branch information
sayboras authored and rolinh committed May 6, 2020
1 parent dd99958 commit f3e39e6
Show file tree
Hide file tree
Showing 8 changed files with 32 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,7 @@ kind: ServiceAccount
metadata:
name: cilium
namespace: {{ .Release.Namespace }}
{{- if .Values.serviceAccount.annotations }}
annotations:
{{ toYaml .Values.serviceAccount.annotations | indent 4 }}
{{- end }}
4 changes: 4 additions & 0 deletions install/kubernetes/cilium/charts/agent/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,10 @@ image: cilium
# update process.
maxUnavailable: 2

# Specifies annotation for service accounts
serviceAccount:
annotations: {}

# Enables monitor sidecar container for specified event types
monitor:
enabled: false
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,9 @@ apiVersion: v1
kind: ServiceAccount
metadata:
namespace: {{ .Release.Namespace }}
name: hubble-ui
name: hubble-ui
{{- if .Values.serviceAccount.annotations }}
annotations:
{{ toYaml .Values.serviceAccount.annotations | indent 4 }}
{{- end }}
{{- end }}
5 changes: 4 additions & 1 deletion install/kubernetes/cilium/charts/hubble-ui/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,10 @@ replicas: 1
# cpu: 100m
# memory: 64Mi
resources: {}


serviceAccount:
annotations: {}

ingress:
enabled: false
annotations: {}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,7 @@ kind: ServiceAccount
metadata:
name: cilium-etcd-sa
namespace: {{ .Release.Namespace }}
{{- if .Values.serviceAccount.annotations }}
annotations:
{{ toYaml .Values.serviceAccount.annotations | indent 4 }}
{{- end }}
3 changes: 3 additions & 0 deletions install/kubernetes/cilium/charts/managed-etcd/values.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,5 @@
image: cilium-etcd-operator
tag: v2.0.7

serviceAccount:
annotations: {}
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,7 @@ kind: ServiceAccount
metadata:
name: cilium-operator
namespace: {{ .Release.Namespace }}
{{- if .Values.serviceAccount.annotations }}
annotations:
{{ toYaml .Values.serviceAccount.annotations | indent 4 }}
{{- end }}
4 changes: 4 additions & 0 deletions install/kubernetes/cilium/charts/operator/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,9 @@ image: operator
# Deprecated: please use synchronizeK8sNodes in install/kubernetes/cilium/values.yaml
synchronizeK8sNodes: true

# Service account annotations
serviceAccount:
annotations: {}

# Specifies the resources for the operator container
resources: {}

0 comments on commit f3e39e6

Please sign in to comment.