fix(nodeclaim): dedup logs marking consolidatable

Signed-off-by: flavono123 <[email protected]>

Author: flavono123

pkg/controllers/nodeclaim/disruption/consolidation.go

@@ -18,6 +18,7 @@ package disruption
 
 import (
 	"context"
+	"time"
 
 	"github.com/samber/lo"
 	"k8s.io/utils/clock"
@@ -69,8 +70,12 @@ func (c *Consolidation) Reconcile(ctx context.Context, nodePool *v1.NodePool, no
 		return reconcile.Result{RequeueAfter: consolidatableTime.Sub(c.clock.Now())}, nil
 	}
 
-	// 6. Otherwise, add the consolidatable status condition
+	// 3. Otherwise, add the consolidatable status condition if not already set
 	nodeClaim.StatusConditions().SetTrue(v1.ConditionTypeConsolidatable)
+	// We sleep here after a set operation since we want to ensure that we are able to read our own writes
+	// so that we avoid duplicating log lines due to quick re-queues from our node watcher
+	time.Sleep(100 * time.Millisecond)
+
 	if !hasConsolidatableCondition {
 		log.FromContext(ctx).V(1).Info("marking consolidatable")
 	}