Skip to content

test: retry tests that flake on transient RPC connection drops#1249

Merged
MoonBoi9001 merged 1 commit into
mainfrom
mb9/retry-tests-hit-by-transient-network-errors
Jul 13, 2026
Merged

test: retry tests that flake on transient RPC connection drops#1249
MoonBoi9001 merged 1 commit into
mainfrom
mb9/retry-tests-hit-by-transient-network-errors

Conversation

@MoonBoi9001

Copy link
Copy Markdown
Member

This PR makes failed tests retry up to 2 times in the 4 indexer-common suites that talk to a live Arbitrum Sepolia RPC endpoint (tap, cost-models, indexing-rules, and poi-disputes), so a momentary connection drop no longer fails a CI run. Each failure is still logged before retrying, so the flakes stay visible in job logs rather than being silently absorbed, and a genuinely broken test still fails after 3 attempts.

The motivation is concrete: over the last 2 days, 4 CI node jobs across 3 runs on 2 unrelated PRs failed with transient transport errors from that endpoint (socket hang up in poi-disputes, ECONNRESET in indexing-rules), each time in a different test, and each time green on rerun. Retries happen through jest, which re-runs the per-test setup hooks, so each attempt starts from the same clean database state the first attempt had.

4 suites call a live Arbitrum Sepolia RPC endpoint, and its transient connection drops
(socket hang up, ECONNRESET) have been failing CI runs on unrelated PRs. Failed tests in
those suites now retry up to 2 times, with each error still logged before retrying.
@github-project-automation github-project-automation Bot moved this to 🗃️ Inbox in Indexer Jul 10, 2026
@MoonBoi9001
MoonBoi9001 marked this pull request as ready for review July 11, 2026 04:17
@MoonBoi9001 MoonBoi9001 added the CI label Jul 13, 2026
@github-project-automation github-project-automation Bot moved this from 🗃️ Inbox to ✅ Approved in Indexer Jul 13, 2026
@MoonBoi9001
MoonBoi9001 merged commit 7fe61cb into main Jul 13, 2026
11 checks passed
@MoonBoi9001
MoonBoi9001 deleted the mb9/retry-tests-hit-by-transient-network-errors branch July 13, 2026 20:41
@github-project-automation github-project-automation Bot moved this from ✅ Approved to 🚗 Merged in Indexer Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: 🚗 Merged

Development

Successfully merging this pull request may close these issues.

2 participants