Amir Alavi
Amir Alavi
Just wanted to report that I am able to reproduce this. I added a unit test below which is failing (it's evicting 2 Pods but the expected count is zero):...
@dk-d-n the fix has been merged (but not yet released). Would you be able to test this scenario using the latest on the main branch? I have added a test...
The issue was addressed in 0.25 Please feel free to open if that is not the case @dk-d-n /close
/assign I can create a PR once #834 is reviewed/approved/merged
On second though, `PodLifeTime` is likely not a good candidate for this given that it is included in conditions at some point in time. For example, take the following example:...
/remove-lifecycle stale /assign @damemi this would be a good opportunity for me to create a new plugin and see how that goes.
>At that point, we can assign out the work to transition the remaining strategies to plugins. I'm late to the part but count me in! 🕺🏼
I can take FailedPods and PodLifeTime. Should have PRs soon RemoveFailedPods PR #861
@damemi I would be happy to contribute to this. Any docs highlighting the decision made to-date?
It would be great to have the following: - pods evicted success - pods evicted failed - pods skipped - total pods under consideration Overall and per strategy.