remove unused meta and rename lablance_allocated

Signed-off-by: Zou Nengren <[email protected]>

Author: zouyee

pkg/scheduler/framework/plugins/noderesources/BUILD

@@ -3,7 +3,7 @@ load("@io_bazel_rules_go//go:def.bzl", "go_library", "go_test")
 go_library(
     name = "go_default_library",
     srcs = [
-        "balance_allocated.go",
+        "balanced_allocation.go",
         "least_allocated.go",
         "most_allocated.go",
         "node_resources.go",
@@ -41,7 +41,7 @@ filegroup(
 go_test(
     name = "go_default_test",
     srcs = [
-        "balance_allocated_test.go",
+        "balanced_allocation_test.go",
         "least_allocated_test.go",
         "most_allocated_test.go",
         "node_resources_test.go",

pkg/scheduler/framework/plugins/noderesources/balance_allocated.go renamed to pkg/scheduler/framework/plugins/noderesources/balanced_allocation.go

@@ -49,9 +49,8 @@ func (ba *BalancedAllocation) Score(ctx context.Context, state *framework.CycleS
 	if !exist {
 		return 0, framework.NewStatus(framework.Error, fmt.Sprintf("node %q does not exist in NodeInfoSnapshot", nodeName))
 	}
-	meta := migration.PriorityMetadata(state)
-
-	s, err := priorities.BalancedResourceAllocationMap(pod, meta, nodeInfo)
+	// BalancedResourceAllocationMap does not use priority metadata, hence we pass nil here
+	s, err := priorities.BalancedResourceAllocationMap(pod, nil, nodeInfo)
 	return s.Score, migration.ErrorToFrameworkStatus(err)
 }
 

pkg/scheduler/framework/plugins/noderesources/balance_allocated_test.go renamed to pkg/scheduler/framework/plugins/noderesources/balanced_allocation_test.go

@@ -48,9 +48,8 @@ func (la *LeastAllocated) Score(ctx context.Context, state *framework.CycleState
 	if !exist {
 		return 0, framework.NewStatus(framework.Error, fmt.Sprintf("node %q does not exist in NodeInfoSnapshot", nodeName))
 	}
-	meta := migration.PriorityMetadata(state)
-
-	s, err := priorities.LeastRequestedPriorityMap(pod, meta, nodeInfo)
+	// LeastRequestedPriorityMap does not use priority metadata, hence we pass nil here
+	s, err := priorities.LeastRequestedPriorityMap(pod, nil, nodeInfo)
 	return s.Score, migration.ErrorToFrameworkStatus(err)
 }
 

pkg/scheduler/framework/plugins/noderesources/least_allocated.go

@@ -48,9 +48,8 @@ func (ma *MostAllocated) Score(ctx context.Context, state *framework.CycleState,
 	if !exist {
 		return 0, framework.NewStatus(framework.Error, fmt.Sprintf("node %q does not exist in NodeInfoSnapshot", nodeName))
 	}
-	meta := migration.PriorityMetadata(state)
-
-	s, err := priorities.MostRequestedPriorityMap(pod, meta, nodeInfo)
+	// MostRequestedPriorityMap does not use priority metadata, hence we pass nil here
+	s, err := priorities.MostRequestedPriorityMap(pod, nil, nodeInfo)
 	return s.Score, migration.ErrorToFrameworkStatus(err)
 }