Skip to content

Commit a6e0ebf

Browse files
authored
Merge pull request #1620 from googs1025/automated-cherry-pick-of-#1599-upstream-release-1.32
Automated cherry pick of #1599: charts: add needed perms if metricsCollector is enabled
2 parents 4c4c31f + 0dd9328 commit a6e0ebf

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

Diff for: charts/descheduler/templates/clusterrole.yaml

+5
Original file line numberDiff line numberDiff line change
@@ -36,4 +36,9 @@ rules:
3636
resourceNames: ["{{ .Values.leaderElection.resourceName | default "descheduler" }}"]
3737
verbs: ["get", "patch", "delete"]
3838
{{- end }}
39+
{{- if and .Values.deschedulerPolicy .Values.deschedulerPolicy.metricsCollector .Values.deschedulerPolicy.metricsCollector.enabled }}
40+
- apiGroups: ["metrics.k8s.io"]
41+
resources: ["pods", "nodes"]
42+
verbs: ["get", "list"]
43+
{{- end }}
3944
{{- end -}}

Diff for: charts/descheduler/values.yaml

+2
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,8 @@ deschedulerPolicy:
9696
# nodeSelector: "key1=value1,key2=value2"
9797
# maxNoOfPodsToEvictPerNode: 10
9898
# maxNoOfPodsToEvictPerNamespace: 10
99+
# metricsCollector:
100+
# enabled: true
99101
# ignorePvcPods: true
100102
# evictLocalStoragePods: true
101103
# evictDaemonSetPods: true

0 commit comments

Comments
 (0)