Skip to content

Commit

Permalink
ci: cirrus: Fix test ranges.
Browse files Browse the repository at this point in the history
The CirrusCI configuration file was not updated back when the size of the
test matrix was reduced.

CirrusCI jobs were complaining of invalid test ranges for a while but
that slipped through the cracks until commit 0224e45 ("ci: Remove
'--recheck' in CI.") uncovered the failure:

  make distcheck 'CFLAGS= ' -j4 'TESTSUITEFLAGS=-j4 501-1000' RECHECK=no SKIP_UNSTABLE=yes
  [...]
  invalid test group: 1000

At the same time, make sure --recheck is passed to ARM jobs (on
CirrusCI).  These have been always more unstable than other CI jobs.

Fixes: 937a9b5 ("ci, tests: Use parallelization permutations for few jobs")
Fixes: 0224e45 ("ci: Remove '--recheck' in CI.")
Signed-off-by: Dumitru Ceara <dceara@redhat.com>
Acked-by: Ales Musil <amusil@redhat.com>
  • Loading branch information
dceara committed Nov 21, 2023
1 parent ae63212 commit 5ef2a08
Showing 1 changed file with 9 additions and 6 deletions.
15 changes: 9 additions & 6 deletions .cirrus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,25 +9,28 @@ arm_unit_tests_task:
ARCH: aarch64
CIRRUS_CLONE_SUBMODULES: true
PATH: ${HOME}/bin:${HOME}/.local/bin:${PATH}
RECHECK: yes
matrix:
- CC: gcc
TESTSUITE: test
TEST_RANGE: -500
TEST_RANGE: -300
- CC: gcc
TESTSUITE: test
TEST_RANGE: 501-1000
TEST_RANGE: 301-600
- CC: gcc
TESTSUITE: test
TEST_RANGE: 1001-
TEST_RANGE: 601-
UNSTABLE: yes
- CC: clang
TESTSUITE: test
TEST_RANGE: -500
TEST_RANGE: -300
- CC: clang
TESTSUITE: test
TEST_RANGE: 501-1000
TEST_RANGE: 301-600
- CC: clang
TESTSUITE: test
TEST_RANGE: 1001-
TEST_RANGE: 601-
UNSTABLE: yes

name: ARM64 ${CC} ${TESTSUITE} ${TEST_RANGE}

Expand Down

0 comments on commit 5ef2a08

Please sign in to comment.