Skip to content

Superseded by #24137: raise AVM check-circuit timeout#23662

Closed
AztecBot wants to merge 2 commits into
nextfrom
cb/avm-check-circuit-timeout
Closed

Superseded by #24137: raise AVM check-circuit timeout#23662
AztecBot wants to merge 2 commits into
nextfrom
cb/avm-check-circuit-timeout

Conversation

@AztecBot

@AztecBot AztecBot commented May 29, 2026

Copy link
Copy Markdown
Collaborator

Superseded by #24137, which carries the same AVM check-circuit timeout fix rebased onto current next and includes the standard CI labels.

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label May 29, 2026
@AztecBot AztecBot changed the title fix(ci): raise AVM check-circuit input timeout fix(ci): raise AVM check-circuit per-tx timeout to 120s Jun 1, 2026
@AztecBot AztecBot force-pushed the cb/avm-check-circuit-timeout branch from b6a8894 to e0f791e Compare June 1, 2026 13:29
AztecBot added a commit that referenced this pull request Jun 1, 2026
The avm-check-circuit job runs bb-avm avm_check_circuit on every dumped
e2e AVM input in parallel, each wrapped in a 30s timeout (exec_test's
timeout -v $TIMEOUT). The runner uses --halt now,fail=1, so a single
timeout fails the whole job.

The e2e_multiple_blobs tx produces a ~700k-row AVM trace. On the default
2 CPUs, trace generation (~22s) plus the row check exceeded 30s and the
check was killed with exit 124 (CI run 26755632012); every other input
passed in 3-6s.

Raise the per-check timeout to a 120s default and make it overridable via
AVM_CHECK_CIRCUIT_TIMEOUT, so the heaviest inputs complete with margin
while the common case still finishes quickly. CPU allocation stays at the
default 2 (the runner core count is tuned so the parallel job count
saturates it at 2 CPUs each); only wall-clock budget was the constraint.

Supersedes the stale draft branch for #23662 (rebased onto current next).
@AztecBot AztecBot added the ci-draft Run CI on draft PRs. label Jun 3, 2026
@AztecBot AztecBot force-pushed the cb/avm-check-circuit-timeout branch 2 times, most recently from 380065d to ffebd3a Compare June 11, 2026 05:04
AztecBot added a commit that referenced this pull request Jun 14, 2026
@AztecBot AztecBot force-pushed the cb/avm-check-circuit-timeout branch from ffebd3a to 24f9452 Compare June 14, 2026 05:10
@AztecBot AztecBot force-pushed the cb/avm-check-circuit-timeout branch from 24f9452 to 0713a8d Compare June 16, 2026 15:14
@AztecBot AztecBot changed the title fix(ci): raise AVM check-circuit per-tx timeout to 120s Superseded by #24137: raise AVM check-circuit timeout Jun 16, 2026
@AztecBot AztecBot closed this Jun 16, 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.

1 participant