Closed nikimanoledaki closed 3 months ago
Hi @nikimanoledaki. Thanks for your PR.
I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
/ok-to-test
@damemi thank you for approving the tests - I fixed the linter issue, would it be possible to run them again please?
/retest pull-descheduler-verify-master
@ingvagabund: The /retest
command does not accept any targets.
The following commands are available to trigger required jobs:
/test pull-descheduler-test-e2e-k8s-master-1-27
/test pull-descheduler-test-e2e-k8s-master-1-28
/test pull-descheduler-test-e2e-k8s-master-1-29
/test pull-descheduler-unit-test-master-master
/test pull-descheduler-verify-build-master
/test pull-descheduler-verify-master
Use /test all
to run all jobs.
I think this can get in before we redesign nodeFit, as @a7i pointed out.
gonna approve and let lgtm's for other maintainers and @logyball
/approve
/test all
Hi @a7i 👋 I would really appreciate your review since we discussed this feature on Slack. Thank you!
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: knelasevero, logyball
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@a7i thanks for taking a look. Is the PR ok to merge? Would appreciate your help with this / next steps, thank you!
The changes look reasonable in general. Thank you for your time and energy. Would you please reduce the number of commits in the PR to make the code changes easier to follow? E.g. making functions/methods private right away, moving code forth and back in a single commit, building the new functions right way in a single commit, etc.
Thank you @ingvagabund! I rewrote the history a bit to remove in-between draft steps & duplications, as well as group unit test commits :) Could you review when you can please? Thank you!
/test all
/cc ingvagabund
@nikimanoledaki I'm working on getting this merged today. We have a CI job that is stuck
/lgtm
/retest-required
thank you for your contribution @nikimanoledaki 🏆
Thank you so much for the reviews and help @a7i @logyball @ingvagabund @knelasevero @damemi! 🎉 🥳
Closes https://github.com/kubernetes-sigs/descheduler/issues/1279 More context: https://github.com/kubernetes-sigs/descheduler/issues/1149
This PR:
pkg/framework/plugins/removepodsviolatinginterpodantiaffinity
->pkg/utils/predicates.go
(for the most part).