From 8a8717c3a9cfc75f20c6f0c37ec643c425d60dec Mon Sep 17 00:00:00 2001 From: Michal Wozniak Date: Wed, 10 Jul 2024 12:27:56 +0200 Subject: [PATCH] Cleanup TODO comment in the Job controller --- pkg/controller/job/job_controller.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/pkg/controller/job/job_controller.go b/pkg/controller/job/job_controller.go index b34860bef66..49170c2288f 100644 --- a/pkg/controller/job/job_controller.go +++ b/pkg/controller/job/job_controller.go @@ -1878,9 +1878,6 @@ func isPodFailed(p *v1.Pod, job *batch.Job) bool { if feature.DefaultFeatureGate.Enabled(features.PodDisruptionConditions) && feature.DefaultFeatureGate.Enabled(features.JobPodFailurePolicy) && job.Spec.PodFailurePolicy != nil { // When PodDisruptionConditions is enabled, orphan Pods and unschedulable // terminating Pods are marked as Failed. So we only need to check the phase. - // TODO(#113855): Stop limiting this behavior to Jobs with podFailurePolicy. - // For now, we do so to avoid affecting all running Jobs without the - // availability to opt-out into the old behavior. return p.Status.Phase == v1.PodFailed } if p.Status.Phase == v1.PodFailed {