Merge pull request #132178 from gavinkflam/127990/add-kubelet-metrics-to-system-monitoring-role

Allow system:monitoring role to access kubelet metrics endpoints

plugin/pkg/auth/authorizer/rbac/bootstrappolicy/policy.go

@@ -283,6 +283,9 @@ func ClusterRoles() []rbacv1.ClusterRole {
 			"/livez", "/readyz", "/healthz",
 			"/livez/*", "/readyz/*", "/healthz/*",
 		).RuleOrDie(),
+
+		// Needed for kubelet metrics
+		rbacv1helpers.NewRule("get").Groups(legacyGroup).Resources("nodes/metrics").RuleOrDie(),
 	}
 
 	if utilfeature.DefaultFeatureGate.Enabled(zpagesfeatures.ComponentFlagz) {

plugin/pkg/auth/authorizer/rbac/bootstrappolicy/testdata/cluster-roles-featuregates.yaml

@@ -1015,6 +1015,12 @@ items:
     - /readyz/*
     verbs:
     - get
+  - apiGroups:
+    - ""
+    resources:
+    - nodes/metrics
+    verbs:
+    - get
   - nonResourceURLs:
     - /flagz
     verbs:

plugin/pkg/auth/authorizer/rbac/bootstrappolicy/testdata/cluster-roles.yaml

@@ -923,6 +923,12 @@ items:
     - /readyz/*
     verbs:
     - get
+  - apiGroups:
+    - ""
+    resources:
+    - nodes/metrics
+    verbs:
+    - get
 - apiVersion: rbac.authorization.k8s.io/v1
   kind: ClusterRole
   metadata: