Merge pull request #566 from zionwu/fix-reclaim

k82cn · web-flow · commit 3d01b07e86e8 · 2019-11-27T17:58:05.000+08:00
Fix wrong condition for reclaim action
diff --git a/pkg/scheduler/actions/reclaim/reclaim.go b/pkg/scheduler/actions/reclaim/reclaim.go
@@ -157,7 +157,7 @@ func (alloc *reclaimAction) Execute(ssn *framework.Session) {
 			for _, v := range victims {
 				allRes.Add(v.Resreq)
 			}
-			if allRes.Less(resreq) {
+			if !resreq.LessEqual(allRes) {
 				klog.V(3).Infof("Not enough resource from victims on Node <%s>.", n.Name)
 				continue
 			}

Original file line number	Diff line number	Diff line change
`@@ -157,7 +157,7 @@ func (alloc reclaimAction) Execute(ssn framework.Session) {`
`157`	`157`	`for _, v := range victims {`
`158`	`158`	`allRes.Add(v.Resreq)`
`159`	`159`	`}`
`160`		`- if allRes.Less(resreq) {`
	`160`	`+ if !resreq.LessEqual(allRes) {`
`161`	`161`	`klog.V(3).Infof("Not enough resource from victims on Node <%s>.", n.Name)`
`162`	`162`	`continue`
`163`	`163`	`}`