Skip to content
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.

Sign up for GitHub

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

Jump to bottom

MAINT: Dont use parallel Sphinx #13807

Merged
merged 1 commit into from Apr 11, 2021
Merged

MAINT: Dont use parallel Sphinx #13807

merged 1 commit into from Apr 11, 2021

Conversation

larsoner
Copy link
Member

@larsoner larsoner commented Apr 5, 2021

Currently takes ~30 minutes to build on CircleCI. Let's see if a non-parallel build is comparable.

@rgommers rgommers added the CI Items related to the CI tools such as CircleCI, GitHub Actions or Azure label Apr 5, 2021
@rgommers
Copy link
Member

rgommers commented Apr 5, 2021

21min 8s before (on the arxiv role PR), 29m 52s here for html + pdf builds. I've tried locally as well on a Macbook (html build only), there it was 20m 10s to 26m 50 s.

Probably just bumping the timeout limit a bit further if it happens again is better than merging this PR.

@larsoner
Copy link
Member Author

larsoner commented Apr 5, 2021

Agreed!

@larsoner larsoner closed this Apr 5, 2021
@mdhaber
Copy link
Contributor

mdhaber commented Apr 10, 2021

Note sure if it's related but I'm seeing:

waiting for workers...
Makefile:127: recipe for target 'html-scipyorg' failed
make: *** [html-scipyorg] Hangup

Too long with no output (exceeded 20m0s): context deadline exceeded

in a few PRs: gh-13832, gh-13298

@ilayn
Copy link
Member

ilayn commented Apr 11, 2021

I think all PRs are being hit with that currently

@rgommers rgommers mentioned this pull request Apr 11, 2021
Closed
@rgommers rgommers reopened this Apr 11, 2021
@rgommers rgommers merged commit 9919653 into scipy:master Apr 11, 2021
@rgommers
Copy link
Member

Merging after all, things steadily got worse. Let's see if this gets our CI status back to green for most PRs ....

@rgommers rgommers mentioned this pull request Apr 11, 2021
Merged
@tylerjereddy tylerjereddy added this to the 1.7.0 milestone Apr 11, 2021
@rgommers rgommers mentioned this pull request Apr 11, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CI Items related to the CI tools such as CircleCI, GitHub Actions or Azure
Projects
None yet
Milestone
1.7.0
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@larsoner @rgommers @mdhaber @ilayn @tylerjereddy