Skip to content

Commit fcb656b

Browse files
vdemeesterkhrm
authored andcommitted
Improve code consistency and fix missing test annotation
Use early returns in timeout handling for better code consistency and readability throughout the reconciliation logic. Add missing @test:execution=parallel annotation to the excessive reconciliation e2e test for proper test categorization. Addresses review feedback from PR tektoncd#9202.
1 parent 8e3b1b2 commit fcb656b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

pkg/reconciler/pipelinerun/pipelinerun.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ func (c *Reconciler) ReconcileKind(ctx context.Context, pr *v1.PipelineRun) pkgr
295295
if timeout != config.NoTimeoutDuration {
296296
waitTime := timeout - elapsed
297297
if finallyWaitTime < waitTime {
298-
waitTime = finallyWaitTime
298+
return controller.NewRequeueAfter(finallyWaitTime)
299299
}
300300
return controller.NewRequeueAfter(waitTime)
301301
}

0 commit comments

Comments
 (0)