Skip to content

Commit c80d308

Browse files
author
gmarek
committed
Scheduler Reduce functions can use PriorityMetadata
1 parent 85fe0f1 commit c80d308

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)