Skip to content

Commit

Permalink
testsuite: skip el capitan test by default
Browse files Browse the repository at this point in the history
Problem: the debian riscv64 build environment is failing on the
el capitan scaling test.

We don't have detailed test output to diagnose the failure but we
can probably assume that this is a fairly under-powered builder.

Set the EXPENSIVE built-in test prereq so that this test only runs
if TEST_LONG is set in the environment or the --long test option is
given.

To avoid losing coverage in github CI, also allow the test to run
if GITHUB_CI is set in the environment, and then set it where the
workflow runs make check.

Fixes chaos#206

more
  • Loading branch information
garlick committed Sep 17, 2024
1 parent 4ac7370 commit a56c166
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ jobs:
--with-tcp-wrappers
- name: make
run: make
- name: make check
- name: GITHUB_CI=t make check
run: make check
- name: make distcheck
run: DISTCHECK_CONFIGURE_FLAGS=--with-redfishpower make distcheck
Expand Down
6 changes: 6 additions & 0 deletions t/t0039-llnl-el-capitan-cluster.t
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,12 @@ test_description='Check LLNL El Capitan config'

. `dirname $0`/sharness.sh

# Setting the --long option or TEST_LONG=t fulfills the EXPENSIVE prereq
if ! test_have_prereq EXPENSIVE && ! test -n "$GITHUB_CI"; then
skip_all='skipping large scale El Capitan test'
test_done
fi

ulimit -n 2048

powermand=$SHARNESS_BUILD_DIRECTORY/src/powerman/powermand
Expand Down

0 comments on commit a56c166

Please sign in to comment.