Skip to content

Commit

Permalink
Merge pull request #363 from jenting/issue-340
Browse files Browse the repository at this point in the history
[velero] Support customize overwrite the cluster administrator (ClusterRole) name
  • Loading branch information
ywk253100 authored Mar 28, 2022
2 parents 22e3e89 + 601f26a commit 3b9234d
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion charts/velero/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ appVersion: 1.8.1
kubeVersion: ">=1.16.0-0"
description: A Helm chart for velero
name: velero
version: 2.29.2
version: 2.29.3
home: https://github.com/vmware-tanzu/velero
icon: https://cdn-images-1.medium.com/max/1600/1*-9mb3AKnKdcL_QD3CMnthQ.png
sources:
Expand Down
2 changes: 1 addition & 1 deletion charts/velero/templates/clusterrolebinding.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,6 @@ subjects:
name: {{ include "velero.serverServiceAccount" . }}
roleRef:
kind: ClusterRole
name: cluster-admin
name: {{ .Values.rbac.clusterAdministratorName }}
apiGroup: rbac.authorization.k8s.io
{{- end }}
2 changes: 2 additions & 0 deletions charts/velero/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -292,6 +292,8 @@ rbac:
create: true
# Whether to create the cluster role binding to give administrator permissions to Velero
clusterAdministrator: true
# Name of the ClusterRole.
clusterAdministratorName: cluster-admin

# Information about the Kubernetes service account Velero uses.
serviceAccount:
Expand Down

0 comments on commit 3b9234d

Please sign in to comment.