-
-
Notifications
You must be signed in to change notification settings - Fork 453
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
CI Build&Test: Split test-long into multiple jobs, repair Coverage.py upload #38090
Conversation
Documentation preview for this PR (built with commit a0262b6; changes) is ready! 🎉 |
53727f8
to
d24f52f
Compare
c1177cc
to
2d31669
Compare
2d31669
to
46f5148
Compare
Is there an example run? This is to save 15 minutes. Is it worth the effort? Workflow scripts are the second-class citizen. But in the long run, they are still burden of the community. The workflow script gets more complicated. Saving 15 minutes is worth it? |
Yes, right here on the PR |
... to completion of all jobs, yes. But the time to completion of test-long is cut in half from 100 minutes to 50 minutes |
From the example run,
Could we remove the warnings? |
OK. Certainly nice reward for the effort. |
Trying in a0262b6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. LGTM.
Thank you! |
…repair Coverage.py upload <!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> test-long (90 minutes) runs after a successful test-new (10 minutes). This is the critical path (100 minutes), longer than build-docs-pdf (85 minutes). We split it into multiple parallel jobs (33 minutes), merging the coverage files in a job afterwards (6 minutes). This also repairs the Coverage.py upload, which apparently got broken. ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> URL: sagemath#38090 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee
test-long (90 minutes) runs after a successful test-new (10 minutes). This is the critical path (100 minutes), longer than build-docs-pdf (85 minutes). We split it into multiple parallel jobs (33 minutes), merging the coverage files in a job afterwards (6 minutes).
This also repairs the Coverage.py upload, which apparently got broken.
📝 Checklist
⌛ Dependencies