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

[CI] Build time optimizations #1712

Merged
merged 13 commits into from Mar 8, 2019

Conversation

@avtolstoy
Copy link
Member

commented Mar 6, 2019

Problem

CI builds take too long.

Solution

I don't want them to take that long to finish. This PR introduces some optimizations to the CI builds:

  1. The number of concurrent jobs is set to 5, with the job build times balanced. You might ask why 5: unfortunately for some reason most of the times Travis doesn't start more than 5 build jobs at a time.
  2. enumerate_build_matrix.sh now uses parallel tool to utilize both cores available on Travis workers
  3. It is now possible to build DeviceOS out of tree (using BUILD_PATH_BASE). This feature is used in enumerate_build_matrix.sh where each build job has its own build directory
  4. Fixes newhal platform builds
  5. Fixes compilation issues with new gcc versions

  • [Internal] [CI] Build time optimizations #1712
  • [Bug fix] [wiring] Fixed a potential (but unlikely due to bounds checking) buffer overflow in time formatting function #1712

@avtolstoy avtolstoy force-pushed the test/ci-benchmark branch 7 times, most recently from 991aebe to ab4e2db Mar 7, 2019

@avtolstoy avtolstoy changed the title [Test] ci: benchmarking [CI] Build time optimizations Mar 7, 2019

@avtolstoy avtolstoy requested review from m-mcgowan and technobly Mar 7, 2019

@avtolstoy avtolstoy added this to the 1.0.2-rc.1 milestone Mar 7, 2019

@avtolstoy avtolstoy force-pushed the test/ci-benchmark branch from ee447c5 to a7215e5 Mar 7, 2019

@avtolstoy avtolstoy referenced this pull request Mar 7, 2019
4 of 6 tasks complete

@avtolstoy avtolstoy force-pushed the test/ci-benchmark branch from 5e18354 to a7c21d5 Mar 8, 2019

@technobly technobly added the bug label Mar 8, 2019

@technobly technobly merged commit b4e8804 into develop Mar 8, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@technobly technobly deleted the test/ci-benchmark branch Mar 8, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.