Remove all from RabbitmqCluster CRD categories to avoid 'kubectl get all' RBAC noise in cluster-wide installs #2102
XavierFigari
started this conversation in
Ideas
Replies: 1 comment
-
|
Hello, IIRC, we did so for the Topology Operator (removing the |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Hi,
I’d like to propose an enhancement regarding the
RabbitmqClusterCRD categories.Today the CRD includes the
allcategory (inspec.names.categories, e.g.["all", "rabbitmq"]), which makesRabbitmqClusterresources show up inkubectl get all.In a cluster-wide installation (operator installed once for the whole cluster / multiple namespaces), this has an undesirable side-effect in multi-tenant environments:
kubectl get allin their namespace to get a quick overview of standard resources (Pods, Services, Deployments, etc.).RabbitmqClusterobjects, and often do not have RBAC permissions tolist/watchthis CRD (intentionally).kubectl get allbecomes noisy and can return “forbidden” errors for users who otherwise have legitimate access to their namespace resources.Proposal
Please consider removing
allfrom the CRD categories and keeping only a more specific category such asrabbitmq(so users can still discover it viakubectl get rabbitmq/kubectl get rabbitmqclusterswhen needed). This is the common approach for many operators, such as MariaDB.Benefits
kubectl get all)If you think keeping
allis important for some workflows, I’d be happy to discuss alternatives (e.g., documenting why it’s inall, or making it optional in installation manifests if that’s feasible).Thanks!
Beta Was this translation helpful? Give feedback.
All reactions