Skip configuring image in read-only systems#818
Skip configuring image in read-only systems#818alessfg wants to merge 2 commits intonginx:masterfrom
Conversation
|
If the user is providing or relying on the scripts in |
|
The intent (and the overall intent of the thread in the linked issue) of this PR is for the scripts to be ignored in read-only systems where the overall goal is for those scripts to be ignored altogether (aka systems that are set to read-only on purpose). An alternative would be to provide an even more "minimal" version of the images that strips all the entrypoint scripts altogether, but I was trying to avoid adding yet another variant. |
|
In that case, wouldn't the simplest solution be to just skip the entrypoint altogether since it isn't providing anything except running the default command? This one fails since it can't create $ docker run -d --read-only --entrypoint nginx nginx:[version] -g 'daemon off'The only difference between these is a little extra output at the beginning, but both run nginx: $ docker run -it --rm --read-only --tmpfs /var/cache/nginx/ --tmpfs /var/run/ --entrypoint nginx nginx -g 'daemon off;'
$ docker run -it --rm --read-only --tmpfs /var/cache/nginx/ --tmpfs /var/run/ nginx |
The image in its current state fails when being run in read-only systems (nginx/docker-nginx-unprivileged#43). This PR aims to fix that.