fix cronjob status.lastSuccessfulTime not populated by a manually tri…

…ggered job (#9881)
kubernetes · Jan 24, 2025 · 38e970c · 38e970c
1 parent 3e47336
commit 38e970c
Showing 1 changed file with 5 additions and 10 deletions.
diff --git a/modules/api/pkg/resource/cronjob/jobs.go b/modules/api/pkg/resource/cronjob/jobs.go
@@ -114,16 +114,11 @@ func TriggerCronJob(client client.Interface,
 
 	jobToCreate := &batch.Job{
 		ObjectMeta: meta.ObjectMeta{
-			Name:        newJobName,
-			Namespace:   namespace,
-			Annotations: annotations,
-			Labels:      labels,
-			OwnerReferences: []meta.OwnerReference{{
-				APIVersion: CronJobAPIVersion,
-				Kind:       CronJobKindName,
-				Name:       cronJob.Name,
-				UID:        cronJob.UID,
-			}},
+			Name:            newJobName,
+			Namespace:       namespace,
+			Annotations:     annotations,
+			Labels:          labels,
+			OwnerReferences: []meta.OwnerReference{*meta.NewControllerRef(cronJob, batch.SchemeGroupVersion.WithKind("CronJob"))},
 		},
 		Spec: cronJob.Spec.JobTemplate.Spec,
 	}