Merge pull request #86577 from draveness/feature/remove-total-num-nodes-in-metdata

fix: remove totalNumNodes from priority metadata
This commit is contained in:
Kubernetes Prow Robot
2019-12-24 08:41:29 -08:00
committed by GitHub

View File

@@ -62,7 +62,6 @@ type priorityMetadata struct {
podSelector labels.Selector
controllerRef *metav1.OwnerReference
podFirstServiceSelector labels.Selector
totalNumNodes int
podTopologySpreadMap *podTopologySpreadMap
}
@@ -76,11 +75,9 @@ func (pmf *MetadataFactory) PriorityMetadata(
if pod == nil {
return nil
}
totalNumNodes := 0
var allNodes []*schedulernodeinfo.NodeInfo
if sharedLister != nil {
if l, err := sharedLister.NodeInfos().List(); err == nil {
totalNumNodes = len(l)
allNodes = l
}
}
@@ -96,7 +93,6 @@ func (pmf *MetadataFactory) PriorityMetadata(
podSelector: getSelector(pod, pmf.serviceLister, pmf.controllerLister, pmf.replicaSetLister, pmf.statefulSetLister),
controllerRef: metav1.GetControllerOf(pod),
podFirstServiceSelector: getFirstServiceSelector(pod, pmf.serviceLister),
totalNumNodes: totalNumNodes,
podTopologySpreadMap: tpSpreadMap,
}
}