Node controller deletePod return true if there are pods pending deletion

This commit is contained in:
derekwaynecarr 2016-08-15 10:21:47 -04:00
parent 5962874414
commit 14a2b261a8

View File

@ -82,7 +82,7 @@ func cleanupOrphanedPods(pods []*api.Pod, nodeStore cache.Store, forcefulDeleteP
}
// deletePods will delete all pods from master running on given node, and return true
// if any pods were deleted.
// if any pods were deleted, or were found pending deletion.
func deletePods(kubeClient clientset.Interface, recorder record.EventRecorder, nodeName, nodeUID string, daemonStore cache.StoreToDaemonSetLister) (bool, error) {
remaining := false
selector := fields.OneTermEqualSelector(api.PodHostField, nodeName)
@ -101,8 +101,9 @@ func deletePods(kubeClient clientset.Interface, recorder record.EventRecorder, n
if pod.Spec.NodeName != nodeName {
continue
}
// if the pod has already been deleted, ignore it
// if the pod has already been marked for deletion, we still return true that there are remaining pods.
if pod.DeletionGracePeriodSeconds != nil {
remaining = true
continue
}
// if the pod is managed by a daemonset, ignore it