ceph / ceph-csi-operator

Kubernetes operator for managing the CephCSI plugins
Apache License 2.0
16 stars 18 forks source link

csiaddons: add new RBAC #166

Closed Madhu-1 closed 2 weeks ago

Madhu-1 commented 2 weeks ago

Describe what this PR does

csiaddons required new RBAC in the next release to create/update the csiaddonsnode object based on the owner deployment/daemonset names of the pods its running with.

Is there anything that requires special attention

Do you have any questions?

No

Is the change backward compatible?

Yes

Are there concerns around backward compatibility?

No

Provide any external context for the change, if any.

RBAC's are required for this new change https://github.com/csi-addons/kubernetes-csi-addons/pull/695

Related issues

Mention any github issues relevant to this PR. Adding below line will help to auto close the issue once the PR is merged.

Fixes: #issue_number

Future concerns

List items that are not part of the PR and do not impact it's functionality, but are work items that can be taken up subsequently.

Checklist:

nb-ohad commented 2 weeks ago

@Madhu-1 Do we already have code that uses this RBAC or is that a preparation for some later code

Madhu-1 commented 2 weeks ago

code that uses this RBAC or is that a preparation for some later cod

This RBAC is used by the sidecar container (which is going to be in next release)/ PR is already merged in csi-addons repo.