search

librenms / docker

LibreNMS Docker image
MIT License
692 stars 280 forks source link

Fix for /opt/librenms/cache ownership #414

Closed jcostom closed 10 months ago

jcostom commented 11 months ago

closes #262

Not a lot of wizardry here - I added a line to the Dockerfile to properly initialize the ownership of /opt/librenms/cache as librenms:librenms instead of nobody:nobody.

Thanks @undefinedid for pointing out from where the condition was arising.

I tested on my installation with no ill effects.

CLAassistant commented 11 months ago

CLA assistant check
All committers have signed the CLA.

crazy-max commented 11 months ago

It should be done in https://github.com/librenms/docker/blob/d0ed3bb7d3b2232cfb6f75222f3b6a09eb9d671d/rootfs/etc/cont-init.d/02-fix-perms.sh#L13

jcostom commented 11 months ago

Fair enough - I pulled the line out of Dockerfile and moved the change to the script you mentioned. Thanks @crazy-max.