Skip to content

test: flag e2e_ha_full afterAll hook timeout as flake under pipelining#23524

Merged
alexghr merged 1 commit into
merge-train/spartanfrom
cb/22301e7d0daa
May 23, 2026
Merged

test: flag e2e_ha_full afterAll hook timeout as flake under pipelining#23524
alexghr merged 1 commit into
merge-train/spartanfrom
cb/22301e7d0daa

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

Summary

Add .test_patterns.yml entry to flag e2e_ha_full.test.ts failing with Exceeded 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

Jest output:

FAIL src/composed/ha/e2e_ha_full.test.ts (1519.394 s)
  ● Test suite failed to run

    thrown: "Exceeded timeout of 1200000 ms for a hook.

     279 |   });
     280 |
    > 281 |   afterAll(async () => {
         |   ^

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_full entries match per-test assertion lines (✕ should coordinate governance voting..., ✕ should distribute work across multiple HA nodes). The afterAll-timeout failure mode surfaces as Test suite failed to run with 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 observed Exceeded timeout of 1200000 ms for a hook line from the failure
  • CI continues to run src/composed/ha/e2e_ha_full.test.ts; the afterAll timeout now alerts palla on #aztec3-ci instead of failing the build

Created by claudebox · group: slackbot

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label May 23, 2026
@alexghr alexghr marked this pull request as ready for review May 23, 2026 07:52
@alexghr alexghr enabled auto-merge (squash) May 23, 2026 07:52
@alexghr alexghr merged commit afbeb1a into merge-train/spartan May 23, 2026
39 of 50 checks passed
@alexghr alexghr deleted the cb/22301e7d0daa branch May 23, 2026 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-skip 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