Skip to content

fix(docs): retry forge build to handle transient solc download DNS failures#23345

Closed
AztecBot wants to merge 1 commit into
nextfrom
cb/f6be19f0ce50
Closed

fix(docs): retry forge build to handle transient solc download DNS failures#23345
AztecBot wants to merge 1 commit into
nextfrom
cb/f6be19f0ce50

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

@AztecBot AztecBot commented May 16, 2026

Closing in favor of #23346, which addresses the same DNS-retry issue but:

  • Targets merge-train/spartan (the train where the CI failure surfaced) instead of next.
  • Uses the existing ci3/retry helper (3 attempts × 5s sleep ≈ 10s total backoff) rather than an inline 5-attempt 5→80s loop, per Santiago's feedback to cap retries under 30s.
  • Also surfaces the per-subdir failure to the outer run_step via an $OUTPUT_DIR/.failed marker — the previous bug was not just "first attempt failed" but that the loop kept going and the function returned 0 even when no artifacts were written.

Original CI failure log: http://ci.aztec-labs.com/bc425a95fd076f0d

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label May 16, 2026
@AztecBot AztecBot closed this May 16, 2026
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.

1 participant