Skip to content

Commit 0c54ec3

Browse files
authored
Merge pull request kubernetes#86577 from draveness/feature/remove-total-num-nodes-in-metdata
fix: remove totalNumNodes from priority metadata
2 parents 175ee0a + 5fd6b8f commit 0c54ec3

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

pkg/scheduler/algorithm/priorities/metadata.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,6 @@ type priorityMetadata struct {
6262
podSelector labels.Selector
6363
controllerRef *metav1.OwnerReference
6464
podFirstServiceSelector labels.Selector
65-
totalNumNodes int
6665
podTopologySpreadMap *podTopologySpreadMap
6766
}
6867

@@ -76,11 +75,9 @@ func (pmf *MetadataFactory) PriorityMetadata(
7675
if pod == nil {
7776
return nil
7877
}
79-
totalNumNodes := 0
8078
var allNodes []*schedulernodeinfo.NodeInfo
8179
if sharedLister != nil {
8280
if l, err := sharedLister.NodeInfos().List(); err == nil {
83-
totalNumNodes = len(l)
8481
allNodes = l
8582
}
8683
}
@@ -96,7 +93,6 @@ func (pmf *MetadataFactory) PriorityMetadata(
9693
podSelector: getSelector(pod, pmf.serviceLister, pmf.controllerLister, pmf.replicaSetLister, pmf.statefulSetLister),
9794
controllerRef: metav1.GetControllerOf(pod),
9895
podFirstServiceSelector: getFirstServiceSelector(pod, pmf.serviceLister),
99-
totalNumNodes: totalNumNodes,
10096
podTopologySpreadMap: tpSpreadMap,
10197
}
10298
}

0 commit comments

Comments
 (0)