Skip to content

[SRE-5664] Phase 5h: Add Git and Git LFS to builder image#2

Merged
itskingori merged 2 commits intomainfrom
builder-phase5h-git-with-lfs
Mar 30, 2026
Merged

[SRE-5664] Phase 5h: Add Git and Git LFS to builder image#2
itskingori merged 2 commits intomainfrom
builder-phase5h-git-with-lfs

Conversation

@itskingori
Copy link
Copy Markdown
Member

@itskingori itskingori commented Mar 30, 2026

Adds git and git-lfs to the builder image and aligns the runtime user with the checkout flow used by the builder job. The new port-control-app builder implementation needs to support repositories that depend on Git LFS, but git-sync does the checkout only and does not hydrate LFS-managed objects.

This image change provides the tooling needed for the follow-up git lfs commands that will be introduced there. It also changes the builder runtime UID/GID to 65533 to match git-sync, which keeps permissions simple across the shared workspace.

Changes

  • Installs git and git-lfs in the builder image
  • Updates the documented image contents to include both tools
  • Changes the builder user and group UID/GID from 1001 to 65533
  • Adds a 1.1.0 changelog entry for Git/Git LFS and UID/GID alignment

Testing

  • docker build -t zappi-image-builder:test .
  • docker run --rm zappi-image-builder:test git --version
  • docker run --rm zappi-image-builder:test git lfs version
  • docker run --rm zappi-image-builder:test id

Related

@itskingori itskingori force-pushed the builder-phase5h-git-with-lfs branch from 3b6741a to 95e2369 Compare March 30, 2026 14:26
@itskingori itskingori marked this pull request as ready for review March 30, 2026 14:27
@itskingori itskingori requested a review from a team as a code owner March 30, 2026 14:27
@itskingori itskingori enabled auto-merge March 30, 2026 14:28
@itskingori itskingori merged commit 085efce into main Mar 30, 2026
1 check passed
@itskingori itskingori deleted the builder-phase5h-git-with-lfs branch March 30, 2026 14:31
@itskingori
Copy link
Copy Markdown
Member Author

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.

2 participants