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.

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

Turn on parallel builds in edx-documentation again. #2107

Open
feanil opened this issue Feb 10, 2023 · 0 comments
Open

Turn on parallel builds in edx-documentation again. #2107

feanil opened this issue Feb 10, 2023 · 0 comments

Comments

@feanil
Copy link
Contributor

feanil commented Feb 10, 2023

Check to see if https://gitlab.com/documatt/sphinx-reredirects/-/merge_requests/10 has merged and been released, if not, give it another nudge and bump this ticket out a few sprints.

Once https://gitlab.com/documatt/sphinx-reredirects/-/merge_requests/10 has merged, we can turn on parallel builds again in run_tests.sh.

See cf90b90 for a more detailed explanation.

@feanil feanil converted this from a draft issue Feb 10, 2023
feanil pushed a commit that referenced this issue Feb 10, 2023
The sphinx-reredirect extension does not declare it self safe or unsafe for
parallel execution which leads to sphinx generating a warning when we build
for testing.

For now remove the parallel execution to get rid of the warning.

Longer Term: I've made a Merge request against the extension to fix the issue.

https://gitlab.com/documatt/sphinx-reredirects/-/merge_requests/10

and I've created #2107 to
track the issue so it doesn't get lost.

In the mean time, turning off the parallel builds doesn't seem to significantly
slow down the builds, a clean build from scratch without parallelization took less
than 6 minutes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Backlog - Other
Development

No branches or pull requests

1 participant