Skip to content

CI: Add a composite action for caching Git LFS objects and saving LFS bandwidth - #4877

Open
seisman wants to merge 3 commits into
mainfrom
ci/lfs
Open

CI: Add a composite action for caching Git LFS objects and saving LFS bandwidth#4877
seisman wants to merge 3 commits into
mainfrom
ci/lfs

Conversation

@seisman

@seisman seisman commented Sep 4, 2026

Copy link
Copy Markdown
Member

Address the LFS bandwidth quota issue reported in #4738 (comment).

This PR adds a composite action to cache Git LFS objects (i.e., baseline images) as GitHub cache, so that git lfs pull only pulls LFS objects when missing. It should help reduce our Git LFS bandwidth.

Comment thread .github/dependabot.yml
updates:
- package-ecosystem: "github-actions"
directory: "/"
directories:

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@seisman seisman changed the title CI: Cache Git LFS objects CI: Add a composite action to cache Git LFS objects and save LFS bandwidth Sep 4, 2026
@seisman seisman added maintenance Boring but important stuff for the core devs skip-changelog Skip adding Pull Request to changelog labels Sep 4, 2026
@seisman seisman added this to the 0.20.0 milestone Sep 4, 2026
@seisman
seisman marked this pull request as ready for review September 4, 2026 11:48
@seisman seisman added the needs review This PR has higher priority and needs review. label Sep 4, 2026
@seisman seisman changed the title CI: Add a composite action to cache Git LFS objects and save LFS bandwidth CI: Add a composite action for caching Git LFS objects and saving LFS bandwidth Sep 4, 2026
@seisman
seisman requested a lite review from Copilot September 4, 2026 12:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The workflow and action changes are internally consistent, align with existing local-action usage in the repo, and implement the stated LFS bandwidth reduction approach without introducing verified correctness issues.

Pull request overview

This PR introduces a local composite GitHub Action to cache Git LFS objects (baseline images) in GitHub Actions cache, then updates CI workflows to use it so git lfs pull only downloads missing objects and reduces LFS bandwidth usage.

Changes:

  • Add a composite action (checkout-lfs) that computes an LFS-object cache key and restores/caches .git/lfs before running git lfs pull.
  • Update CI workflows to disable actions/checkout LFS fetching and instead run the new composite action.
  • Update Dependabot configuration to also monitor GitHub Actions used under /.github/actions/*.
File summaries
File Description
.github/workflows/ci_tests.yaml Disables checkout LFS and adds a step to run the new checkout-lfs composite action.
.github/workflows/ci_tests_dev.yaml Same CI adjustment for the dev CI workflow matrix.
.github/dependabot.yml Expands Dependabot’s GitHub Actions update scope to include local action directories.
.github/actions/checkout-lfs/action.yml New composite action that caches/restores .git/lfs and runs git lfs pull.
Review details
  • Files reviewed: 4/4 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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

Labels

maintenance Boring but important stuff for the core devs needs review This PR has higher priority and needs review. skip-changelog Skip adding Pull Request to changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants