Fix the issue Iluvatar device scheduling policy binpack and spread are reversed

qiangwei1983 · qiangwei1983 · commit 0b81f91c09d3 · 2026-01-19T11:06:10.000+08:00
Signed-off-by: 魏强 &lt;qiang.wei@iluvatar.com&gt;
diff --git a/pkg/device/iluvatar/device.go b/pkg/device/iluvatar/device.go
@@ -283,7 +283,7 @@ func (ilu *IluvatarDevices) Fit(devices []*device.DeviceUsage, request device.Co
 	var tmpDevs map[string]device.ContainerDevices
 	tmpDevs = make(map[string]device.ContainerDevices)
 	reason := make(map[string]int)
-	for i := range devices {
+	for i := len(devices) - 1; i >= 0; i-- {
 		dev := devices[i]
 		klog.V(4).InfoS("scoring pod", "pod", klog.KObj(pod), "device", dev.ID, "Memreq", k.Memreq, "MemPercentagereq", k.MemPercentagereq, "Coresreq", k.Coresreq, "Nums", k.Nums, "device index", i)
 
diff --git a/pkg/device/iluvatar/device_test.go b/pkg/device/iluvatar/device_test.go
@@ -556,7 +556,7 @@ func Test_Fit(t *testing.T) {
 			annos:      map[string]string{},
 			wantOK:     true,
 			wantLen:    1,
-			wantDevIDs: []string{"dev-0"},
+			wantDevIDs: []string{"dev-1"},
 		},
 		{
 			name: "fit fail: memory not enough",

Original file line number	Diff line number	Diff line change
`@@ -556,7 +556,7 @@ func Test_Fit(t *testing.T) {`
`556`	`556`	`annos: map[string]string{},`
`557`	`557`	`wantOK: true,`
`558`	`558`	`wantLen: 1,`
`559`		`- wantDevIDs: []string{"dev-0"},`
	`559`	`+ wantDevIDs: []string{"dev-1"},`
`560`	`560`	`},`
`561`	`561`	`{`
`562`	`562`	`name: "fit fail: memory not enough",`