-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Timeout seems to have reduced to 600 seconds. #6308
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I think this bug was introduced when replacing the setting in #6040. |
I think that I had a build passing after the commit got merged, e.g. https://readthedocs.org/projects/colour/builds/9794465/, I'm not sure how long it takes for the code to get rolled to the production server though. It is kind of a bummer because I was intending to release a new version of Colour over the weekend but won't be able because I don't have a matching documentation build. Let me know if there is anything I can do to help! |
We don't deploy changes that often, usually once or twice per month https://docs.readthedocs.io/en/stable/faq.html#what-commit-of-read-the-docs-is-in-production I've investigated this. The rename of the settings is correct, but we didn't pull the new changes for the ops repo, that's why it's using the defaults of |
This should be fixed, sorry about that. |
No worries and thanks @ericholscher! I have a build that went through! <3 |
Details
Expected Result
A description of what you wanted to happen
Build exited due to time out:
Actual Result
A description of what actually happened
The build should pass :) It seems like the timeout has been decreased from 900secs to 600secs recently as I have a few builds that died after 10mins. It is possible to increase the timeout please?
Cheers,
Thomas
The text was updated successfully, but these errors were encountered: