Skip to content

fix: limit prover node to 1 pending job in long proving time test#20589

Closed
alexghr wants to merge 1 commit intomerge-train/spartanfrom
ag/fix-long-proving-time-flake
Closed

fix: limit prover node to 1 pending job in long proving time test#20589
alexghr wants to merge 1 commit intomerge-train/spartanfrom
ag/fix-long-proving-time-flake

Conversation

@alexghr
Copy link
Contributor

@alexghr alexghr commented Feb 17, 2026

Summary

  • Fixes intermittent timeout (~8% failure rate) in e2e_epochs/epochs_long_proving_time by setting proverNodeMaxPendingJobs: 1 in the test setup
  • When checkpoints land in different epochs due to timing, a second proof is needed but gets aborted when a newer epoch's proving job starts and triggers broker cleanup of the older epoch's data
  • Limiting to 1 pending job prevents the abort, matching the test's own assertion that maxJobCount === 1

Test plan

  • CI should pass the epochs_long_proving_time test consistently now

🤖 Generated with Claude Code

@alexghr alexghr enabled auto-merge (squash) February 17, 2026 13:53
The epochs_long_proving_time test intermittently times out (~8% failure rate)
because when checkpoints land in different epochs due to timing, a second proof
is needed but gets aborted by the broker when a newer epoch's proving job starts
and triggers cleanup of the older epoch's data. Setting proverNodeMaxPendingJobs
to 1 prevents new jobs from being created while one is in-flight, which matches
the test's own assertion that maxJobCount equals 1.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@AztecBot
Copy link
Collaborator

AztecBot commented Feb 18, 2026

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/3beca071305f9a62�3beca071305f9a628;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_p2p/duplicate_attestation_slash.test.ts (201s) (code: 0) group:e2e-p2p-epoch-flakes

@alexghr
Copy link
Contributor Author

alexghr commented Feb 18, 2026

superseded by #20617

@alexghr alexghr closed this Feb 18, 2026
auto-merge was automatically disabled February 18, 2026 11:19

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants