Skip to content

Optimize Dockerfile #75

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

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

MH0386
Copy link

@MH0386 MH0386 commented Jul 15, 2025

This pull request updates the Dockerfile to use a newer Python base image, improve package management, and enhance configurability for the server. Key changes include upgrading the Python version, switching to a more efficient package installation method, and adding a new environment variable to allow the server to listen on all network interfaces.

Base image and package management updates:

  • Updated the base image from python:3.11-slim to python:3.13-slim for compatibility with the latest Python features and improvements.
  • Replaced the pip-based installation of the uv package with a pre-built binary from ghcr.io, improving efficiency and reducing build complexity.
  • Updated the command to install the mcp-server-qdrant package using the uv tool install command instead of uv pip install, aligning with the new package management approach.

Server configuration improvements:

  • Added the FASTMCP_HOST environment variable to make the server listen on all network interfaces (0.0.0.0), improving accessibility in containerized environments.
  • Updated the CMD instruction to use a JSON array format for better compatibility with Docker's execution model.

@kacperlukawski kacperlukawski requested review from joein and Copilot July 21, 2025 14:48
Copilot

This comment was marked as outdated.

@MH0386 MH0386 marked this pull request as ready for review July 25, 2025 00:23
@MH0386 MH0386 requested a review from Copilot July 25, 2025 13:07
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR optimizes the Dockerfile by upgrading the Python base image, modernizing package management with uv, and improving server network accessibility in containerized environments.

  • Upgrades Python base image from 3.11-slim to 3.13-slim
  • Switches from pip-based uv installation to pre-built binary from ghcr.io
  • Adds FASTMCP_HOST environment variable to enable network interface binding
Comments suppressed due to low confidence (1)

Dockerfile:6

  • The uv version 0.8.3 may not exist. As of my knowledge cutoff in January 2025, the latest stable versions were in the 0.4.x range. Please verify that version 0.8.3 is available at ghcr.io/astral-sh/uv before using it.
COPY --from=ghcr.io/astral-sh/uv:0.8.3 /uv /uvx /bin/

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