-
Notifications
You must be signed in to change notification settings - Fork 668
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
Release 1.28.1 #1285
Release 1.28.1 #1285
Conversation
…yPolicy` constraints (kubernetes-sigs#1218) * Add handling for node eligibility * Make tests buildable * Update topologyspreadconstraint.go * Updated test cases failing * squashed changes for test case addition corrected function name refactored duplicate TopoContraint check logic Added more test cases for testing node eligibility scenario Added 5 test cases for testing scenarios related to node eligibility * topologySpreadConstraints e2e: `nodeTaintsPolicy` and `nodeAffinityPolicy` constraints --------- Co-authored-by: Marc Power <marcpow@microsoft.com> Co-authored-by: nitindagar0 <81955199+nitindagar0@users.noreply.github.com>
Signed-off-by: Amir Alavi <amiralavi7@gmail.com>
fix: structured err logs in eviction
Make it match both the deployment parallel https://github.com/kubernetes-sigs/descheduler/blob/33e9a52385394a3f4b2cb55bdf717fe52f1a852f/charts/descheduler/templates/cronjob.yaml#L71 As well as the documented arg https://github.com/kubernetes-sigs/descheduler/blob/master/docs/user-guide.md#balance-cluster-by-pod-age
helm: update cronjob args argument to avoid unmarshal error
fix: golangci config
…pace-docs docs: fix evictableNamespaces example
update image references to point to v0.28.0
Pods that don't pass the nodeFit condition currently log an unsuppressable error message to logs. This changes the log level to info as it's a normal operating condition. Signed-off-by: Antoine Deschênes <antoine.deschenes@linux.com>
helm: support of `timeZone` for CronJob
* Update values.yaml * Update deployment.yaml * Update cronjob.yaml * fix indentation issue in helm chart
Bump dependencies to address CVE-2023-44487
…-fixes CVE 2023 44487 fixes
Signed-off-by: Amir Alavi <amiralavi7@gmail.com>
Signed-off-by: Amir Alavi <amiralavi7@gmail.com>
…itialization Signed-off-by: Amir Alavi <amiralavi7@gmail.com>
Signed-off-by: Amir Alavi <amiralavi7@gmail.com>
topologyspreadconstraint: support `matchLabelKeys`
thanks @a7i , when you finish an squash the commits please let me know |
Hi @JaneLiuL we do not squash cherry-picks / release branches |
thanks, @a7i , when you finish please let me know~~ |
Thanks@JaneLiuL it is ready for review |
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.
/lgtm
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: knelasevero The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Step 1 in the release step for v0.28.1
https://github.com/kubernetes-sigs/descheduler/blob/master/docs/release-guide.md#flowchart