Repository navigation
build(deps): bump the infra-base-images group across 2 directories with 4 updates #286
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -24,7 +24,7 @@ RUN sed -i 's/mcp = FastMCP("postgres-mcp")/from mcp.server.transport_security i | |
|
|
||
| RUN uv sync --frozen --no-dev | ||
|
|
||
| FROM public.ecr.aws/docker/library/python:3.12-slim-bookworm@sha256:6a0e06f5ba0b373d051aabd3fffb9b873f27f9a2a3a14bcbe37dee6f0e3bbc70 | ||
| FROM public.ecr.aws/docker/library/python:3.15-slim-bookworm@sha256:0e019bbdfb5d9874cb2389f1c2884b64aa2ac97c99960b5bc521cef978c0617d | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Keep both stages on the same Python version, or rebuild the environment for the new runtime. Prompt To Fix With AIThis is a comment left during a code review.
Path: src/agent_env/env/envs/service_db/Dockerfile.db-mcp
Line: 27
Comment:
**Database tools cannot start**
`Dockerfile.db-mcp` still builds `/app/.venv` with Python 3.12, but now copies it into a Python 3.15 image. The copied environment depends on the builder's Python 3.12 interpreter and packages. Copying `/app` does not install that interpreter in the final image, so `postgres-mcp` cannot start.
Keep both stages on the same Python version, or rebuild the environment for the new runtime.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly. |
||
| RUN groupadd -r app && useradd -r -g app app | ||
| COPY --from=builder --chown=app:app /app /app | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PostgreSQL 18 uses
/var/lib/postgresql/18/dockeras its defaultPGDATA, but_snapshot_servicedbstill copies/var/lib/postgresql/data. Snapshot creation therefore cannot export the database.That function also builds snapshots on PostgreSQL 16, which cannot read PostgreSQL 18 data files even after the copy path is fixed. Update snapshot capture and its base image together, or retain PostgreSQL 16.
Prompt To Fix With AI