Populate DisruptedPods field for non-dry run

This commit is contained in:
Ted Yu 2019-07-05 04:37:27 +08:00 committed by Ted Yu
parent f978c4cab5
commit 2dae73e3d9

View File

@ -206,15 +206,15 @@ func (r *EvictionREST) checkAndDecrement(namespace string, podName string, pdb p
} }
pdb.Status.PodDisruptionsAllowed-- pdb.Status.PodDisruptionsAllowed--
if pdb.Status.DisruptedPods == nil {
pdb.Status.DisruptedPods = make(map[string]metav1.Time)
}
// If this is a dry-run, we don't need to go any further than that. // If this is a dry-run, we don't need to go any further than that.
if dryRun == true { if dryRun == true {
return nil return nil
} }
if pdb.Status.DisruptedPods == nil {
pdb.Status.DisruptedPods = make(map[string]metav1.Time)
}
// Eviction handler needs to inform the PDB controller that it is about to delete a pod // Eviction handler needs to inform the PDB controller that it is about to delete a pod
// so it should not consider it as available in calculations when updating PodDisruptions allowed. // so it should not consider it as available in calculations when updating PodDisruptions allowed.
// If the pod is not deleted within a reasonable time limit PDB controller will assume that it won't // If the pod is not deleted within a reasonable time limit PDB controller will assume that it won't