fix: skip waiting for Jobs with TTL in status watcher

Jobs with ttlSecondsAfterFinished set will be automatically deleted
after completion. The kstatus watcher would get stuck waiting for these
Jobs when they become NotFound after the TTL controller deletes them.

This fix skips waiting for Jobs that have TTLSecondsAfterFinished set,
similar to how paused Deployments are skipped. This matches the behavior
from the legacy Helm 3 waiter which ignored deleted hooks.

Fixes #31786

Signed-off-by: mehrdadbn9 <mehrdadbiukian@gmail.com>

pkg/kube/statuswait.go

@@ -33,6 +33,7 @@ import (
 	"github.com/fluxcd/cli-utils/pkg/kstatus/watcher"
 	"github.com/fluxcd/cli-utils/pkg/object"
 	appsv1 "k8s.io/api/apps/v1"
+	batchv1 "k8s.io/api/batch/v1"
 	"k8s.io/apimachinery/pkg/api/meta"
 	"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
 	"k8s.io/client-go/dynamic"
@@ -179,6 +180,10 @@ func (w *statusWaiter) wait(ctx context.Context, resourceList ResourceList, sw w
 			if value.Spec.Paused {
 				continue
 			}
+		case *batchv1.Job:
+			if value.Spec.TTLSecondsAfterFinished != nil {
+				continue
+			}
 		}
 		obj, err := object.RuntimeToObjMeta(resource.Object)
 		if err != nil {

pkg/kube/statuswait_test.go

@@ -123,6 +123,23 @@ status:
     message: "Job has reached the specified backoff limit"
 `
 
+var jobWithTTLManifest = `
+apiVersion: batch/v1
+kind: Job
+metadata:
+  name: job-with-ttl
+  namespace: default
+  generation: 1
+spec:
+  ttlSecondsAfterFinished: 0
+status:
+  succeeded: 1
+  active: 0
+  conditions:
+  - type: Complete
+    status: "True"
+`
+
 var podCompleteManifest = `
 apiVersion: v1
 kind: Pod
@@ -412,6 +429,10 @@ func TestStatusWait(t *testing.T) {
 			name:         "paused deployment passes",
 			objManifests: []string{pausedDeploymentManifest},
 		},
+		{
+			name:         "job with TTL is skipped",
+			objManifests: []string{jobWithTTLManifest},
+		},
 	}
 
 	for _, tt := range tests {
@@ -471,6 +492,10 @@ func TestWaitForJobComplete(t *testing.T) {
 			objManifests:  []string{jobReadyManifest},
 			expectErrStrs: []string{"resource Job/default/ready-not-complete not ready. status: InProgress", "context deadline exceeded"},
 		},
+		{
+			name:         "job with TTL is skipped",
+			objManifests: []string{jobWithTTLManifest},
+		},
 	}
 
 	for _, tt := range tests {
@@ -532,6 +557,10 @@ func TestWatchForReady(t *testing.T) {
 			objManifests:  []string{podCurrentManifest},
 			expectErrStrs: []string{"resource Pod/ns/current-pod not ready. status: InProgress", "context deadline exceeded"},
 		},
+		{
+			name:         "job with TTL is skipped",
+			objManifests: []string{jobWithTTLManifest},
+		},
 	}
 
 	for _, tt := range tests {