Skip to content

fix(ci): install Git LFS on self-hosted builds - #6056

Merged
renecannao merged 1 commit into
GH-Actionsfrom
fix/ci-install-git-lfs
Aug 13, 2026
Merged

fix(ci): install Git LFS on self-hosted builds#6056
renecannao merged 1 commit into
GH-Actionsfrom
fix/ci-install-git-lfs

Conversation

@renecannao

@renecannao renecannao commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary

Install git-lfs on a self-hosted CI runner only when it is absent, before actions/checkout performs LFS smudging.

Root cause

All CI-builds matrix legs stopped during checkout on ci-vm-1 because the workflow enables lfs: true while that runner image lacks the git-lfs executable. The Docker build container never starts before checkout.

Validation

  • Parsed .github/workflows/ci-builds.yml as YAML.
  • Asserted the guarded install step precedes checkout and invokes apt-get install --yes git-lfs.
  • git diff --check passes.

Summary by cubic

Installs git-lfs on self-hosted CI runners when missing, before actions/checkout smudges LFS. Previously, builds on ci-vm-1 failed during checkout due to missing git-lfs; now checkout completes and Docker builds start.

Review notes

  • Adds a pre-checkout step gated to self-hosted runners with inputs.trusted and when steps.cache-check.outputs.cache-hit is not true.
  • The step no-ops if git-lfs exists; otherwise runs apt-get update and installs git-lfs via sudo non-interactively.
  • GitHub-hosted runners are unchanged; they already include git-lfs.

Written for commit 51aae4c. Summary will update on new commits.

Review in cubic

Summary by CodeRabbit

  • Chores
    • Improved self-hosted build setup by automatically installing Git LFS when it is unavailable.
    • Added verification to confirm the installed Git LFS version before continuing.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 73644261-7aa0-4f7d-b543-2caefd9c9ae6

📥 Commits

Reviewing files that changed from the base of the PR and between 67def10 and 51aae4c.

📒 Files selected for processing (1)
  • .github/workflows/ci-builds.yml
📜 Recent review details
⏰ Context from checks skipped due to timeout. (1)
  • GitHub Check: Gitar
🔇 Additional comments (1)
.github/workflows/ci-builds.yml (1)

206-220: LGTM!


📝 Walkthrough

Walkthrough

The 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.

Changes

CI Git LFS setup

Layer / File(s) Summary
Self-hosted runner Git LFS preflight
.github/workflows/ci-builds.yml
The workflow skips installation when git-lfs is available. Otherwise, it installs Git LFS before checkout and reports the installed version.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Mergeability Score: ⚪ Minimal · up to 51aae

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

A rabbit checks the runner's track,
Finds Git LFS—or brings it back.
APT hops in with careful feet,
Then checkout makes the repo complete.
CI bounds onward, neat and sweet.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main CI change: installing Git LFS on self-hosted builds.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ci-install-git-lfs

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.

❤️ Share

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

@gitar-bot

gitar-bot Bot commented Aug 13, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Installs git-lfs conditionally on self-hosted runners prior to checkout, fixing CI build failures on ci-vm-1. No issues found.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@renecannao
renecannao marked this pull request as ready for review August 13, 2026 15:13
@renecannao
renecannao merged commit 735ddf5 into GH-Actions Aug 13, 2026
2 checks passed
@sonarqubecloud

Copy link
Copy Markdown

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