mik-laj commented on a change in pull request #19189: URL: https://github.com/apache/airflow/pull/19189#discussion_r737982064
########## File path: docs/docker-stack/build.rst ########## @@ -203,14 +203,22 @@ You should be aware, about a few things: `best practices of Dockerfiles <https://docs.docker.com/develop/develop-images/dockerfile_best-practices/>`_ to make sure your image is lean and small. -* The PyPI dependencies in Apache Airflow are installed in the user library, of the "airflow" user, so - PIP packages are installed to ``~/.local`` folder as if the ``--user`` flag was specified when running PIP. - Note also that using ``--no-cache-dir`` is a good idea that can help to make your image smaller. +* Using ``--no-cache-dir`` is a good idea that can help to make your image smaller. + +* The PyPI dependencies in Apache Airflow are installed in the ``/.venv`` folder. This is virtualenv where + airflow and all dependent packages are installed, following latest ``pip`` recommendations and upcoming + `PEP 668 <https://www.python.org/dev/peps/pep-0668/>`_. The ``PATH`` inside the image is set to point first Review comment: ```suggestion :pep:`668`. The ``PATH`` inside the image is set to point first ``` I am not sure if it will be of any benefit to us, but Sphinx implemented it for some reason (I believe). See: https://www.sphinx-doc.org/en/master/usage/restructuredtext/roles.html -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
