Skip to content

Commit

Permalink
remove client clusterrole/rolbinding since they're not currently used
Browse files Browse the repository at this point in the history
  • Loading branch information
ndhanushkodi authored and Ashwin Venkatesh committed Nov 9, 2021
1 parent 2779acb commit 2299c2a
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 54 deletions.
24 changes: 0 additions & 24 deletions charts/consul/templates/client-dns-clusterrole.yaml

This file was deleted.

22 changes: 0 additions & 22 deletions charts/consul/templates/client-dns-clusterrolebinding.yaml

This file was deleted.

9 changes: 1 addition & 8 deletions charts/consul/templates/client-role.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ metadata:
chart: {{ template "consul.chart" . }}
heritage: {{ .Release.Service }}
release: {{ .Release.Name }}
{{- if (or .Values.dns.enabled .Values.global.acls.manageSystemACLs .Values.global.enablePodSecurityPolicies .Values.global.openshift.enabled) }}
{{- if (or .Values.global.acls.manageSystemACLs .Values.global.enablePodSecurityPolicies .Values.global.openshift.enabled) }}
rules:
{{- if .Values.global.enablePodSecurityPolicies }}
- apiGroups: ["policy"]
Expand All @@ -36,13 +36,6 @@ rules:
verbs:
- use
{{- end}}
{{- if .Values.dns.enabled }}
- apiGroups: [""]
resources:
- services
verbs:
- get
{{- end }}
{{- else}}
rules: []
{{- end }}
Expand Down

0 comments on commit 2299c2a

Please sign in to comment.