.travis.yml: remove travis matrix and do a single sequential run #3085

Merged
merged 3 commits into from Apr 4, 2017

Conversation

Projects
None yet
4 participants
Contributor

fgimenez commented Mar 27, 2017

With these changes each build will only require one travis slot. Also, if the static or unit tests fail we won't allocate linode resources.

fgimenez added some commits Mar 27, 2017

Collaborator

mvo5 commented Mar 28, 2017

The change looks fine. Just wondering why we change it. One advantage of running static/unit/integration is that a static/unit test failure is super quick to spot this way. But I'm not against it, mostly curious about the motivation.

Contributor

fgimenez commented Mar 28, 2017

@mvo5 thanks for your review, the main goal is to only use one travis slot on each PR instead of 3 as we do now, this hopefully will reduce the initial wait time before the jobs start to execute.

During the discussion it was proposed a sequential execution so that we don't allocate linode resources if static or unit test fails, but this would impose an upfront delay in all the PR builds because of the unit test duration (currently about 6min, but this will increase over time). With the approach in this changeset we use a single travis slot and the unit tests are executed as a spread task in parallel with others.

mvo5 approved these changes Mar 29, 2017

@niemeyer niemeyer merged commit 98d20e7 into snapcore:master Apr 4, 2017

6 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
xenial-amd64 autopkgtest finished (success)
Details
xenial-i386 autopkgtest finished (success)
Details
xenial-ppc64el autopkgtest finished (success)
Details
yakkety-amd64 autopkgtest finished (success)
Details
zesty-amd64 autopkgtest finished (success)
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment