Repository navigation
Restore one-minute retries for healthcheck delivery - #798
Merged
Merged
Conversation
Keep database and executor reporting, retry failed deliveries after one minute, and log the actual error. Leave registry jobs unchanged.
Include nested Fetch causes, error codes, and connection details in server logs. Keep healthcheck timing and registry job behavior unchanged.
f-f
approved these changes
Oct 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The healthcheck changes in #795 kept the database and executor checks, but changed failed ping delivery from a one-minute retry to a five-minute retry. That gives a transient delivery failure fewer chances to recover before Healthchecks alerts.
This restores the previous timing: wait five minutes after an accepted report, and retry failed deliveries after one minute. It keeps the operational metadata and ten-second request timeout, and logs the actual delivery error. Registry jobs and compiler-matrix scheduling are unchanged.
I verified recovery with a mock that rejected the first ping with HTTP 503. The previous version missed the recovery deadline; this version recovered within 90 seconds while registry jobs ran. This is already manually deployed.