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

Add caching of apt-get + pip to .travis.yml #5678

Merged
merged 2 commits into from Dec 12, 2017

Conversation

Projects
None yet
6 participants
@geky
Member

geky commented Dec 8, 2017

Significantly reduces the overhead we have on spinning up Travis jobs, which was mostly spent downloading gcc-arm-embedded. Cuts the cost of the smaller jobs in half.

Before:
image

After:
image

The most useful thing to us is it reduces how long it takes to debug issues with Travis.

cc @bridadan, @0xc0170, @theotherjimmy

@theotherjimmy

Looks good.

@geky geky force-pushed the geky:travis-caching branch from 3b98524 to 0f06a43 Dec 8, 2017

@geky geky force-pushed the geky:travis-caching branch from 0f06a43 to b7c1598 Dec 8, 2017

@geky

This comment has been minimized.

Member

geky commented Dec 8, 2017

Also, since jobs are cheap to spin up now, I split out the doc checks into a "docs" job.

@bridadan

Noice!

Any idea what will happen if we want to upgrade the version of GCC ARM?

@geky

This comment has been minimized.

Member

geky commented Dec 8, 2017

You can specify a specific version (which will fall back to downloading):

sudo apt-get install gcc-arm-embedded=15:4.9.3+svn231177-1

Or you can just pull the "reset cache" lever in Travis.

@bridadan

This comment has been minimized.

Contributor

bridadan commented Dec 8, 2017

Cool, so when a new GCC ARM embedded version is release will just bump that release, sweet!

@kegilbert

This comment has been minimized.

Contributor

kegilbert commented Dec 8, 2017

@geky This is awesome, thanks!

@0xc0170 0xc0170 added the needs: CI label Dec 10, 2017

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Dec 10, 2017

/morph build

@mbed-ci

This comment has been minimized.

mbed-ci commented Dec 10, 2017

Build : SUCCESS

Build number : 673
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5678/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build

@mbed-ci

This comment has been minimized.

@mbed-ci

This comment has been minimized.

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Dec 11, 2017

@0xc0170 0xc0170 merged commit 992c905 into ARMmbed:master Dec 12, 2017

11 checks passed

AWS-CI uVisor Build & Test Success
Details
ci-morph-build build completed
Details
ci-morph-exporter build completed
Details
ci-morph-test test completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
travis-ci/docs Local docs testing has passed
Details
travis-ci/events Local events testing has passed
Details
travis-ci/littlefs Local littlefs testing has passed
Details
travis-ci/mbed2 Local mbed2 testing has passed
Details
travis-ci/tools Local tools testing has passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment