test: flag e2e_ha_full afterAll hook timeout as flake under pipelining#23524
Merged
Conversation
alexghr
approved these changes
May 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add
.test_patterns.ymlentry to flage2e_ha_full.test.tsfailing withExceeded timeout of 1200000 ms for a hook(afterAll teardown) as a flake. This is what dequeued PR #23344 from the merge queue at 2026-05-22T22:07:25Z.Investigation
cijob: https://github.com/AztecProtocol/aztec-packages/actions/runs/26312443012/job/77464166767Jest output:
The teardown of the HA full test exceeds the 20-minute hook timeout under pipelining. The suite was migrated to pipelining in #23463; this is a sibling flake of #23501 and #23515.
Why a new entry?
The existing
e2e_ha_fullentries match per-test assertion lines (✕ should coordinate governance voting...,✕ should distribute work across multiple HA nodes). The afterAll-timeout failure mode surfaces asTest suite failed to runwith no per-test assertion line, so neither existing entry catches it and the failure escalates to FAILED.Follow-up
Proper source fix is to tighten the HA full afterAll teardown so it does not block on services that have already hung. This patch is the quick unblock; owner palla is the author of the pipelining migration (#23463).
Test plan
error_regex: "Exceeded timeout of [0-9]+ ms for a hook"matches the observedExceeded timeout of 1200000 ms for a hookline from the failuresrc/composed/ha/e2e_ha_full.test.ts; the afterAll timeout now alerts palla on#aztec3-ciinstead of failing the buildCreated by claudebox · group:
slackbot