Ensure defaults and all resource mutation are considered before compa…

…ring existing vs desired state of resources when reconciling updates, by dry-running the updates before (#356)
Kuadrant · Dec 4, 2023 · 1b6d635 · 1b6d635
1 parent 2acf31b
commit 1b6d635
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/pkg/reconcilers/base_reconciler.go b/pkg/reconcilers/base_reconciler.go
@@ -126,6 +126,11 @@ func (b *BaseReconciler) ReconcileResource(ctx context.Context, obj, desired cli
 		return b.DeleteResource(ctx, desired)
 	}
 
+	desired.SetResourceVersion(obj.GetResourceVersion())
+	if err := b.Client().Update(ctx, desired, client.DryRunAll); err != nil {
+		return err
+	}
+
 	update, err := mutateFn(obj, desired)
 	if err != nil {
 		return err