Fix bug where scan was marked as done even though one of multiple persistence providers failed

Author: J12934

operator/controllers/execution/scan_controller.go

@@ -662,13 +662,23 @@ func (r *ScanReconciler) startPersistenceProvider(scan *executionv1.Scan) error
 }
 
 func allJobsCompleted(jobs *batch.JobList) jobCompletionType {
+	hasFailed := false
+	hasCompleted := true
+
 	for _, job := range jobs.Items {
 		if job.Status.Failed > 0 {
-			return failed
+			hasFailed = true
 		} else if job.Status.Succeeded == 0 {
-			return incomplete
+			hasCompleted = false
 		}
 	}
+
+	if hasFailed {
+		return failed
+	}
+	if hasCompleted {
+		return completed
+	}
 	return incomplete
 }