-
Notifications
You must be signed in to change notification settings - Fork 27
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
Reduce RBAC grants for CCM #109
Comments
It cannot. We give the user the option to use the configmap in any namespace. Sure, most use it in For the rest, open a PR for it? |
Make sense 👍
I can't promise that I'll find time to do that. |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /close not-planned |
@k8s-triage-robot: Closing this issue, marking it as "Not Planned". In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/reopen |
/remove-lifecycle rotten |
@cprivitere: Reopened this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
/triage accepted |
Even though CCM is rather highly privileged process on the cluster, for the sake of lease privilege principle and to better describe what CCM actually touches, current RBAC rules could be reduced. For example:
ClusterRole
is used forendpoints
objects. This could be moved toRole
in deployment namespace.ClusterRole
is used forcoordination.k8s.io
objects. This could be moved toRole
in deployment namespace.ClusterRole
is used forconfigmaps
objects to manage MetalLB ConfigMap. This could possibly be reduced to onlymetallb-system
namespace. Do note, that this will also require bindingextension-apiserver-authentication-reader
system role to be able to readextension-apiserver-authentication
ConfigMap
.The text was updated successfully, but these errors were encountered: