Thor Wu
Thor Wu
Thank you for you feedback. https://github.com/volcano-sh/volcano/blob/f203e26c357cebdacfe528a2269aeafb6655e418/pkg/scheduler/actions/preempt/preempt.go#L228 Let me debug the ssn.TaskOrderFn function. /assign @Thor-wl
Well, after a short discussion, i got your scenario. You want to evict victimsQueue from the lowest priority to highest priority without considering nodes first. Right?
Got it. Sounds reasonable. Let me take a discussion in the community.
> > Got it. Sounds reasonable. Let me take a discussion in the community. > > thank you ！ > it seems add a **NodeOrderFn** may work in the preemption，...
@talcoh2x @snirkop89 I'm also taking a test about this bug. Can you provide your scheduler configuration?
@snirkop89 Hi, Snir. I've taken a look at the bug and preemption was broken indeed. There are several reasons about that. Firstly, the podgroup for job with high priority cannot...
> @Thor-wl Hi, I'm also studying the preemption behavior of Volcano, and found the same problem. It seems that the JobStarvingFn of the gang plugin forbids preemption from a job...
> https://github.com/volcano-sh/volcano/blob/1b2630605b3f6669b5db9a77abd12d08f923e6d3/pkg/scheduler/actions/preempt/preempt.go#L124-L126 > > > Pods with `Preemptable` = false will not be preempted, but it seems that `task.Preemptable` is false by default if we don't set annotation or label....
> That's great to hear. Thank you for the fast response and feedback. No worries. The fix is under discussion.
/cc @huone1