Skip to content

Commit 9980291

Browse files
author
Kubernetes Submit Queue
authored
Merge pull request kubernetes#35623 from gmarek/meta
Automatic merge from submit-queue Scheduler Reduce functions can use PriorityMetadata
2 parents 2bbc801 + c80d308 commit 9980291

File tree

5 files changed

+5
-5
lines changed

5 files changed

+5
-5
lines changed

plugin/pkg/scheduler/algorithm/priorities/node_affinity.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ func CalculateNodeAffinityPriorityMap(pod *api.Pod, meta interface{}, nodeInfo *
7878
}, nil
7979
}
8080

81-
func CalculateNodeAffinityPriorityReduce(pod *api.Pod, result schedulerapi.HostPriorityList) error {
81+
func CalculateNodeAffinityPriorityReduce(pod *api.Pod, meta interface{}, result schedulerapi.HostPriorityList) error {
8282
var maxCount int
8383
for i := range result {
8484
if result[i].Score > maxCount {

plugin/pkg/scheduler/algorithm/priorities/taint_toleration.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ func ComputeTaintTolerationPriorityMap(pod *api.Pod, meta interface{}, nodeInfo
8888
}, nil
8989
}
9090

91-
func ComputeTaintTolerationPriorityReduce(pod *api.Pod, result schedulerapi.HostPriorityList) error {
91+
func ComputeTaintTolerationPriorityReduce(pod *api.Pod, meta interface{}, result schedulerapi.HostPriorityList) error {
9292
var maxCount int
9393
for i := range result {
9494
if result[i].Score > maxCount {

plugin/pkg/scheduler/algorithm/priorities/test_util.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ func priorityFunction(mapFn algorithm.PriorityMapFunction, reduceFn algorithm.Pr
5151
result = append(result, hostResult)
5252
}
5353
if reduceFn != nil {
54-
if err := reduceFn(pod, result); err != nil {
54+
if err := reduceFn(pod, nil, result); err != nil {
5555
return nil, err
5656
}
5757
}

plugin/pkg/scheduler/algorithm/types.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ type PriorityMapFunction func(pod *api.Pod, meta interface{}, nodeInfo *schedule
3636
// final scores for all nodes.
3737
// TODO: Figure out the exact API of this method.
3838
// TODO: Change interface{} to a specific type.
39-
type PriorityReduceFunction func(pod *api.Pod, result schedulerapi.HostPriorityList) error
39+
type PriorityReduceFunction func(pod *api.Pod, meta interface{}, result schedulerapi.HostPriorityList) error
4040

4141
// MetdataProducer is a function that computes metadata for a given pod.
4242
type MetadataProducer func(pod *api.Pod, nodeNameToInfo map[string]*schedulercache.NodeInfo) interface{}

plugin/pkg/scheduler/generic_scheduler.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -315,7 +315,7 @@ func PrioritizeNodes(
315315
wg.Add(1)
316316
go func(index int, config algorithm.PriorityConfig) {
317317
defer wg.Done()
318-
if err := config.Reduce(pod, results[index]); err != nil {
318+
if err := config.Reduce(pod, meta, results[index]); err != nil {
319319
appendError(err)
320320
}
321321
}(i, priorityConfig)

0 commit comments

Comments
 (0)