chaos / powerman

cluster power control
GNU General Public License v2.0
43 stars 19 forks source link

testsuite: skip huge test unless requested #210

Closed garlick closed 1 month ago

garlick commented 1 month ago

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.

Disable the test unless TEST_LONG is set in the test environment. Set TEST_LONG when the github workflow runs make check, so we won't lose this coverage in github CI.

Fixes #206

garlick commented 1 month ago

oops might need to do this another way

garlick commented 1 month ago

TEST_LONG=t make check always fails because one of the sharness tests checks that EXPENSIVE tests don't run by default.

Switched the prereq to EXPENSIVE (the correct prereq name to go with TEST_LONG) and also let the test run if GITHUB_CI is set in the environment, then set that in the workflow.

garlick commented 1 month ago

Sure I'll add that and then set MWP. Thanks!