scheduler: fix scaling factor 100 for burstable pod (#2242)

Signed-off-by: wangjianyu.wjy <[email protected]> Co-authored-by: wangjianyu.wjy <[email protected]>
koordinator-sh · Nov 4, 2024 · a5f0fbd · a5f0fbd
1 parent eb47585
commit a5f0fbd
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/pkg/scheduler/plugins/loadaware/estimator/default_estimator.go b/pkg/scheduler/plugins/loadaware/estimator/default_estimator.go
@@ -75,7 +75,6 @@ func estimatedUsedByResource(requests, limits corev1.ResourceList, resourceName
 	requestQuantity := requests[resourceName]
 	var quantity resource.Quantity
 	if limitQuantity.Cmp(requestQuantity) > 0 {
-		scalingFactor = 100
 		quantity = limitQuantity
 	} else {
 		quantity = requestQuantity

diff --git a/pkg/scheduler/plugins/loadaware/estimator/default_estimator_test.go b/pkg/scheduler/plugins/loadaware/estimator/default_estimator_test.go
@@ -102,7 +102,7 @@ func TestDefaultEstimatorEstimatePod(t *testing.T) {
 				},
 			},
 			want: map[corev1.ResourceName]int64{
-				corev1.ResourceCPU:    8000,
+				corev1.ResourceCPU:    6800,
 				corev1.ResourceMemory: 6012954214, // 5.6Gi
 			},
 		},

diff --git a/pkg/scheduler/plugins/loadaware/load_aware_test.go b/pkg/scheduler/plugins/loadaware/load_aware_test.go
@@ -2277,7 +2277,7 @@ func TestScore(t *testing.T) {
 					},
 				},
 			},
-			wantScore:  88,
+			wantScore:  90,
 			wantStatus: nil,
 		},
 		{
-Original file line number
+Diff line change
@@ Expand Up / @@ -2277,7 +2277,7 @@ func TestScore(t *testing.T) { @@
     					},
     				},
     			},
-    			wantScore:  88,
+    			wantScore:  90,
     			wantStatus: nil,
     		},
     		{
@@ Expand Down @@