Skip to content

Conversation

akalipetis
Copy link
Contributor

  • merge ENV definitions in a single layer
  • remove VOLUME definition to avoid leftover volumes and fix crashes
  • use COPY --link when copying uv binaries

* merge `ENV` definitions in a single layer
* remove `VOLUME` definition to avoid leftover volumes and fix crashes
* use `COPY --link` when copying uv binaries
@akalipetis akalipetis requested review from a team and Copilot September 11, 2025 14: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 structure for better build performance and runtime behavior by consolidating environment variables, removing problematic volume declarations, and using modern Docker features.

  • Consolidated multiple ENV statements into a single layer to reduce image size
  • Removed VOLUME declaration to prevent leftover volumes and runtime crashes
  • Added --link flag to COPY instruction for improved build performance

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@parisk parisk left a comment

Choose a reason for hiding this comment

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

LGTM 🦎

@akalipetis akalipetis merged commit 74e04cd into main Sep 11, 2025
8 checks passed
@akalipetis akalipetis deleted the hotfix/volumes-ci branch September 11, 2025 14:32
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