potiuk commented on code in PR #49681:
URL: https://github.com/apache/airflow/pull/49681#discussion_r2058644814
##########
airflow-core/docs/howto/docker-compose/docker-compose.yaml:
##########
@@ -78,6 +78,7 @@ x-airflow-common:
- ${AIRFLOW_PROJ_DIR:-.}/logs:/opt/airflow/logs
- ${AIRFLOW_PROJ_DIR:-.}/config:/opt/airflow/config
- ${AIRFLOW_PROJ_DIR:-.}/plugins:/opt/airflow/plugins
+ - ${AIRFLOW_PROJ_DIR:-.}:/opt/airflow
Review Comment:
There are a number of things that can wrong if you mount project dir.
Currently Airflow image has this:
```
airflow@5ed628cee08b:/opt/airflow$ find .
.
./logs
./dags
./airflow.cfg
./airflow.db
```
Depending on what other things you have in that directory (for example if
you run docker compose in airflow sources) - they might start interfering with
installed airflow (dependin what you are running and which directory you are
in, python might put the current working directory on PYTHONPATH.
So weird things might happen when you do it.
I'd say better way is to create a volume in docker that will be shared
between the containers- not mount it locally - but that might require some way
of initializing the config so that multiple airflow instances do not overwrite
each other's config (which anyhow is the same problem if you mount your local
volume).
--
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]
