search

nextstrain / nextstrain.org

The Nextstrain website
https://nextstrain.org
GNU Affero General Public License v3.0
87 stars 49 forks source link

dev: Wait up to 60s for the Heroku build status to change to "succeeded" #800

Closed tsibley closed 4 months ago

tsibley commented 4 months ago

It used to be that once the Heroku build log stream response completed, their API would return a build status of "succeeded". Recently however it seems like now there's a delay between the log stream response completing and their API returning the up-to-date build status. Build in a 60s buffer wait time to account for that.

Resolves https://github.com/nextstrain/nextstrain.org/issues/794.

Checklist

tsibley commented 4 months ago

This worked in my local testing in isolation. I'll merge it and it will then test itself.

tsibley commented 4 months ago

The CI for merge wasn't a full test of the new behaviour, as the stochastic "status": "pending" wasn't present on the first check.