Skip to content

fix(test): fix flaky duplicate_attestation_slash test#21753

Merged
PhilWindle merged 1 commit intomerge-train/spartanfrom
claudebox/fix-l1-tx-utils-ci
Mar 18, 2026
Merged

fix(test): fix flaky duplicate_attestation_slash test#21753
PhilWindle merged 1 commit intomerge-train/spartanfrom
claudebox/fix-l1-tx-utils-ci

Conversation

@AztecBot
Copy link
Collaborator

Summary

  • Added skipPushProposedBlocksToArchiver: true to both malicious node configs in duplicate_attestation_slash.test.ts
  • Without this flag, the second malicious node receives the first's block proposal via mock gossipsub and yields instead of building its own block, preventing the equivocation scenario needed for offense detection
  • This matches the pattern already used in duplicate_proposal_slash.test.ts which has the same flag with the comment: "Prevent HA peer proposals from being added to the archiver, so both malicious nodes build their own blocks instead of one yielding to the other"

Test plan

  • ./bootstrap.sh build yarn-project passes
  • CI runs duplicate_attestation_slash.test.ts successfully

Detailed analysis: https://gist.github.com/AztecBot/000ee6113d23edae3fae601304654698

ClaudeBox log: https://claudebox.work/s/4d7ac2d20eb5f07b?run=1

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Mar 18, 2026
@PhilWindle PhilWindle marked this pull request as ready for review March 18, 2026 17:47
@PhilWindle PhilWindle enabled auto-merge March 18, 2026 17:47
@PhilWindle PhilWindle merged commit b2c4fd2 into merge-train/spartan Mar 18, 2026
32 of 40 checks passed
@PhilWindle PhilWindle deleted the claudebox/fix-l1-tx-utils-ci branch March 18, 2026 17:58
github-merge-queue bot pushed a commit that referenced this pull request Mar 18, 2026
BEGIN_COMMIT_OVERRIDE
fix(p2p): fall back to maxTxsPerCheckpoint for per-block tx validation
(#21605)
chore: fixing M3 devcontainer builds (#21611)
fix: clamp finalized block to oldest available in world-state (#21643)
chore: fix proving logs script (#21335)
fix: (A-649) tx collector bench test (#21619)
fix(validator): process block proposals from own validator keys in HA
setups (#21603)
fix: add bounds when allocating arrays in deserialization (#21622)
fix: skip handleChainFinalized when block is behind oldest available
(#21656)
chore: demote finalized block skip log to trace (#21661)
fix: skip -march auto-detection for cross-compilation presets (#21356)
chore: revert "add bounds when allocating arrays in deserialization"
(#21622) (#21666)
fix: capture txs not available error reason in proposal handler (#21670)
fix: estimate gas in bot and make BatchCall.simulate() return
SimulationResult (#21676)
fix: prevent HA peer proposals from blocking equivocation in duplicate
proposal test (#21673)
fix(p2p): penalize peers for errors during response reading (#21680)
feat(sequencer): add build-ahead config and metrics (#20779)
chore: fixing build on mac (#21685)
fix: HA deadlock for last block edge case (#21690)
fix: process all contract classes in storeBroadcastedIndividualFunctions
(A-683) (#21686)
chore: add slack success post on nightly scenario (#21701)
fix(builder): persist contractsDB across blocks within a checkpoint
(#21520)
fix: only delete logs from rolled-back blocks, not entire tag (A-686)
(#21687)
chore(p2p): lower attestation pool per-slot caps to 2 (#21709)
chore(p2p): remove unused method (#21678)
fix(p2p): penalize peer on tx rejected by pool (#21677)
fix(test): workaround slow mock creation (#21708)
fix(sequencer): fix checkpoint budget redistribution for multi-block
slots (#21692)
fix: batch checkpoint unwinding in handleEpochPrune (A-690) (#21668)
fix(sequencer): add missing opts arg to checkpoint_builder tests
(#21733)
fix: race condition in fast tx collection (#21496)
fix: increase default postgres disk size from 1Gi to 10Gi (#21741)
fix: update batch_tx_requester tests to use RequestTracker (#21734)
chore: replace dead BOOTSTRAP_TO env var with bootstrap.sh build arg
(#21744)
fix(sequencer): extract gas and blob configs from valid requests only
(A-677) (#21747)
fix: deflake attempt for l1_tx_utils (#21743)
fix(test): fix flaky keystore reload test (#21749)
fix(test): fix flaky duplicate_attestation_slash test (#21753)
feat(pipeline): introduce pipeline views for building (#21026)
END_COMMIT_OVERRIDE
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