Merge pull request #1591 from dippynark/use-semantic-deep-equal

fix: Retry foghorn LighthouseJob update using original variable instead of DeepCopy variable
jenkins-x · Jan 9, 2024 · f7cb024 · f7cb024
2 parents 867b0b5 + c654951
commit f7cb024
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pkg/foghorn/controller.go b/pkg/foghorn/controller.go
@@ -130,7 +130,7 @@ func (r *LighthouseJobReconciler) Reconcile(ctx context.Context, req ctrl.Reques
 			}
 			return nil
 		}
-		if err := r.retryModifyJob(ctx, req.NamespacedName, jobCopy, f); err != nil {
+		if err := r.retryModifyJob(ctx, req.NamespacedName, &job, f); err != nil {
 			r.logger.Errorf("Failed to update LighthouseJob status: %s", err)
 			return ctrl.Result{}, err
 		}