kubernetes
/
autoscaler
mirror of https://github.com/kubernetes/autoscaler.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Move node-exitence check to top of checkPredicates method

This commit is contained in:
Łukasz Osipiuk 2020-01-30 12:07:55 +01:00
parent 6ed2636f10
commit a8a5ef71ae
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
cluster-autoscaler/simulator/scheduler_based_predicates_checker.go
View File

@ -125,7 +125,7 @@ func NewSchedulerBasedPredicateChecker(kubeClient kube_client.Interface, stop <-
return checker, nil
}
// FitsAnyNode checks if the given pod can be place on any of the given nodes.
// FitsAnyNode checks if the given pod can be placed on any of the given nodes.
func (p *SchedulerBasedPredicateChecker) FitsAnyNode(clusterSnapshot ClusterSnapshot, pod *apiv1.Pod, nodeInfos map[string]*scheduler_nodeinfo.NodeInfo) (string, error) {
if clusterSnapshot != nil {
if nodeInfos != nil {
@ -252,6 +252,12 @@ func (p *SchedulerBasedPredicateChecker) checkPredicatesDeprecated(pod *apiv1.Po
}
func (p *SchedulerBasedPredicateChecker) checkPredicates(clusterSnapshot ClusterSnapshot, pod *apiv1.Pod, nodeName string) *PredicateError {
nodeInfo, err := clusterSnapshot.NodeInfos().Get(nodeName)
if err != nil {
errorMessage := fmt.Sprintf("Error obtaining NodeInfo for name %s; %v", nodeName, err)
return NewPredicateError(InternalPredicateError, "", errorMessage, nil, emptyString)
}
p.delegatingSharedLister.UpdateDelegate(clusterSnapshot)
defer p.delegatingSharedLister.ResetDelegate()
@ -266,12 +272,6 @@ func (p *SchedulerBasedPredicateChecker) checkPredicates(clusterSnapshot Cluster
emptyString)
}
nodeInfo, err := clusterSnapshot.NodeInfos().Get(nodeName)
if err != nil {
errorMessage := fmt.Sprintf("Error obtaining NodeInfo for name %s; %v", nodeName, err)
return NewPredicateError(InternalPredicateError, "", errorMessage, nil, emptyString)
}
filterStatuses := p.framework.RunFilterPlugins(context.TODO(), state, pod, nodeInfo)
for filterName, filterStatus := range filterStatuses {
if !filterStatus.IsSuccess() {