Skip to content

Fix Docker build failure due to missing source files in builder stage #18

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.

Sign up for GitHub

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

Merged
royisme merged 2 commits into from
Nov 6, 2025
Merged

Fix Docker build failure due to missing source files in builder stage #18

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
8fd2c52
Initial plan
Copilot Nov 6, 2025
219e826
fix: Fix Docker build by copying source files before package installa…
Copilot Nov 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 10 additions & 2 deletions docker/Dockerfile.full
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,17 @@ RUN pip install uv

WORKDIR /app

# Copy source files needed for package installation
COPY pyproject.toml ./
# Create dummy README for pip install (pyproject.toml references it)
RUN touch README.md && uv pip install --system -e .
COPY api ./api
COPY core ./core
COPY services ./services
COPY monitoring ./monitoring
COPY mcp_tools ./mcp_tools
COPY start.py start_mcp.py mcp_server.py config.py main.py ./

# Install the package and its dependencies
RUN uv pip install --system .
Comment on lines +27 to +28
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot Nov 6, 2025

Choose a reason for hiding this comment

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

P1 Badge Restore README copy before installing package

The builder stage now copies pyproject.toml and the source packages before uv pip install --system ., but it no longer creates or copies README.md even though pyproject.toml declares readme = "README.md". During the install step setuptools will attempt to read that file and the build will fail with FileNotFoundError. Consider copying README.md (or reinstating the temporary file) before running the install. The same problem exists in the minimal and standard Dockerfiles.

Useful? React with 👍 / 👎.


# ============================================
# Final stage
Expand Down
13 changes: 10 additions & 3 deletions docker/Dockerfile.minimal
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,17 @@ RUN pip install uv

WORKDIR /app

# Copy and install dependencies
# Copy source files needed for package installation
COPY pyproject.toml ./
# Create dummy README for pip install (pyproject.toml references it)
RUN touch README.md && uv pip install --system -e .
COPY api ./api
COPY core ./core
COPY services ./services
COPY monitoring ./monitoring
COPY mcp_tools ./mcp_tools
COPY start.py start_mcp.py mcp_server.py config.py main.py ./

# Install the package and its dependencies
RUN uv pip install --system .

# ============================================
# Final stage
Expand Down
12 changes: 10 additions & 2 deletions docker/Dockerfile.standard
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,17 @@ RUN pip install uv

WORKDIR /app

# Copy source files needed for package installation
COPY pyproject.toml ./
# Create dummy README for pip install (pyproject.toml references it)
RUN touch README.md && uv pip install --system -e .
COPY api ./api
COPY core ./core
COPY services ./services
COPY monitoring ./monitoring
COPY mcp_tools ./mcp_tools
COPY start.py start_mcp.py mcp_server.py config.py main.py ./

# Install the package and its dependencies
RUN uv pip install --system .

# ============================================
# Final stage
Expand Down