Skip to content

Commit 9890c05

Browse files
authored
Explicitly requeue during backoff ephemeral runner (#4152)
1 parent 3b5693e commit 9890c05

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

controllers/actions.github.com/ephemeralrunner_controller.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -201,12 +201,16 @@ func (r *EphemeralRunnerReconciler) Reconcile(ctx context.Context, req ctrl.Requ
201201
backoffDuration := failedRunnerBackoff[len(ephemeralRunner.Status.Failures)]
202202
nextReconciliation := lastFailure.Add(backoffDuration)
203203
if !lastFailure.IsZero() && now.Before(&metav1.Time{Time: nextReconciliation}) {
204+
requeueAfter := nextReconciliation.Sub(now.Time)
204205
log.Info("Backing off the next reconciliation due to failure",
205206
"lastFailure", lastFailure,
206207
"nextReconciliation", nextReconciliation,
207-
"requeueAfter", nextReconciliation.Sub(now.Time),
208+
"requeueAfter", requeueAfter,
208209
)
209-
return ctrl.Result{RequeueAfter: now.Sub(nextReconciliation)}, nil
210+
return ctrl.Result{
211+
Requeue: true,
212+
RequeueAfter: requeueAfter,
213+
}, nil
210214
}
211215

212216
secret := new(corev1.Secret)

0 commit comments

Comments
 (0)