chore: adds requeing duration while finalizing node

kloudlite · Jan 20, 2025 · b21246b · b21246b
1 parent ea06526
commit b21246b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 12 deletions.
diff --git a/operators/nodepool/internal/node-controller/controller.go b/operators/nodepool/internal/node-controller/controller.go
@@ -209,7 +209,8 @@ func (r *Reconciler) finalize(req *rApi.Request[*clustersv1.Node]) stepResult.Re
 		return check.Completed()
 	}
 
-	return check.StillRunning(fmt.Errorf("node will be deleted at %s", t.Format(time.RFC3339))).NoRequeue().RequeueAfter(time.Since(t))
+	requeueAfter := time.Since(t)
+	return check.StillRunning(fmt.Errorf("node will be deleted at %s, requeueing after %.2fs", t.Format(time.RFC3339), requeueAfter.Seconds())).NoRequeue().RequeueAfter(requeueAfter)
 }
 
 func (r *Reconciler) SetupWithManager(mgr ctrl.Manager) error {
@@ -229,16 +230,6 @@ func (r *Reconciler) SetupWithManager(mgr ctrl.Manager) error {
 		}),
 	)
 
-	// builder.Watches(
-	// 	&source.Kind{Type: &corev1.Node{}},
-	// 	handler.EnqueueRequestsFromMapFunc(func(obj client.Object) []reconcile.Request {
-	// 		if v, ok := obj.GetLabels()[constants.NodeNameKey]; ok {
-	// 			return []reconcile.Request{{NamespacedName: fn.NN("", v)}}
-	// 		}
-	// 		return nil
-	// 	}),
-	// )
-
 	builder.WithOptions(controller.Options{MaxConcurrentReconciles: r.Env.MaxConcurrentReconciles})
 	builder.WithEventFilter(rApi.ReconcileFilter())
 	return builder.Complete(r)

diff --git a/operators/nodepool/main.go b/operators/nodepool/main.go
@@ -1,8 +1,8 @@
 package main
 
 import (
-	"github.com/kloudlite/operator/operator"
 	"github.com/kloudlite/operator/operators/nodepool/controller"
+	"github.com/kloudlite/operator/toolkit/operator"
 )
 
 func main() {