From a1b8777d42d07e8c4a66d23f325edeaca9c45ed4 Mon Sep 17 00:00:00 2001 From: "Bobby (Babak) Salamat" Date: Mon, 4 Mar 2019 13:14:25 -0800 Subject: [PATCH] Revert "Merge pull request #73934 from bsalamat/num_cpu" This reverts commit cfe4ca8012156aad8e3ee8659fe3f346e74e99b4, reversing changes made to f16035600a4b9abf7e145f0ca2f1d1798c30c9a3. --- pkg/scheduler/algorithm/predicates/metadata.go | 5 ++--- pkg/scheduler/algorithm/priorities/interpod_affinity.go | 3 +-- pkg/scheduler/core/generic_scheduler.go | 7 +++---- 3 files changed, 6 insertions(+), 9 deletions(-) diff --git a/pkg/scheduler/algorithm/predicates/metadata.go b/pkg/scheduler/algorithm/predicates/metadata.go index 8e27d4f721..99baed71ab 100644 --- a/pkg/scheduler/algorithm/predicates/metadata.go +++ b/pkg/scheduler/algorithm/predicates/metadata.go @@ -19,7 +19,6 @@ package predicates import ( "context" "fmt" - "runtime" "sync" "k8s.io/klog" @@ -416,7 +415,7 @@ func getTPMapMatchingExistingAntiAffinity(pod *v1.Pod, nodeInfoMap map[string]*s appendTopologyPairsMaps(existingPodTopologyMaps) } } - workqueue.ParallelizeUntil(context.TODO(), runtime.NumCPU(), len(allNodeNames), processNode) + workqueue.ParallelizeUntil(context.TODO(), 16, len(allNodeNames), processNode) return topologyMaps, firstError } @@ -504,7 +503,7 @@ func getTPMapMatchingIncomingAffinityAntiAffinity(pod *v1.Pod, nodeInfoMap map[s appendResult(node.Name, nodeTopologyPairsAffinityPodsMaps, nodeTopologyPairsAntiAffinityPodsMaps) } } - workqueue.ParallelizeUntil(context.TODO(), runtime.NumCPU(), len(allNodeNames), processNode) + workqueue.ParallelizeUntil(context.TODO(), 16, len(allNodeNames), processNode) return topologyPairsAffinityPodsMaps, topologyPairsAntiAffinityPodsMaps, firstError } diff --git a/pkg/scheduler/algorithm/priorities/interpod_affinity.go b/pkg/scheduler/algorithm/priorities/interpod_affinity.go index 24af001ffa..62540a5941 100644 --- a/pkg/scheduler/algorithm/priorities/interpod_affinity.go +++ b/pkg/scheduler/algorithm/priorities/interpod_affinity.go @@ -18,7 +18,6 @@ package priorities import ( "context" - "runtime" "sync" "k8s.io/api/core/v1" @@ -212,7 +211,7 @@ func (ipa *InterPodAffinity) CalculateInterPodAffinityPriority(pod *v1.Pod, node } } } - workqueue.ParallelizeUntil(context.TODO(), runtime.NumCPU(), len(allNodeNames), processNode) + workqueue.ParallelizeUntil(context.TODO(), 16, len(allNodeNames), processNode) if pm.firstError != nil { return nil, pm.firstError } diff --git a/pkg/scheduler/core/generic_scheduler.go b/pkg/scheduler/core/generic_scheduler.go index 804a3f5c42..bf6f4b78fa 100644 --- a/pkg/scheduler/core/generic_scheduler.go +++ b/pkg/scheduler/core/generic_scheduler.go @@ -20,7 +20,6 @@ import ( "context" "fmt" "math" - "runtime" "sort" "strings" "sync" @@ -490,7 +489,7 @@ func (g *genericScheduler) findNodesThatFit(pod *v1.Pod, nodes []*v1.Node) ([]*v // Stops searching for more nodes once the configured number of feasible nodes // are found. - workqueue.ParallelizeUntil(ctx, runtime.NumCPU(), int(allNodes), checkNode) + workqueue.ParallelizeUntil(ctx, 16, int(allNodes), checkNode) filtered = filtered[:filteredLen] if len(errs) > 0 { @@ -696,7 +695,7 @@ func PrioritizeNodes( } } - workqueue.ParallelizeUntil(context.TODO(), runtime.NumCPU(), len(nodes), func(index int) { + workqueue.ParallelizeUntil(context.TODO(), 16, len(nodes), func(index int) { nodeInfo := nodeNameToInfo[nodes[index].Name] for i := range priorityConfigs { if priorityConfigs[i].Function != nil { @@ -944,7 +943,7 @@ func selectNodesForPreemption(pod *v1.Pod, resultLock.Unlock() } } - workqueue.ParallelizeUntil(context.TODO(), runtime.NumCPU(), len(potentialNodes), checkNode) + workqueue.ParallelizeUntil(context.TODO(), 16, len(potentialNodes), checkNode) return nodeToVictims, nil }