Merge pull request #190 from cliveseldon/helm_ambassador_fix

Remove ambassador role and rolebinding from helm script
SeldonIO · Jul 12, 2018 · 2200512 · 2200512
2 parents 75a3555 + 43bed7e
commit 2200512
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/helm-charts/seldon-core/templates/rbac.yaml b/helm-charts/seldon-core/templates/rbac.yaml
@@ -64,6 +64,7 @@ subjects:
   - kind: ServiceAccount
     name: {{ .Values.rbac.service_account.name }}
     namespace: {{ .Release.Namespace }}
+{{- if .Values.ambassador.enabled }}
 ---
 apiVersion: rbac.authorization.k8s.io/v1beta1
 kind: Role
@@ -94,6 +95,7 @@ roleRef:
 subjects:
 - kind: ServiceAccount
   name: {{ .Values.rbac.service_account.name }}
-  namespace: {{ .Release.Namespace }}    
+  namespace: {{ .Release.Namespace }}
+{{- end }}  
 {{- end }}
 {{- end }}