reduce dataImage unmanagedRetryDelay to 10 seconds

Signed-off-by: Himanshu Roy <hroy@redhat.com>
metal3-io · Nov 6, 2024 · c1a83e1 · c1a83e1
1 parent 2a379d7
commit c1a83e1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/controllers/metal3.io/baremetalhost_controller.go b/controllers/metal3.io/baremetalhost_controller.go
@@ -1593,7 +1593,7 @@ func (r *BareMetalHostReconciler) handleDataImageActions(prov provisioner.Provis
 		if hostDeleteTriggered {
 			detachReason = "BMH requested to be deleted"
 		}
-		info.log.Info("%w, requesting DataImage detach", "Reason", detachReason)
+		info.log.Info("Requesting DataImage detach", "Reason", detachReason)
 		if attachedURL != "" || dirty {
 			info.log.Info("Detaching DataImage, Reason = %w", "Reason", detachReason)
 			err := r.detachDataImage(prov, info, dataImage)

diff --git a/controllers/metal3.io/dataimage_controller.go b/controllers/metal3.io/dataimage_controller.go
@@ -36,8 +36,9 @@ import (
 )
 
 const (
-	dataImageRetryDelay  = time.Second * 60
-	dataImageUpdateDelay = time.Second * 30
+	dataImageRetryDelay          = time.Second * 60
+	dataImageUpdateDelay         = time.Second * 30
+	dataImageUnmanagedRetryDelay = time.Second * 10
 )
 
 // DataImageReconciler reconciles a DataImage object.
@@ -129,7 +130,7 @@ func (r *DataImageReconciler) Reconcile(ctx context.Context, req ctrl.Request) (
 
 	if hasDetachedAnnotation(bmh) {
 		reqLogger.Info("the host is detached, not running reconciler")
-		return ctrl.Result{Requeue: true, RequeueAfter: unmanagedRetryDelay}, nil
+		return ctrl.Result{Requeue: true, RequeueAfter: dataImageUnmanagedRetryDelay}, nil
 	}
 
 	// If the reconciliation is paused, requeue