Skip to content

fix(ci): retry sqlite3mc-wasm download on transient DNS/TLS failures#23333

Merged
spalladino merged 1 commit into
merge-train/spartanfrom
claudebox/requeue-pr-23253-fee-settings
May 16, 2026
Merged

fix(ci): retry sqlite3mc-wasm download on transient DNS/TLS failures#23333
spalladino merged 1 commit into
merge-train/spartanfrom
claudebox/requeue-pr-23253-fee-settings

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

Motivation

The merge-train/spartan train PR (#23253) was dequeued from the merge queue this morning because grind run x9 failed during compile_all:

==> Downloading sqlite3mc-2.2.4-sqlite-3.50.4-wasm.zip
curl: (6) Could not resolve host: release-assets.githubusercontent.com

CI logs:

The branch CI on the same commit (run 25958983932) passed — only one of the 10 grind shards hit the DNS flake, but the merge-queue fail-fast tore the whole run down. The other 9 grinds and the ARM run were still pending when the queue dropped #23253.

Approach

Add curl retry flags to yarn-project/sqlite3mc-wasm/scripts/vendor.sh so a one-off Could not resolve host (or any other transient curl failure) doesn't fail the build. --retry 5 --retry-delay 2 --retry-all-errors --retry-connrefused gives ~10s of total backoff, which is plenty for a momentary DNS hiccup but bounded for genuine outages.

This is the only curl in the yarn-project build path that hits GitHub release assets, so this is a targeted fix rather than a sweep.

Verification

./bootstrap.sh ci requires EC2 spawn and isn't runnable from inside the container. Locally verified that vendor.sh ensure still downloads and validates the pinned artifact correctly.

ClaudeBox log: https://claudebox.work/s/89dacb14037285cd?run=1

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels May 16, 2026
@AztecBot
Copy link
Copy Markdown
Collaborator Author

Flakey Tests

🤖 says: This CI run detected 2 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/fcf14f1f9ad9a117�fcf14f1f9ad9a1178;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_p2p/preferred_gossip_network.test.ts (213s) (code: 0) group:e2e-p2p-epoch-flakes
\033FLAKED\033 (8;;http://ci.aztec-labs.com/14a2168e27865c3e�14a2168e27865c3e8;;�): yarn-project/kv-store/scripts/run_test.sh src/sqlite-opfs/internal/ordered-binary-browser.test.ts (2s) (code: 0)

@spalladino spalladino marked this pull request as ready for review May 16, 2026 13:24
@spalladino spalladino enabled auto-merge (squash) May 16, 2026 13:24
@spalladino spalladino merged commit f0578c8 into merge-train/spartan May 16, 2026
55 of 61 checks passed
@spalladino spalladino deleted the claudebox/requeue-pr-23253-fee-settings branch May 16, 2026 13:24
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.

2 participants