search

ckulka / baikal-docker

Provides a ready-to-go Baikal server, incl. docker-compose.yml & Systemd service file
https://hub.docker.com/r/ckulka/baikal
MIT License
369 stars 46 forks source link

Allow skipping run of `/docker-entrypoint.d/40-fix-baikal-file-permissions.sh` #134

Closed sethidden closed 11 months ago

sethidden commented 1 year ago

Hey, Your image is used in the TrueCharts helm chart repository for TrueNAS. It's used to basically have a point and click installation of Baikal. See https://truecharts.org/charts/incubator/baikal/

I've noticed an issue with the image and described my observations and a workaround here: https://0x2f.pl/posts/truecharts-baikal/

I take it that this is the same issue as https://github.com/docker/for-linux/issues/388

Could you please add a SKIP_CHOWN environment variable, that if set, would skip running 40-fix-baikal-file-permissions.sh? This would allow the container to respond to a Kubernetes health check in a timely manner.

Alternatively, it may be just easier to publish a new tag like baikal-docker:nginx-raw where 40-fix-baikal-file-permissions.sh is completely missing from the image.

sethidden commented 11 months ago

Thanks!

ckulka commented 11 months ago

Not yet published, I hope to get that out to Docker Hub this weekend 👍

sethidden commented 11 months ago

I see that 0.9.4 is now published. TrueCharts app already picked up the 0.9.4 Docker image version, but doesn't set the BAIKAL_SKIP_CHOWN=1 vars by default - I've raised an issue here https://github.com/truecharts/charts/issues/16573

For anybody that's reading, Baikal was recently moved from Incubator train to Stable, so make sure to delete and recreate your app so that you get the chart with the new 0.9.4. Until the above linked issue is resolved, you can Edit your app in the UI and do this to make it work:

image

Thanks again :)