[GitHub] [airflow] bsdz commented on a change in pull request #15592: Better description of UID/GID behaviour in image and quickstart

[GitBox <gi...@apache.org>]

bsdz commented on a change in pull request #15592:
URL: https://github.com/apache/airflow/pull/15592#discussion_r623232904



##########
File path: docs/apache-airflow/start/docker.rst
##########
@@ -207,3 +211,41 @@ What's Next?
 ============
 
 From this point, you can head to the :doc:`/tutorial` section for further examples or the :doc:`/howto/index` section if you're ready to get your hands dirty.
+
+.. _docker-compose-env-variables:
+
+Environment variables supported by Docker Compose
+=================================================
+
++--------------------------------+-----------------------------------------------------+--------------------------+
+|   Variable                     | Description                                         | Default                  |
++================================+=====================================================+==========================+
+| ``AIRFLOW_IMAGE_NAME``         | Airflow Image to use.                               | apache/airflow:|version| |
++--------------------------------+-----------------------------------------------------+--------------------------+
+| ``AIRFLOW_UID``                | UID of the user to run Airflow containers as.       | ``50000``                |
+|                                | Override if you want to use use non-default Airflow |                          |
+|                                | UID (for example when you map folders from host,    |                          |
+|                                | it should be set to result of ``id -u`` call. If    |                          |
+|                                | you change it from default 50000, you must set      |                          |
+|                                | ``AIRFLOW_GID`` to ``0``. When it is changed,       |                          |

Review comment:
       Perhaps reword slightly to "When it is changed, a 2nd user with name ``default`` and uid ``AIRFLOW_UID`` is dynamically created ...". Might also be worth adding "This variable should not be confused with a variable of the same name in the base Dockerfile which determines the UID of the ``airflow`` user.".




-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org