add missing pod disruption reasons to isPodDead

isPodDead had checks for TerminationByKubelet and DeletionByTaintManager.
It was missing checks for

PreemptionByScheduler
EvictionByEvictionAPI
DeletionByPodGC

ref: https://kubernetes.io/docs/concepts/workloads/pods/disruptions/#pod-disruption-conditions

This PR adds those reasons.

Author: anik120

pkg/controller/registry/reconciler/grpc.go

@@ -542,36 +542,28 @@ func imageChanged(logger *logrus.Entry, updatePod *corev1.Pod, servingPods []*co
 }
 
 func isPodDead(pod *corev1.Pod) bool {
-	for _, check := range []func(*corev1.Pod) bool{
-		isPodDeletedByTaintManager,
-		isPodTerminatedByKubelet,
+	for _, agentReason := range []string{
+		"PreemptionByScheduler",
+		"DeletionByTaintManager",
+		"EvictionByEvictionAPI",
+		"DeletionByPodGC",
+		"TerminationByKubelet",
 	} {
-		if check(pod) {
+		if isPodDeletedByExternalAgent(pod, agentReason) {
 			return true
 		}
 	}
 	return false
 }
 
-func isPodDeletedByTaintManager(pod *corev1.Pod) bool {
+// isPodDeletedByExternalAgent checks if the pod has one of the conditions
+// listed in https://kubernetes.io/docs/concepts/workloads/pods/disruptions/#pod-disruption-conditions
+func isPodDeletedByExternalAgent(pod *corev1.Pod, agentReason string) bool {
 	if pod.DeletionTimestamp == nil {
 		return false
 	}
 	for _, condition := range pod.Status.Conditions {
-		if condition.Type == corev1.DisruptionTarget && condition.Reason == "DeletionByTaintManager" && condition.Status == corev1.ConditionTrue {
-			return true
-		}
-	}
-	return false
-}
-
-// This reason is set when the Pod was evicted due to resource pressure on the Node
-func isPodTerminatedByKubelet(pod *corev1.Pod) bool {
-	if pod.DeletionTimestamp == nil {
-		return false
-	}
-	for _, condition := range pod.Status.Conditions {
-		if condition.Type == corev1.DisruptionTarget && condition.Reason == "TerminationByKubelet" && condition.Status == corev1.ConditionTrue {
+		if condition.Type == corev1.DisruptionTarget && condition.Reason == agentReason && condition.Status == corev1.ConditionTrue {
 			return true
 		}
 	}