dstandish commented on a change in pull request #22304:
URL: https://github.com/apache/airflow/pull/22304#discussion_r829354931
##########
File path: docs/apache-airflow/start/docker.rst
##########
@@ -22,6 +22,26 @@ Running Airflow in Docker
This quick-start guide will allow you to quickly start Airflow with
:doc:`CeleryExecutor </executor/celery>` in Docker. This is the fastest way to
start Airflow.
+Customizing the quick-start Docker Compose
+==========================================
+
+DO NOT attempt to customize images and the Docker Compose if you do not know
exactly what you are doing,
+know Docker Compose or do not will to debug and resolve problems on your own.
If you do not
Review comment:
```suggestion
do not know Docker Compose, or are not prepared to debug and resolve
problems on your own. If you do not
```
##########
File path: docs/apache-airflow/start/docker.rst
##########
@@ -22,6 +22,26 @@ Running Airflow in Docker
This quick-start guide will allow you to quickly start Airflow with
:doc:`CeleryExecutor </executor/celery>` in Docker. This is the fastest way to
start Airflow.
Review comment:
If docker compose is really intended as a dev tool, maybe this line
should be reworked to reflect that. I'm not sure exactly what the language
should be. But maybe something like
```suggestion
This quick-start guide will allow you to quickly get Airflow up and running
with :doc:`CeleryExecutor </executor/celery>` in Docker. For running airflow
in production, we recommend running on kubernetes with the official helm chart.
```
--
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]
