queue_retry test: measure retries from when each attempt started

jcoffey-dev commented

2026-09-23 04:13:13 +00:00

Owner

The server sets a deferred recipient's next retry from its clock when the
attempt defers, in whole seconds. The test subtracted its own clock taken
when the loop next saw the message, after saving and reporting, so
whenever that lag crossed a second boundary the 2 s retry measured 1 s
and the test failed. Under load, after the other SMTP tests, that was
most runs.

It now measures from when the test started the attempt, which the
server's deferral can only follow, by under a second: each retry is its
interval or one more. Each position is still checked against its own
interval, so a wrong schedule still fails.

Summary

Related issues

Upstream files

Clean room

Testing

The server sets a deferred recipient's next retry from its clock when the attempt defers, in whole seconds. The test subtracted its own clock taken when the loop next saw the message, after saving and reporting, so whenever that lag crossed a second boundary the 2 s retry measured 1 s and the test failed. Under load, after the other SMTP tests, that was most runs. It now measures from when the test started the attempt, which the server's deferral can only follow, by under a second: each retry is its interval or one more. Each position is still checked against its own interval, so a wrong schedule still fails.  ## Summary  ## Related issues  ## Upstream files  ## Clean room  ## Testing

jcoffey-dev added 1 commit 2026-09-23 04:13:13 +00:00

queue_retry test: measure retries from when each attempt started

ci / name-check (pull_request) Successful in 2m30s

Details

ci / build (pull_request) Successful in 7m33s

Details

86bf2432a2

The server sets a deferred recipient's next retry from its clock when the
attempt defers, in whole seconds. The test subtracted its own clock taken
when the loop next saw the message, after saving and reporting, so
whenever that lag crossed a second boundary the 2 s retry measured 1 s
and the test failed. Under load, after the other SMTP tests, that was
most runs.

It now measures from when the test started the attempt, which the
server's deferral can only follow, by under a second: each retry is its
interval or one more. Each position is still checked against its own
interval, so a wrong schedule still fails.

jcoffey-dev merged commit ce6882fe93 into main

2026-09-23 04:21:38 +00:00

queue_retry test: measure retries from when each attempt started #25

Summary

Related issues

Upstream files

Clean room

Testing