Skip to content

Commit

Permalink
CI: Travis: reduce the number of stages to two
Browse files Browse the repository at this point in the history
Travis stages are designed not to run after the first that fails. We
never used them for that, only for nicer presentation of
results. Running all tests independently of each other (e.g.: qemu and
doxygen) even when one fails is more important than presentation, so
reduce the number of stages to just two: "buildonly" and "tests"
Travis job name: doesn't look as good alone as when combined
with a stage name but it's enough to identify jobs.

Reducing to just one stage is tempting but it would require a lot more
naming surgery so let's start with this small diff and two stages for
now.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
  • Loading branch information
marc-hb authored and lgirdwood committed Apr 11, 2020
1 parent b12a662 commit 3808dfe
Showing 1 changed file with 13 additions and 10 deletions.
23 changes: 13 additions & 10 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,18 +9,20 @@ services:
- docker

# We don't need the strict sequence of stages; for now we use stages
# only as presentation labels. The strict order hurts only in case of a
# failure however there are only basic, non-flaky tests here.
# only as presentation labels. Nothing in stage "tests" will run if
# anything in buildonly fails; in the future we may want to get rid of
# stages.
stages:
- buildonly
- qemutest
- testbench
- doxygen
- tests


# 'name:'-less jobs appear with their env, e.g.: PLATFORM=sue
jobs:
include:

# stage buildonly

- &build-platform
stage: buildonly
before_install:
Expand All @@ -34,17 +36,18 @@ jobs:
env: PLATFORM='jsl'

- name: "./scripts/build-tools.sh"
stage: buildonly
before_install: *docker-pull-sof
script: ./scripts/docker-run.sh ./scripts/build-tools.sh

- name: "./scripts/host-build-all.sh"
stage: buildonly
before_install: *docker-pull-sof
script: ./scripts/docker-run.sh ./scripts/host-build-all.sh -l


# stage tests

- &qemuboottest
stage: qemutest
stage: tests
script:
- sed -i $(($(grep "config HAVE_AGENT" -n src/platform/Kconfig | cut -f1 -d:)+2))"s/default y/default n/" src/platform/Kconfig
- ./scripts/docker-run.sh ./scripts/xtensa-build-all.sh -r -j $PLATFORM
Expand All @@ -67,7 +70,7 @@ jobs:
env: PLATFORM='imx8 imx8x imx8m'


- stage: testbench
- name: testbench
# Disabled for as long as host-testbench.sh ignores failures, see
# https://github.com/thesofproject/sof/issues/2752
if: false
Expand All @@ -78,7 +81,7 @@ jobs:
- ./scripts/host-testbench.sh


- stage: doxygen
- name: doxygen

before_install: sudo apt-get -y install ninja-build doxygen graphviz

Expand Down

0 comments on commit 3808dfe

Please sign in to comment.