Skip to content

Commit

Permalink
fix serviceaccount mismatch issue for operator. (istio#26756)
Browse files Browse the repository at this point in the history
* fix serviceaccount mismatch issue for operator.

* fix operator tests.
  • Loading branch information
morvencao authored Aug 24, 2020
1 parent 4323b37 commit 7d99a56
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion manifests/charts/istio-operator/templates/clusterrole.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
creationTimestamp: null
name: istio-operator
name: istio-operator{{- if not (eq .Values.revision "") }}-{{ .Values.revision }}{{- end }}
rules:
# istio groups
- apiGroups:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
kind: ClusterRoleBinding
apiVersion: rbac.authorization.k8s.io/v1
metadata:
name: istio-operator
name: istio-operator{{- if not (eq .Values.revision "") }}-{{ .Values.revision }}{{- end }}
subjects:
- kind: ServiceAccount
name: istio-operator
name: istio-operator{{- if not (eq .Values.revision "") }}-{{ .Values.revision }}{{- end }}
namespace: {{.Values.operatorNamespace}}
roleRef:
kind: ClusterRole
name: istio-operator
name: istio-operator{{- if not (eq .Values.revision "") }}-{{ .Values.revision }}{{- end }}
apiGroup: rbac.authorization.k8s.io
---
2 changes: 1 addition & 1 deletion manifests/charts/istio-operator/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ spec:
labels:
name: istio-operator
spec:
serviceAccountName: istio-operator
serviceAccountName: istio-operator{{- if not (eq .Values.revision "") }}-{{ .Values.revision }}{{- end }}
containers:
- name: istio-operator
image: {{.Values.hub}}/operator:{{.Values.tag}}
Expand Down

0 comments on commit 7d99a56

Please sign in to comment.