[PATCH v2] docs: clarify and improve docker dev docs
Daniel Axtens
dja at axtens.net
Thu Jun 24 12:23:16 AEST 2021
Applied.
Daniel Axtens <dja at axtens.net> writes:
> The docker dev docs:
>
> - had the links for docker and docker-compose swapped.
>
> - had a old docker install link.
>
> - lacked an up-front explanation of the requirement that a regular user
> be able to manage the docker daemon (and a fairly unhelpful reference
> link in the most appropriate note block.)
>
> Fix it all.
>
> Reported-by: Emily Shaffer <emilyshaffer at google.com>
> Signed-off-by: Daniel Axtens <dja at axtens.net>
>
> ---
>
> v2: properly use the link format. sigh.
> ---
> docs/development/installation.rst | 20 ++++++++++++++------
> 1 file changed, 14 insertions(+), 6 deletions(-)
>
> diff --git a/docs/development/installation.rst b/docs/development/installation.rst
> index ff81229dfbd9..d16177ccd282 100644
> --- a/docs/development/installation.rst
> +++ b/docs/development/installation.rst
> @@ -22,7 +22,13 @@ Patchwork provides a Docker-based environment for quick configuration of a
> development environment. This is the preferred installation method. To
> configure Patchwork using Docker:
>
> -#. Install `docker`_ and `docker-compose`_.
> +#. Install `docker`_ and `docker-compose`_. [1]_ Patchwork assumes that you
> + have Docker configured to allow a non-root user to manage Docker, as
> + outlined in the `Docker post-install instructions`__.
> +
> + .. [1] Depending on your distro, `docker-compose` may also be available as a
> + package.
> + __ post-install_
>
> #. Create a ``.env`` file in the root directory of the project and store your
> ``UID`` and ``GID`` attribute there.
> @@ -125,9 +131,10 @@ For more information on Docker itself, please refer to the `docker`_ and
>
> ERROR: Couldn't connect to the Docker daemon at http+docker://localunixsocket - is it running?
>
> - ensure you have correctly installed Docker, added your user to the
> - ``docker`` group, and started the daemon, per the `docker documentation
> - <docker>`_.
> + ensure you have correctly installed Docker, and have followed the `Docker
> + post-install instructions`__.
> +
> + __ post-install_
>
> .. note::
>
> @@ -141,8 +148,9 @@ For more information on Docker itself, please refer to the `docker`_ and
>
> __ https://github.com/docker/compose/issues/2380
>
> -.. _docker: https://docs.docker.com/compose/install/
> -.. _docker-compose: https://docs.docker.com/engine/installation/linux/
> +.. _docker: https://docs.docker.com/engine/install/
> +.. _docker-compose: https://docs.docker.com/compose/install/
> +.. _post-install: https://docs.docker.com/engine/install/linux-postinstall/#manage-docker-as-a-non-root-user
>
>
> Manual Installation
> --
> 2.30.2
More information about the Patchwork
mailing list