logger icon indicating copy to clipboard operation
logger copied to clipboard

FEAT - Docker Support + Docker Compose file

Open phil-flip opened this issue 1 year ago • 4 comments

Here is my little implementation of Docker, inspired by https://github.com/curtisf/logger/issues/43 Since I changed some var names inside the code, it needs to be retested without Docker and see, if that still works. The Docker Compose file has been changed in a way to prevent the need for a docker specific .env file. The normal one can be used. The DB migration is a hotfix at best and should be redone. I wanted to prevent the use of the .dockerenv check as that might cause issues with other platforms like K8s. If a merge is undesirable that fine. But I also didn't want to keep it to my self. Let me know, if something doesn't work.

phil-flip avatar Apr 05 '24 01:04 phil-flip

Thanks, I was actually too lazy to do that :D

uchihaservice avatar Apr 08 '24 07:04 uchihaservice

You're fine. It's just unfortunate, that the owner doesn't seem to merge PRs.

phil-flip avatar Apr 11 '24 16:04 phil-flip

Could you check out/review the docker-compose.yml in the branch named "selfhost"? That's likely going to be the one that gets distributed for selfhosting in the future.

curtisf avatar Apr 24 '24 22:04 curtisf

Everyone has their own taste in Docker-Compose- and Dockerfiles. Personally I don't like it, because it has a lot of extra stuff in it, which is done automatically done by the Docker Compose-Plugin. On the other side, I seem to miss the setup for the DB. The health check is a nice touch tho. Your Dockerfile should definitely be cleaned up, cause it the current state it requires maintenance when a new file type or folder is added.

phil-flip avatar Apr 26 '24 20:04 phil-flip