Skip to content
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

rbac toggles broker roles / rolebindings #453

Closed
wants to merge 1 commit into from

Conversation

Mortom123
Copy link
Contributor

This implements the optional creation of roles/rolebinding for the broker. All other PSP-related resources were previously already toggleable, now broker is also toggleable.
However, I'm unsure if this has implications on the functionality of Pulsar as a whole.

Motivation

Some cluster settings prohibit the creation of roles/rolebindings (e.g. Rancher), therefore settings rbac.enabled = False should also prevent the creation of any of these resources.

  • Make sure that the change passes the CI checks.

@lhotari
Copy link
Member

lhotari commented Jan 31, 2024

Some cluster settings prohibit the creation of roles/rolebindings (e.g. Rancher)

Are namespace level roles / rolebinding also prohibited? What is the cluster setting that prevents this? I'm aware that OpenShift and Tanzu are usually handled with namespace level permissions so ClusterRole/ClusterRoleBindings don't work.

@Mortom123
Copy link
Contributor Author

Mortom123 commented Jan 31, 2024

Yes. Our cluster settings prohibit everything role related (cluster or not). They can be created, but only from admin users. I'm unsure what the setting is but the provider is Rancher.

@Mortom123 Mortom123 closed this Feb 8, 2024
@Mortom123 Mortom123 deleted the feature/optional_roles branch February 13, 2024 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants