Better error checking in GetDaemonSetPodsForNode
This commit is contained in:
parent
0dca11fa11
commit
e8eb30e75a
|
|
@ -42,8 +42,14 @@ func GetDaemonSetPodsForNode(nodeInfo *schedulernodeinfo.NodeInfo, daemonsets []
|
||||||
|
|
||||||
for _, ds := range daemonsets {
|
for _, ds := range daemonsets {
|
||||||
pod := newPod(ds, nodeInfo.Node().Name)
|
pod := newPod(ds, nodeInfo.Node().Name)
|
||||||
if err := predicateChecker.CheckPredicates(clusterSnapshot, pod, simulator.FakeNodeInfoForNodeName(nodeInfo.Node().Name)); err == nil {
|
err := predicateChecker.CheckPredicates(clusterSnapshot, pod, simulator.FakeNodeInfoForNodeName(nodeInfo.Node().Name))
|
||||||
|
if err == nil {
|
||||||
result = append(result, pod)
|
result = append(result, pod)
|
||||||
|
} else if err.ErrorType() == simulator.NotSchedulablePredicateError {
|
||||||
|
// ok; we are just skipping this daemonset
|
||||||
|
} else {
|
||||||
|
// unexpected error
|
||||||
|
return nil, fmt.Errorf("unexpected error while calling PredicateChecker; %v", err)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return result, nil
|
return result, nil
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue