Skip to content

Commit 9920be9

Browse files
committed
fix: skip heavy recursion tests in debug builds to fix nightly CI timeout
Several recursive verifier test suites (AvmRecursionConstraintTest, ECCVMRecursiveTests, GoblinRecursiveVerifierTests, BoomerangTwoLayerAvmRecursiveVerifierTests, etc.) were not included in the debug-build skip list. These tests take 7-10+ minutes each in debug mode, exceeding the default 600s per-test timeout in CI. Also added these suites to the extra-resources pattern (CPUS=4:MEM=8g) for any that still run.
1 parent cf16159 commit 9920be9

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

barretenberg/cpp/bootstrap.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -245,19 +245,19 @@ function test_cmds_native {
245245
awk '/^[a-zA-Z]/ {suite=$1} /^[ ]/ {print suite$1}' | \
246246
grep -v 'DISABLED_' | \
247247
while read -r test; do
248-
# Skip heavy recursion tests in debug builds — they take 400-600s and the same
248+
# Skip heavy recursion tests in debug builds — they take 400-600s+ and the same
249249
# code paths are already exercised (with assertions) by faster tests in the suite.
250250
# Keep WithoutPredicate/1.GenerateVKFromConstraints (241s) so that the debug-only
251251
# native_verification_debug path in honk_recursion_constraint.cpp is still exercised.
252-
if [[ "$native_preset" == *debug* ]] && [[ "$test" =~ ^(HonkRecursionConstraintTest|ChonkRecursionConstraintTest|AvmRecursionInnerCircuitTests) ]]; then
252+
if [[ "$native_preset" == *debug* ]] && [[ "$test" =~ ^(HonkRecursionConstraintTest|ChonkRecursionConstraintTest|AvmRecursionInnerCircuitTests|AvmRecursionConstraintTest|AvmRecursiveTests\.TwoLayer|PaddingVariants/AvmRecursiveTestsParameterized\.TwoLayer|BoomerangTwoLayerAvmRecursiveVerifierTests|ECCVMRecursiveTests|GoblinRecursiveVerifierTests|GoblinAvmRecursiveVerifierTests|BoomerangGoblinRecursiveVerifierTests|BoomerangGoblinAvmRecursiveVerifierTests) ]]; then
253253
if [[ "$test" != "HonkRecursionConstraintTestWithoutPredicate/1.GenerateVKFromConstraints" ]]; then
254254
continue
255255
fi
256256
fi
257257
local prefix=$hash
258258
# A little extra resource for these tests.
259259
# IPARecursiveTests fails with 2 threads.
260-
if [[ "$test" =~ ^(AcirAvmRecursionConstraint|ChonkKernelCapacity|AvmRecursiveTests|IPARecursiveTests|HonkRecursionConstraintTest|ChonkRecursionConstraintTest) ]]; then
260+
if [[ "$test" =~ ^(AcirAvmRecursionConstraint|ChonkKernelCapacity|AvmRecursiveTests|AvmRecursionConstraintTest|IPARecursiveTests|HonkRecursionConstraintTest|ChonkRecursionConstraintTest|ECCVMRecursiveTests|GoblinRecursiveVerifierTests|GoblinAvmRecursiveVerifierTests|BoomerangGoblinRecursiveVerifierTests|BoomerangGoblinAvmRecursiveVerifierTests|BoomerangTwoLayerAvmRecursiveVerifierTests|PaddingVariants/AvmRecursiveTestsParameterized) ]]; then
261261
prefix="$prefix:CPUS=4:MEM=8g"
262262
fi
263263
echo -e "$prefix barretenberg/cpp/scripts/run_test.sh $bin_name $test"

0 commit comments

Comments
 (0)