search

hltcoe / concrete-python

Python modules and scripts for working with Concrete, a data serialization format for NLP
https://concrete-python.readthedocs.io
Other
20 stars 8 forks source link

Remove additional user in Docker image (run as root) #9

Closed ccmaymay closed 2 years ago

ccmaymay commented 2 years ago

The convention in Docker is that everything runs as root. I mistakenly designed the Docker container with an additional non-root "concrete" user that container processes run as. I think this behavior will surprise users of the image; it also makes adding onto the image more laborious as the current user has to be switched back to root before many common operations can be performed.

ccmaymay commented 2 years ago

Ooops this has already been done... hooray