Do not requeue PlanExecution reconciliation if Instance does not exis…

…t. (#657)

* Do not requeue PlanExecution reconciliation if Instance does not exist.

If an Instance does not exist, the PlanExecution controller returns an error when its PlanExecution is reconciled. Because the PlanExecution controller returns an error if the Instance does not exist and controller-runtime will automatically requeue an object for reconciliation if an error is returned, we essentially enter an infinite loop reconciling deleted Instances.

This modifies the method to only log that the Instance does not exist and not return an error so that it will not be requeued.

This cuts test run time from ~80 seconds to ~20 and fixes the flakiness with the create-operator-in-operator test.

* Only skip requeuing for IsNotFound errors.

pkg/controller/planexecution/planexecution_controller.go

@@ -220,6 +220,11 @@ func (r *ReconcilePlanExecution) Reconcile(request reconcile.Request) (reconcile
 			planExecution.Spec.Instance.Name,
 			planExecution.Spec.Instance.Namespace,
 			err)
+
+		if errors.IsNotFound(err) {
+			return reconcile.Result{}, nil
+		}
+
 		return reconcile.Result{}, err
 	}