fix(ci): install Git LFS on self-hosted builds - #6056
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📜 Recent review details⏰ Context from checks skipped due to timeout. (1)
🔇 Additional comments (1)
📝 WalkthroughWalkthroughThe CI workflow now checks for Git LFS on trusted self-hosted runners before checkout. If Git LFS is missing and no cache is available, the workflow installs it with noninteractive APT commands. ChangesCI Git LFS setup
Estimated code review effort: 1 (Trivial) | ~5 minutes Mergeability Score: ⚪ Minimal · up to This localized CI workflow change installs Git LFS before checkout when needed, with no actionable merge-blocking risk remaining after normal checks and review. Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Code Review ✅ ApprovedInstalls OptionsAuto-apply is off → Gitar will not commit updates to this branch. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |
|



Summary
Install
git-lfson a self-hosted CI runner only when it is absent, beforeactions/checkoutperforms LFS smudging.Root cause
All CI-builds matrix legs stopped during checkout on
ci-vm-1because the workflow enableslfs: truewhile that runner image lacks thegit-lfsexecutable. The Docker build container never starts before checkout.Validation
.github/workflows/ci-builds.ymlas YAML.apt-get install --yes git-lfs.git diff --checkpasses.Summary by cubic
Installs
git-lfson self-hosted CI runners when missing, beforeactions/checkoutsmudges LFS. Previously, builds onci-vm-1failed during checkout due to missinggit-lfs; now checkout completes and Docker builds start.Review notes
self-hostedrunners withinputs.trustedand whensteps.cache-check.outputs.cache-hitis not true.git-lfsexists; otherwise runsapt-get updateand installsgit-lfsviasudonon-interactively.git-lfs.Written for commit 51aae4c. Summary will update on new commits.
Summary by CodeRabbit