Skip to content

Commit 2eaf948

Browse files
committed
fix: increase test timeout for heavy recursion tests in debug build
HonkRecursionConstraintTest and ChonkRecursionConstraintTest routinely take 400-600s in debug builds, hitting the 600s default timeout. Bump to 900s. Also add ChonkRecursionConstraintTest to the CPUS=4:MEM=8g resource group. This is a direct cherry-pick of the fix from PR #22347 (already on merge-train/barretenberg) to unblock the nightly debug build which has been failing for 6+ consecutive days.
1 parent d7934b9 commit 2eaf948

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

barretenberg/cpp/bootstrap.sh

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -248,9 +248,13 @@ function test_cmds_native {
248248
local prefix=$hash
249249
# A little extra resource for these tests.
250250
# IPARecursiveTests fails with 2 threads.
251-
if [[ "$test" =~ ^(AcirAvmRecursionConstraint|ChonkKernelCapacity|AvmRecursiveTests|IPARecursiveTests|HonkRecursionConstraintTest) ]]; then
251+
if [[ "$test" =~ ^(AcirAvmRecursionConstraint|ChonkKernelCapacity|AvmRecursiveTests|IPARecursiveTests|HonkRecursionConstraintTest|ChonkRecursionConstraintTest) ]]; then
252252
prefix="$prefix:CPUS=4:MEM=8g"
253253
fi
254+
# These tests routinely take 400-600s in debug builds; bump from the 600s default.
255+
if [[ "$test" =~ ^(HonkRecursionConstraintTest|ChonkRecursionConstraintTest) ]]; then
256+
prefix="$prefix:TIMEOUT=900s"
257+
fi
254258
echo -e "$prefix barretenberg/cpp/scripts/run_test.sh $bin_name $test"
255259
done || (echo "Failed to list tests in $bin" && exit 1)
256260
done

0 commit comments

Comments
 (0)