From 7726fcb9449bcfeb059f367b273709b57471464d Mon Sep 17 00:00:00 2001 From: Christian Schlotter Date: Sun, 26 Jun 2022 01:55:01 +0200 Subject: [PATCH] topology controller should takeover ownership for fields at clusterClass.spec.topology --- internal/controllers/topology/cluster/reconcile_state.go | 1 + 1 file changed, 1 insertion(+) diff --git a/internal/controllers/topology/cluster/reconcile_state.go b/internal/controllers/topology/cluster/reconcile_state.go index 10de4fb222b9..ce359618370e 100644 --- a/internal/controllers/topology/cluster/reconcile_state.go +++ b/internal/controllers/topology/cluster/reconcile_state.go @@ -441,6 +441,7 @@ func (r *Reconciler) reconcileCluster(ctx context.Context, s *scope.Scope) error // Check differences between current and desired state, and eventually patch the current object. patchHelper, err := r.patchHelperFactory(ctx, s.Current.Cluster, s.Desired.Cluster, structuredmerge.IgnorePaths{ {"spec", "controlPlaneEndpoint"}, // this is a well known field that is managed by the Cluster controller, topology should not express opinions on it. + {"spec", "topology"}, // this is a well known part read as input for the topology controller, topology should not try to patch variables on it. }) if err != nil { return errors.Wrapf(err, "failed to create patch helper for %s", tlog.KObj{Obj: s.Current.Cluster})