address minor comment

Arvind Thirumurugan · Arvind Thirumurugan · commit e85a7a012d6c · 2025-12-10T16:47:05.000-08:00
Signed-off-by: Arvind Thirumurugan &lt;arvindth@microsoft.com&gt;
diff --git a/approval-controller-metric-collector/approval-request-controller/pkg/controller/controller.go b/approval-controller-metric-collector/approval-request-controller/pkg/controller/controller.go
@@ -193,7 +193,7 @@ func (r *Reconciler) reconcileApprovalRequestObj(ctx context.Context, approvalRe
 	// Check workload health and approve if all workloads are healthy
 	if err := r.checkWorkloadHealthAndApprove(ctx, approvalReqObj, clusterNames, updateRunName, stageName); err != nil {
 		klog.ErrorS(err, "Failed to check workload health", "approvalRequest", approvalReqRef)
-		return ctrl.Result{RequeueAfter: 15 * time.Second}, err
+		return ctrl.Result{}, err
 	}
 
 	// Requeue after 15 seconds to check again (will stop if approved in next reconciliation)

Original file line number	Diff line number	Diff line change
`@@ -193,7 +193,7 @@ func (r *Reconciler) reconcileApprovalRequestObj(ctx context.Context, approvalRe`
`193`	`193`	`// Check workload health and approve if all workloads are healthy`
`194`	`194`	`if err := r.checkWorkloadHealthAndApprove(ctx, approvalReqObj, clusterNames, updateRunName, stageName); err != nil {`
`195`	`195`	`klog.ErrorS(err, "Failed to check workload health", "approvalRequest", approvalReqRef)`
`196`		`- return ctrl.Result{RequeueAfter: 15 * time.Second}, err`
	`196`	`+ return ctrl.Result{}, err`
`197`	`197`	`}`
`198`	`198`
`199`	`199`	`// Requeue after 15 seconds to check again (will stop if approved in next reconciliation)`