Applied.

Daniel Axtens <d...@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 <emilyshaf...@google.com>
> Signed-off-by: Daniel Axtens <d...@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
_______________________________________________
Patchwork mailing list
Patchwork@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/patchwork

Reply via email to