Skip to content

docs(todo): the smoke's HTTPS round on a Hugging Face 429; the GGUF cache's egress - #585

Merged
bernardladenthin merged 1 commit into
mainfrom
todo/ci-hf-429-and-cache-egress
Oct 27, 2026
Merged

bernardladenthin merged 1 commit into
mainfrom
todo/ci-hf-429-and-cache-egress

Conversation

@bernardladenthin

Copy link
Copy Markdown
Owner

Summary

Two CI weak points, noted in TODO.md ("Other open verification and maintenance"):

  • The smoke's HTTPS round depends on Hugging Face. Run 40554119477 (PR fix(patches): 0022 quantizes an expert's rows once; measured with real weights #583, re-run): in the linux-aarch64 smoke row round 1 passed, then round 2's --model-url download of stories260K.gguf got HTTP 429 and the server exited before becoming healthy. Fix: retry round 2 after a pause when the server log names a 429/5xx status, and fail only on TLS or connection errors or when the retries run out.
  • The 6.1 GB GGUF cache is restored about 20 times per run. In run 40580317285 (main) one restore failed with "Egress is over the account limit"; the other 14 restores in that run succeeded. The two Android jobs restore all of it just to keep the draft model. Recorded not to delete the entry, since a cache miss re-downloads from Hugging Face. The fix is a second, small entry. The repository's caches total 11.1 GB, which is over GitHub's 10 GB per repository.

Test plan

  • Numbers from the jobs and annotations of runs 40554119477 and 40580317285 and from gh cache list / actions/cache/usage.
  • CI is green on this branch
  • Docs updated

Related issues / PRs

Refs #583

Checklist

  • I have read CONTRIBUTING.md and CODE_OF_CONDUCT.md
  • My commits follow Conventional Commits
  • No security-sensitive changes

🤖 Generated with Claude Code

https://claude.ai/code/session_01DuAkLtHrCTx81vP5Mt4jad

…ache's egress

Run 40554119477: the linux-aarch64 smoke row failed when huggingface.co
answered 429 to round 2's download. Run 40580317285: one restore of the
6.1 GB model cache was refused with "Egress is over the account limit".
Both noted with the fix that keeps what they test; the cache must not be
deleted for it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DuAkLtHrCTx81vP5Mt4jad
@coderabbitai

coderabbitai Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Caution

Review failed

The pull request is closed.

Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@bernardladenthin
bernardladenthin merged commit 321c6b6 into main Oct 27, 2026
10 of 17 checks passed
@bernardladenthin
bernardladenthin deleted the todo/ci-hf-429-and-cache-egress branch October 27, 2026 21:51
@sonarqubecloud

Copy link
Copy Markdown

This branch had an error being deployed

1 failed deployment
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