-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add RBAC rules for CSI csidrivers CRD #19
Add RBAC rules for CSI csidrivers CRD #19
Conversation
/assign @msau42 |
deploy/kubernetes/rbac.yaml
Outdated
@@ -24,6 +24,9 @@ rules: | |||
- apiGroups: [""] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the events and nodes rules are not needed. Also, can you change the name of the cluster role to something like "cluster-driver-registrar-role"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated PR. Removed nodes and events RBAC rules and renamed clusterrole.
Fixes #3 |
Thanks so much! This lgtm. Please squash |
ab1a651
to
ff95ba0
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: msau42, sarjeet2013 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Add the RBAC rules for csidrivers crd which was missing from rbac.yaml
Tested without the rules and get errors in creating csidrivers CRD objects.
E0109 18:08:45.418972 1 k8s_register.go:118] Failed to create CSIDriver object: csidrivers.csi.storage.k8s.io is forbidden: User "system:serviceaccount:mapr-csi:csi-provisioner-sa" cannot create resource "csidrivers" in API group "csi.storage.k8s.io" at the cluster scope
With fix, Able to get the correct CRD object being created for csidrivers.