Skip to content

Conversation

@AEnterprise
Copy link
Contributor

use alpine minimal base image significantly reduce double data storage in different layers use php-fpm daemon for more efficient and faster php execution

image size 437.4MB => 337.8MB, 23% less

some improvements to consider for the future

  • the vendor folder is massive, especially with the aws sdk taking 40MB+. maybe another library can do the same with less bloat? i doubt we really need all that
  • probably possible to do some more fine tuning of php-fpm in the future to work more optimized for koel but i stuck to mostly default for now

use alpine minimal base image significantly reduce double data storage in different layers use php-fpm daemon for more
efficient and faster php execution

image size 437.4MB => 337.8MB, 23% less
@AEnterprise AEnterprise mentioned this pull request Jan 9, 2026
@phanan
Copy link
Member

phanan commented Jan 21, 2026

Since I'm no Docker expert: how does it affect existing Docker-based installations?

@AEnterprise
Copy link
Contributor Author

it does not affect existing docker installations for running koel, none of the koel files or mount locations changed.
the standalone php runtime to run php commands in the container (for compose and such) is still present

people should notice faster response times as the dedicated daemon for php enables much more optimalizations and resource re-use between endpoint calls (think keeping connection sockets open, not having to allocate all memory from scratch, pottential to further tweak it for jit runtime optimalizations)

the only thing that's really "lost" is that the base layers is now a minimalistic one instead of a full os. some tools like vi are no longer in the container. but honestly how many people where using it? it's not a beginner friendly editor, and you're supposed to mount the env file into the container so you might as well edit it with your favorite editor from the outside. much easier

this smaller base layer is also much more secure because it significantly reduced the attack vectors that could be exploited.

@phanan
Copy link
Member

phanan commented Jan 22, 2026

Alright, let's do it!

@phanan phanan merged commit 3e334cf into koel:master Jan 22, 2026
1 check passed
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