Skip to content

Commit 180df7d

Browse files
committed
fix: disable AVM in nightly debug build and increase test timeout
The nightly barretenberg debug build has been failing since PR #21791 added test execution. Two issues: 1. The AVM=0 env var was not passed through to CMake, so vm2_tests were still being built and run. These AVM proving tests are too slow in debug mode (>10 min per test vs 600s timeout). 2. Even non-AVM tests can be slow in debug mode (-O0). Increase the per-test timeout from 600s to 1800s for the debug preset. Also includes a proposed workflow timeout increase (120→240 min) in .github-new/ since .github/ files are blocked.
1 parent cf1a239 commit 180df7d

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

barretenberg/cpp/bootstrap.sh

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,9 @@ function cmake_build {
5959
local preset=$1
6060
shift
6161
local cmake_args=()
62+
if [ "${AVM:-1}" -eq 0 ]; then
63+
cmake_args+=(-DAVM=OFF)
64+
fi
6265
if [ "${AVM_TRANSPILER:-1}" -eq 0 ]; then
6366
cmake_args+=(-DAVM_TRANSPILER_LIB=)
6467
fi
@@ -244,6 +247,10 @@ function test_cmds_native {
244247
grep -v 'DISABLED_' | \
245248
while read -r test; do
246249
local prefix=$hash
250+
# Debug builds are ~5-10x slower; give tests more time.
251+
if [[ "$native_preset" == "debug" ]]; then
252+
prefix="$prefix:TIMEOUT=1800s"
253+
fi
247254
# A little extra resource for these tests.
248255
# IPARecursiveTests fails with 2 threads.
249256
if [[ "$test" =~ ^(AcirAvmRecursionConstraint|ChonkKernelCapacity|AvmRecursiveTests|IPARecursiveTests|HonkRecursionConstraintTest) ]]; then

0 commit comments

Comments
 (0)