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

Separate test and release stages in travis #1007

Merged
merged 1 commit into from
Jul 10, 2018

Conversation

mcspr
Copy link
Collaborator

@mcspr mcspr commented Jun 29, 2018

  • Revert test to being serial. Releases are using parallel jobs.
  • Some cosmetic changes in build.sh

Matrix expansion is somehow slower? I did watch #986 merge build, 10 minutes per 10 minutes total. Which is strange, bc jobs did not start parallel at all.

@mcspr mcspr mentioned this pull request Jun 29, 2018
@mcspr
Copy link
Collaborator Author

mcspr commented Jun 30, 2018

About matrix expansion. Is this limit set?
https://docs.travis-ci.com/user/customizing-the-build/#Limiting-Concurrent-Jobs
Last build is sequential too (723.1, only then 723.2 etc.)
https://travis-ci.org/xoseperez/espurna/builds/398680461

Comparing repo builds of the same commit I suspect limit is set for xoseperez/espurna:
https://travis-ci.org/mcspr/espurna/builds/398759537 - 3:45 minutes real time / 6:17 minutes total across 2 jobs
https://travis-ci.org/xoseperez/espurna/builds/398759545 - 6:56 minutes real time (should be half that) / 6:59 minutes total

@xoseperez
Copy link
Owner

Yes, it was limited to 1, changed it to 4 again.

- Revert test to being serial. Releases are using parallel jobs.
- Some cosmetic changes in build.sh
@mcspr mcspr force-pushed the parallel-travis-jobs branch from 6f801c6 to 5fdf4aa Compare July 1, 2018 22:08
@mcspr
Copy link
Collaborator Author

mcspr commented Jul 1, 2018

gtg then. made tests parallel again without 4th job, releases are still 4 jobs to keep that as fast as possible.

@xoseperez xoseperez merged commit 95ac8c6 into xoseperez:dev Jul 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants