-
Notifications
You must be signed in to change notification settings - Fork 13.7k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Better diagnostics and self-healing of docker-compose (#17484)
There are several ways people might get the quick-start docker-compose running messed up (especially on linux): 1) they do not run initialization steps and run docker-compose-up 2) they do not run docker-compose-init first Also on MacOS/Windows default memory/disk settings are not enough to run Airflow via docker-compose and people are reporting "Airflow not working" where they simply do not allocate enough resources. Finally the docker compose does not support all versions of airflow and various problems might occur when you use this docker compose with old version of airflow. This change adds the following improvements: * automated check of minimum version of airflow supported * mkdir -p in the directories creation in instructions * automated checking if AIRFLOW_UID has been set (and printing error and instruction link in case it is not) * prints warning about too-low memory, cpu, disk allocation and instruction link where to read about it * automated fixing of ownership of the directories created in case they were not created initially and ended up owned by root user (cherry picked from commit 763860c)
- Loading branch information
Showing
2 changed files
with
74 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters