fix: avoid null ptr of timeouts (#233)

* fix: avoid null ptr of timeouts

* refactor

Author: Haivilo

controllers/resources/bentorequest_controller.go

@@ -1874,10 +1874,12 @@ func (r *BentoRequestReconciler) generateModelSeederJob(ctx context.Context, opt
 					},
 				},
 			},
-			Template:              *podTemplateSpec,
-			ActiveDeadlineSeconds: ptr.To(int64(opt.BentoRequest.Spec.ImageBuildTimeout.Seconds())),
+			Template: *podTemplateSpec,
 		},
 	}
+	if opt.BentoRequest.Spec.ImageBuildTimeout != nil {
+		job.Spec.ActiveDeadlineSeconds = ptr.To(int64(opt.BentoRequest.Spec.ImageBuildTimeout.Seconds()))
+	}
 	err = ctrl.SetControllerReference(opt.BentoRequest, job, r.Scheme)
 	if err != nil {
 		err = errors.Wrapf(err, "set controller reference for job %s", job.Name)
@@ -2295,10 +2297,12 @@ func (r *BentoRequestReconciler) generateImageBuilderJob(ctx context.Context, op
 					},
 				},
 			},
-			Template:              *podTemplateSpec,
-			ActiveDeadlineSeconds: ptr.To(int64(opt.BentoRequest.Spec.ImageBuildTimeout.Seconds())),
+			Template: *podTemplateSpec,
 		},
 	}
+	if opt.BentoRequest.Spec.ImageBuildTimeout != nil {
+		job.Spec.ActiveDeadlineSeconds = ptr.To(int64(opt.BentoRequest.Spec.ImageBuildTimeout.Seconds()))
+	}
 	err = ctrl.SetControllerReference(opt.BentoRequest, job, r.Scheme)
 	if err != nil {
 		err = errors.Wrapf(err, "set controller reference for job %s", job.Name)