search

reddit / rollingpin

fast deploy to lots of servers
https://redditblog.com/2017/06/02/the-evolution-of-code-deploys-at-reddit/
Other
145 stars 33 forks source link

Add harold timeout #61

Closed eaceaser closed 4 years ago

eaceaser commented 4 years ago

adds a 5 second timeout to harold requests.

When the status API times out (this behavior already existed):

failed to fetch deploy status: timed out

When the harold progress event times out (this is new timeout behavior):

harold: request timed out after 5 seconds (/harold/deploy/progress)

spladug commented 4 years ago

(addTimeout could probably be applied to the per-server work to handle the hung host issue as well)