[BUG] machine-learning and nginx should use alpine #945

Closed
opened 2026-02-04 23:40:17 +03:00 by OVERLORD · 4 comments
Owner

Originally created by @uhthomas on GitHub (Jun 13, 2023).

The bug

Given everything else in Immich uses alpine, the machine-learning and nginx images should remain consistent.

The OS that Immich Server is running on

N/A

Version of Immich Server

N/A

Version of Immich Mobile App

N/A

Platform with the issue

  • Server
  • Web
  • Mobile

Your docker-compose.yml content

N/A

Your .env content

N/A

Reproduction steps

N/A

Additional information

No response

Originally created by @uhthomas on GitHub (Jun 13, 2023). ### The bug Given everything else in Immich uses alpine, the machine-learning and nginx images should remain consistent. ### The OS that Immich Server is running on N/A ### Version of Immich Server N/A ### Version of Immich Mobile App N/A ### Platform with the issue - [X] Server - [ ] Web - [ ] Mobile ### Your docker-compose.yml content ```YAML N/A ``` ### Your .env content ```Shell N/A ``` ### Reproduction steps ```bash N/A ``` ### Additional information _No response_
Author
Owner

@bo0tzz commented on GitHub (Jun 13, 2023):

I believe the tools/libraries we use for machine-learning don't play nice with alpine, although I don't remember the details and can't find the discussions on it anymore 🤔

@bo0tzz commented on GitHub (Jun 13, 2023): I believe the tools/libraries we use for machine-learning don't play nice with alpine, although I don't remember the details and can't find the discussions on it anymore 🤔
Author
Owner

@uhthomas commented on GitHub (Jun 13, 2023):

With the nginx image, switching to alpine reduces the image size dramatically from 142.545772 MB to 46.124232 MB.

@uhthomas commented on GitHub (Jun 13, 2023): With the nginx image, switching to alpine reduces the image size dramatically from 142.545772 MB to 46.124232 MB.
Author
Owner

@mertalev commented on GitHub (Jun 15, 2023):

Alpine for ML is a solid no since Python wheels depend on glibc while Alpine uses musl. This change would mean compiling every Python dependency, including very large ones like PyTorch.

@mertalev commented on GitHub (Jun 15, 2023): Alpine for ML is a solid no since Python wheels depend on glibc while Alpine uses musl. This change would mean compiling every Python dependency, including very large ones like PyTorch.
Author
Owner

@uhthomas commented on GitHub (Jun 15, 2023):

Makes sense, I think that's all the reason I needed. I'll leave this open for #2754 but will de-scope ml.

@uhthomas commented on GitHub (Jun 15, 2023): Makes sense, I think that's all the reason I needed. I'll leave this open for #2754 but will de-scope ml.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#945