Skip to content

ci(hf): use temporary repos per CI job to avoid concurrent commit races#7370

Open
kszucs wants to merge 1 commit intoapache:mainfrom
kszucs:hf-temp-repo
Open

ci(hf): use temporary repos per CI job to avoid concurrent commit races#7370
kszucs wants to merge 1 commit intoapache:mainfrom
kszucs:hf-temp-repo

Conversation

@kszucs
Copy link
Copy Markdown
Member

@kszucs kszucs commented Apr 8, 2026

Summary

  • HF dataset/bucket behavior tests were failing with HTTP 412 errors because multiple CI jobs (core, Go, Node.js, Java) committed to the same shared git-based repo simultaneously, causing branch update races.
  • Added a hf-temp-repo GitHub Action (node24, zero dependencies) that creates a temporary HF repo/bucket per CI job and auto-deletes it via a post-run hook.
  • Updated hf_dataset and hf_bucket service actions to use unique per-job repos instead of shared ones from 1Password.
  • Removed unused huggingface read-only service action.

Test plan

  • Verify hf_dataset CI jobs create isolated repos and no longer race
  • Verify hf_bucket CI jobs create isolated buckets
  • Verify post-run cleanup deletes temporary repos/buckets after job completion

@kszucs kszucs requested review from Xuanwo and tisonkun as code owners April 8, 2026 13:11
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" labels Apr 8, 2026
HF dataset tests were failing with 412 errors because multiple CI jobs
(core, Go, Node.js, Java) committed to the same shared git-based repo
simultaneously. Each job now creates its own temporary HF repo/bucket
and deletes it after the job via a node20 action post-run hook.
@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Apr 8, 2026

I will merge this PR into my PR directly so that we can make the CI fully pass (with your commit inside)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants