Skip to content

Fix Docker Build CI by correcting backend requirements path in Dockerfile - #2

Merged
Aathi-27 merged 2 commits into
mainfrom
copilot/fix-docker-build-job
Jul 12, 2026
Merged

Fix Docker Build CI by correcting backend requirements path in Dockerfile#2
Aathi-27 merged 2 commits into
mainfrom
copilot/fix-docker-build-job

Conversation

Copilot AI commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

The Docker Build GitHub Actions job failed because the builder stage tried to copy requirements.txt from the repository root, but the backend dependency file is located under rag-system/. This PR updates the Docker build input path so image builds can resolve Python dependencies correctly.

  • Root cause

    • Builder stage referenced a non-existent file path:
      • COPY requirements.txt .
  • Change

    • Updated the builder stage to copy the actual backend requirements file:
      • COPY rag-system/requirements.txt .
  • Impact

    • Aligns Dockerfile path resolution with repository layout.
    • Unblocks the CI Docker Build job from failing at dependency copy/install stage.
# before
COPY requirements.txt .
RUN pip install --no-cache-dir --prefix=/install -r requirements.txt

# after
COPY rag-system/requirements.txt .
RUN pip install --no-cache-dir --prefix=/install -r requirements.txt

Copilot AI changed the title [WIP] Fix failing GitHub Actions job Docker Build Fix Docker Build CI by correcting backend requirements path in Dockerfile Jul 12, 2026
Copilot AI requested a review from Aathi-27 July 12, 2026 20:06

@Aathi-27 Aathi-27 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

verified

@Aathi-27
Aathi-27 marked this pull request as ready for review July 12, 2026 20:39
@Aathi-27
Aathi-27 merged commit 078be6d into main Jul 12, 2026
3 checks passed

@Aathi-27 Aathi-27 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

approved

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