Skip to content

test(ci): skip e2e_multi_validator_node_key_store entirely on merge-train/spartan#23544

Merged
PhilWindle merged 1 commit into
merge-train/spartanfrom
cb/dc484e7becc6
May 24, 2026
Merged

test(ci): skip e2e_multi_validator_node_key_store entirely on merge-train/spartan#23544
PhilWindle merged 1 commit into
merge-train/spartanfrom
cb/dc484e7becc6

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

Summary

PR #23344 was dequeued again at https://github.com/AztecProtocol/aztec-packages/actions/runs/26370196367. The flake-flag added in #23528 (error_regex: "Tx dropped by P2P node") wasn't enough — ci3 retries once and the multi-validator web3signer suite has now been seen failing in a second pipelining-related mode that the regex doesn't match.

Both observed modes hit the same should build blocks & attest with multiple validator keys case:

  • Tx dropped by P2P node at Promise.all index 0 (the original mode, ci.aztec-labs.com/9a5fa90aa18f62e7).
  • Proposer misses the slot's 5-attestation deadline → AttestationTimeoutError: Timeout collecting attestations for slot 29: 0/5, followed by Simulation error: Block 0x… not found in the node. This might indicate a reorg has occurred. (ci.aztec-labs.com/b91d3218b5e88ae4).

Following the same pattern as #23542 for e2e_ha_full: collapse the existing error_regex entry into a single skip: true so the suite is outright skipped on merge-train/spartan until proposer pipelining stabilises.

Notes

Requested in Slack by @PhilWindle.


Created by claudebox · group: slackbot

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label May 24, 2026
@PhilWindle PhilWindle marked this pull request as ready for review May 24, 2026 20:33
@PhilWindle PhilWindle merged commit 26aa7c1 into merge-train/spartan May 24, 2026
26 of 31 checks passed
@PhilWindle PhilWindle deleted the cb/dc484e7becc6 branch May 24, 2026 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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