fix(ci): increase e2e_prover_full_real timeout after circuit changes#22100
Merged
Conversation
The merge-next brought circuit gate count increases (CHONK +577, ECCVM +579) and AVM circuit complexity changes from SHA256 PIL refactoring. These combined to push the e2e_prover_full_real test just over its previous 20-minute CI timeout. - CI timeout: 20m → 40m - Jest timeout: 15m → 35m - beforeAll timeout: 2m → 5m Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
e2e_prover_full_realCI timeout from 20m to 40m inend-to-end/bootstrap.shTIMEOUTconstant from 15m (900s) to 35m (2100s) infull.test.tsbeforeAllJest timeout from 2m to 5mRoot Cause
The
merge-nextPR (#22089) merged 111 commits intospartanthat introduced multiple cumulative performance regressions for the full prover test:bitwise.pil,memory.pil,precomputed.pil)padding_indicator_arraymultiplication per roundThese changes collectively push the test from ~17-18 minutes to over 20 minutes, causing it to be killed by the 20-minute OS timeout.
Test plan
merge-train/spartanwith this fixe2e_prover_full_realcompletes within the new 40-minute timeout🤖 Generated with Claude Code