[GitHub] kocolosk edited a comment on issue #110: Conditionally set ownership and permissions in entrypoint

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org


With regards,
Apache Git Services