fix(ci): hydrate LFS dependencies in builds - #6055
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🧰 Additional context used🧠 Learnings (1)📚 Learning: 2026-07-08T13:19:04.649ZApplied to files:
🔇 Additional comments (1)
📝 WalkthroughWalkthroughThe CI workflow now enables Git LFS in the repository checkout step while preserving the existing checkout settings. ChangesCI Git LFS checkout
Estimated code review effort: 1 (Trivial) | ~2 minutes Mergeability Score: ⚪ Minimal · up to This localized CI change enables required LFS-backed dependencies during builds, addressing the reported checkout failure. No actionable merge-blocking risk remains beyond normal checks and review. Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsGit: Failed to clone repository. Please run the 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 ✅ ApprovedEnables Git LFS hydration in the reusable CI-builds workflow checkout step to prevent AWS SDK bundle checksum failures. No issues found. 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 |
|



What changed
Enable Git LFS hydration in the reusable CI-builds checkout.
Root cause
The AWS SDK source archive is intentionally stored in Git LFS. CI-builds checked out pull-request commits with
lfs: false, leaving the pointer file in place. The vendored bundle checksum then failed before ProxySQL or its tests could build.Validation
lfs: true.git diff --checkpasses.This PR targets
GH-Actions; it is required before rerunning the standard CI-builds jobs for PR #6048.Summary by cubic
Hydrates Git LFS files during CI builds so the AWS SDK archive is present. Previously
actions/checkoutused lfs: false and left pointer files, causing vendored bundle checksum failures; now it sets lfs: true so builds and tests proceed.actions/checkoutstep in .github/workflows/ci-builds.yml.Written for commit fc4ba9d. Summary will update on new commits.
Summary by CodeRabbit