Skip to content

hotfix(deploy): install git in Lambda Dockerfile#160

Merged
cipher813 merged 1 commit into
mainfrom
hotfix/dockerfile-install-git
May 5, 2026
Merged

hotfix(deploy): install git in Lambda Dockerfile#160
cipher813 merged 1 commit into
mainfrom
hotfix/dockerfile-install-git

Conversation

@cipher813
Copy link
Copy Markdown
Owner

Summary

The Phase 2 Lambda deploy job has been failing on:
```
ERROR: Cannot find command 'git' - do you have 'git' installed and in your PATH?
```

The Lambda Python 3.12 base image doesn't ship with git, but the Dockerfile uses `pip install ... @ git+https://...` for alpha-engine-lib. Surfaced today by PR #159's post-merge Deploy run against a fresh build.

Fix

Adds `RUN microdnf install -y git && microdnf clean all` before the pip install. Same one-line fix applied to alpha-engine-research Dockerfile after PR #105's lib-public flip exposed the same gap there.

Test plan

  • Diff parity with alpha-engine-research Dockerfile (line-for-line same install)
  • Deploy workflow re-runs cleanly post-merge

Out of scope

The `FromPlatformFlagConstDisallowed` warning on line 1 (--platform=linux/amd64) is a separate buildkit lint — doesn't block builds, leave for follow-up.

🤖 Generated with Claude Code

The Phase 2 Lambda deploy job has been failing on:

  ERROR: Cannot find command 'git' - do you have 'git' installed and
  in your PATH?

The Lambda Python 3.12 base image (public.ecr.aws/lambda/python:3.12)
doesn't ship with git, but the Dockerfile uses `pip install ... @
git+https://...` which requires it. Surfaced today when PR #159's
post-merge Deploy fired against a fresh build (vs the prior in-flight
image which had a cached install layer that masked the gap).

Fix: same one-line microdnf install applied to alpha-engine-research
Dockerfile after PR #105's lib-public flip. AL2023 minimal package
manager; image-size impact ~25MB.

Out of scope: the FromPlatformFlagConstDisallowed warning (line 1)
about `--platform=linux/amd64` is a separate buildkit lint that
doesn't block builds — leave for a follow-up.

Test plan
- [x] Diff parity with alpha-engine-research Dockerfile (same line)
- [ ] Deploy workflow re-runs cleanly post-merge

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@cipher813 cipher813 merged commit b9528a8 into main May 5, 2026
1 check passed
@cipher813 cipher813 deleted the hotfix/dockerfile-install-git branch May 5, 2026 18:53
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