fix: check job completion on re-entry to prevent premature pre-init cleanup

The plan.Phase == TaskPlanComplete check at the top of
reconcilePreInitializing was bypassing the isJobComplete guard on
subsequent reconcile loops, allowing cleanup to proceed while seid was
still syncing to halt-height. Add the Job completion check to the
re-entry path so cleanup only happens after the Job actually finishes.

Author: bdchatham

internal/controller/node/pre_init.go

@@ -24,6 +24,14 @@ func (r *SeiNodeReconciler) reconcilePreInitializing(ctx context.Context, node *
 	plan := node.Status.PreInitPlan
 
 	if plan.Phase == seiv1alpha1.TaskPlanComplete {
+		if len(plan.Tasks) > 0 {
+			job := &batchv1.Job{}
+			key := types.NamespacedName{Name: preInitJobName(node), Namespace: node.Namespace}
+			if err := r.Get(ctx, key, job); err == nil && !isJobComplete(job) && !isJobFailed(job) {
+				log.FromContext(ctx).Info("pre-init plan complete, waiting for seid to reach halt-height", "job", job.Name)
+				return ctrl.Result{RequeueAfter: taskPollInterval}, nil
+			}
+		}
 		if err := r.cleanupPreInit(ctx, node); err != nil {
 			return ctrl.Result{}, fmt.Errorf("cleaning up pre-init resources: %w", err)
 		}