-
Notifications
You must be signed in to change notification settings - Fork 10
Remove dependency on kube-rbac-proxy by adopting controller-runtime's native authn/authz for metrics
#1226
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
Remove dependency on kube-rbac-proxy by adopting controller-runtime's native authn/authz for metrics
#1226
Conversation
8ee3622 to
cbe1faa
Compare
…'s native authn/authz for metrics
cbe1faa to
2e4d763
Compare
aece9f2 to
3e5c218
Compare
5c3d7cf to
752cb0a
Compare
752cb0a to
2b5375c
Compare
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.
Some minor cleanup left overs. We should use this PR to remove all webhook related configuration (even if commented out) to keep the kustomizations clean.
As discussed offline, can we remove all webhook components in a separate PR? This PR is mainly focused on replacing |
…'s native authn/authz for metrics (#1226) * Remove dependency on `kube-rbac-proxy` by adopting controller-runtime's native authn/authz for metrics * Enhance `cert-manager` integration for metrics endpoints
Overview
This PR removes the soon-to-be deprecated
kube-rbac-proxydependency and replaces it with Controller-Runtime's built-in authentication and authorization for securing the metrics endpoint. This simplifies setup, enhances security, and follows the latest Kubebuilder best practices.Key Changes
kube-rbac-proxydependency.cert-managerintegration to secure metrics with TLS encryption.Fixes #1203
Ref:
kubernetes-sigs/kubebuilder#3907
kubernetes-sigs/controller-runtime#2407
kubernetes-sigs/kubebuilder#4400
kubernetes-sigs/kubebuilder#4558
kubernetes-sigs/kubebuilder/docs/reference/metrics (v4.5.0)