kocolosk edited a comment on issue #110: Conditionally set ownership and permissions in entrypoint URL: https://github.com/apache/couchdb-docker/pull/110#issuecomment-431581945 OK after sleeping on it I went ahead and applied the `-f` flag everywhere and rebased things into a clean set. I also tried a smoke test on whether this is actually helping by commenting out all of this code in the entry point and then timing the two `chown` options manually inside a container running on my laptop: ``` root@002a9518bc6f:/opt/couchdb# time find /opt/couchdb \! \( -user couchdb -group couchdb \) -exec chown -f couchdb:couchdb '{}' + real 0m0.015s user 0m0.000s sys 0m0.000s root@002a9518bc6f:/opt/couchdb# time chown -R couchdb:couchdb /opt/couchdb real 0m3.992s user 0m0.000s sys 0m0.370s ``` Seems like a good idea 😄 There are about 1600 files in a vanilla CouchDB install so anyone with lots of (correctly-permissioned) databases would likely see an even bigger improvement.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services