Update pkg/reconciler/managed/reconciler.go

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

Author: jwefers

pkg/reconciler/managed/reconciler.go

@@ -1471,17 +1471,16 @@ func (r *Reconciler) Reconcile(ctx context.Context, req reconcile.Request) (resu
 	if !policy.ShouldUpdate() {
 		reconcileAfter := r.pollIntervalHook(managed, r.pollInterval)
 		log.Debug("Skipping update due to managementPolicies. Reconciliation succeeded", "requeue-after", time.Now().Add(reconcileAfter))
-
 		// since there is a diff between spec and upstream, we want to inform the user
 		// about the diff, but that we will not update it - indicating that we WOULD perform
 		// changes if we could. This is helpful in migration scenarios to crossplane where
 		// the diff between actual and desired must be analyzed for potential impacts first,
 		// before giving Crossplane control over the resource.
-		status.MarkConditions(
-			xpv1.ReconcileForbidden().
-				WithMessage(fmt.Sprintf(
-					"External resource differs from desired state, but will not update: %s",
-					observation.Diff)))
+		msg := "External resource differs from desired state, but will not update due to managementPolicy."
+		if observation.Diff != "" {
+			msg = fmt.Sprintf("%s Diff: %s", msg, observation.Diff)
+		}
+		status.MarkConditions(xpv1.ReconcileForbidden().WithMessage(msg))
 
 		return reconcile.Result{RequeueAfter: reconcileAfter}, errors.Wrap(r.client.Status().Update(ctx, managed), errUpdateManagedStatus)
 	}