I used this in class. My students were missing the build command, because it is a bit hidden in the text. I suggest making it more obvious that the image needs to be built before running it.
Currently, instructions to copy the default configuration from the
nginx container to the host uses three commands. This can be done more
efficiently with one command (assuming the presence of "cat" in the
container which should be a safe).
Update the instructions to use the single command.
The code block contains the paths that should be mapped,
`/var/cache/nginx` and `/var/run` for readonly cases.
However, the description immediately above the code block
also listed two paths. The first, `/var/cache`, would require
additional dirs to already exist within the dir. For example,
the application will fail to start if the host didn't create
an `nginx` dir prior to starting the container.
This change updates the path in the description so it matches
the path mapped in the code block and no dirs need to exist
before starting the container.
The templates script that uses `envsubst` is not available in the current stable version (1.18).
Hope this information saves others some debugging time.
Michael Zangl