Skip to content

ci: retry a dependency fetch that failed on DNS or a server error - #553

Merged
foxnne merged 3 commits into
mainfrom
ci/retry-dns-failure
Oct 18, 2026
Merged

foxnne merged 3 commits into
mainfrom
ci/retry-dns-failure

Conversation

@foxnne

@foxnne foxnne commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

What changes

scripts/ci-retry.sh now retries a dependency fetch whose DNS lookup failed.

On #549 (Zig 0.17), Tests (Linux) failed twice, about 15 minutes apart, fetching codeberg's translate-c (a git+https dependency under dvui):

error: unable to discover remote git server capabilities: NameServerFailure

The script matched only TemporaryNameServerFailure, so it printed "not a fetch failure — not retrying" and failed on the first attempt. Codeberg resolved fine from a laptop at the same time, and other runs with a warm cache never fetched it. The pattern is now NameServerFailure, which covers both, plus the git fetch's own wording, unable to discover remote git server capabilities. It also retries a server error (bad HTTP response code: '5xx): an SDL fetch on #549 failed with '500 Internal Server Error' and was not retried. A 4xx, a failing test or a compile error still fails on its first attempt.

SDK impact

  • None
  • Core-only or additive: reaches plugins at the next SDK release
  • Fingerprint moved: recorded in sdk/src/version.zig, sdk_version untouched, PR labelled sdk
  • SDK release: bumps sdk_version, lists the sdk PRs since the last sdk-v* tag

Verified

  • bash -n scripts/ci-retry.sh passes.
  • The new patterns match the exact lines from both failed logs, and a 404 does not match.
  • Not seen retrying in CI: that needs the DNS failure to happen again.

Follow-ups

🤖 Generated with Claude Code

A git+https dependency (codeberg's translate-c, under dvui on Zig 0.17) failed
with `NameServerFailure` twice on a Linux runner. The retry script matched only
`TemporaryNameServerFailure`, so it gave up on the first attempt.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@foxnne
foxnne enabled auto-merge (squash) October 18, 2026 15:28
An SDL fetch on #549 failed with `bad HTTP response code: '500 Internal Server
Error'`, which the script did not count as a fetch failure. A 4xx still fails
at once.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@foxnne foxnne changed the title ci: retry a dependency fetch whose DNS lookup failed ci: retry a dependency fetch that failed on DNS or a server error Oct 18, 2026
@foxnne
foxnne merged commit 46874b5 into main Oct 18, 2026
11 checks passed
@foxnne
foxnne deleted the ci/retry-dns-failure branch October 18, 2026 17:01
@foxnne foxnne mentioned this pull request Oct 18, 2026
2 of 3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant