Skip to content

fix: backport #21673 — prevent HA peer proposals from blocking equivocation in duplicate proposal test#21693

Open
AztecBot wants to merge 6 commits intobackport-to-v4-next-stagingfrom
claudebox/backport-21673-v4-next
Open

fix: backport #21673 — prevent HA peer proposals from blocking equivocation in duplicate proposal test#21693
AztecBot wants to merge 6 commits intobackport-to-v4-next-stagingfrom
claudebox/backport-21673-v4-next

Conversation

@AztecBot
Copy link
Collaborator

@AztecBot AztecBot commented Mar 17, 2026

Summary

Backport of #21673 to v4-next.

Adds dontStartSequencer: true and skipPushProposedBlocksToArchiver: true to the malicious nodes in the duplicate_proposal_slash e2e test. This prevents HA peer proposals from being added to the archiver, so both malicious nodes build their own blocks instead of one yielding to the other.

Note: v4-next also needed dontStartSequencer: true (from PR #21603) which was already on next when the original PR was merged, so this backport includes both properties.

Conflict Resolution

The cherry-pick conflicted because v4-next lacked the dontStartSequencer: true line added by PR #21603. Resolution was to accept both new properties (dontStartSequencer and skipPushProposedBlocksToArchiver).

Commits

  1. Cherry-pick with conflicts (original attempt)
  2. Conflict resolution — accept both new properties

Test plan

  • duplicate_proposal_slash e2e test should pass with these changes
  • No other tests affected (only malicious nodes in this specific test are changed)

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Mar 17, 2026
@AztecBot AztecBot force-pushed the claudebox/backport-21673-v4-next branch from ae4789b to f6ccf4e Compare March 17, 2026 17:51
@AztecBot AztecBot changed the title fix: prevent HA peer proposals from blocking equivocation in duplicate proposal test (#21673) fix: backport #21673 — prevent HA peer proposals from blocking equivocation in duplicate proposal test Mar 17, 2026
@spalladino spalladino added the ci-draft Run CI on draft PRs. label Mar 18, 2026
@spalladino spalladino marked this pull request as ready for review March 18, 2026 12:45
@spalladino spalladino enabled auto-merge (squash) March 18, 2026 12:53
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.

3 participants