Skip to content

fix: bump timeout and resources for AVM recursion tests in debug builds#22488

Closed
AztecBot wants to merge 1 commit into
nextfrom
claudebox/fix-bb-debug-build
Closed

fix: bump timeout and resources for AVM recursion tests in debug builds#22488
AztecBot wants to merge 1 commit into
nextfrom
claudebox/fix-bb-debug-build

Conversation

@AztecBot

@AztecBot AztecBot commented Apr 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fixes nightly barretenberg debug build failure (CI run).

AvmRecursionInnerCircuitTests.Tampering was timing out at the default 600s limit. The test runs 3 iterations of prove+verify (~3.5 minutes each in debug mode), totaling ~10 minutes — well over the 600s default. Exit code 124 (timeout).

Root cause: The test name AvmRecursionInnerCircuitTests was not matched by the existing regex patterns in barretenberg/cpp/bootstrap.sh that grant extra resources and extended timeouts.

Fix:

  • Add AvmRecursionInnerCircuitTests to both the resource (CPUS=4:MEM=8g) and timeout (TIMEOUT=900s) regex patterns
  • Add AvmRecursionConstraintTest to the resource regex (uses up to 7.5GB memory in debug builds)

Test plan

  • Debug build compiles locally (all 1111 targets)
  • Verified test name matches updated regex patterns
  • CI log analysis confirms timeout was the only failure

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Apr 12, 2026
@AztecBot AztecBot force-pushed the claudebox/fix-bb-debug-build branch from f5de5f4 to 64e3d4a Compare April 13, 2026 06:10
@AztecBot AztecBot changed the title fix: skip heavy recursion tests in nightly debug builds fix: disable AVM in nightly debug build and increase test timeout Apr 13, 2026
AvmRecursionInnerCircuitTests.Tampering was timing out at 600s in the nightly
debug build. The test runs 3 iterations of prove+verify (~3.5min each in debug
mode), exceeding the default timeout.

Add AvmRecursionInnerCircuitTests to the TIMEOUT=900s list and to the
CPUS=4:MEM=8g resource list. Also add AvmRecursionConstraintTest to
the resource list since it uses up to 7.5GB of memory.
@AztecBot AztecBot force-pushed the claudebox/fix-bb-debug-build branch from 64e3d4a to 75a1125 Compare April 13, 2026 06:20
@AztecBot AztecBot changed the title fix: disable AVM in nightly debug build and increase test timeout fix: bump timeout and resources for AVM recursion tests in debug builds Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants