Skip to content

test: skip flaky sentinel_status_slash on merge-train/spartan#23443

Draft
AztecBot wants to merge 1 commit into
merge-train/spartanfrom
claudebox/skip-flaky-sentinel-status-slash
Draft

test: skip flaky sentinel_status_slash on merge-train/spartan#23443
AztecBot wants to merge 1 commit into
merge-train/spartanfrom
claudebox/skip-flaky-sentinel-status-slash

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

Summary

Skip sentinel_status_slash.parallel.test.ts to unblock merge-train/spartan.

Failure

Merge-train CI run http://ci.aztec-labs.com/1779299975599110 failed on:

FAIL src/e2e_p2p/sentinel_status_slash.parallel.test.ts
  e2e_p2p_sentinel_status_slash
    ✕ slashes the proposer with INACTIVITY when checkpoint validation records unvalidated

Expected: "checkpoint-unvalidated"
Received: "blocks-missed"

Test log: http://ci.aztec-labs.com/852ace33ed5b6f03

Root Cause

The test (added in #23286) is timing-sensitive. It spawns one malicious validator with broadcastInvalidBlockProposal: true and expects honest observers to:

  1. Receive the malicious invalid block proposal
  2. Reject it via re-execution state_mismatch
  3. Skip pushing to their archiver
  4. Receive the malicious checkpoint proposal and fail to validate it → record unvalidated

In the failing run, the observer recorded blocks-missed, meaning it never saw the malicious node's block proposal at all. Despite the warp-to-slot-before-target and mockGossipSubNetwork: true setup, the proposer-pipelining build phase for the target slot lands on the malicious node before the network has fully settled.

The test failed twice in a row (initial run + retry), so the existing e2e-p2p-epoch-flakes group didn't catch it.

Fix

Skip the test (assigned to palla) so the merge-train can advance while the timing issue is investigated. Follows the same approach used for epochs_l1_reorgs.parallel.test.ts.

Slack: https://aztecprotocol.slack.com/archives/C0AU8BULZHC/p1779301250083769

ClaudeBox log: https://claudebox.work/s/856f19a40ad6abca?run=1

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels May 20, 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