BasPH commented on code in PR #36513: URL: https://github.com/apache/airflow/pull/36513#discussion_r1438914237
########## docs/apache-airflow/core-concepts/overview.rst: ########## @@ -43,13 +43,26 @@ An Airflow installation generally consists of the following components: Basic airflow architecture -------------------------- -This is the basic architecture of Airflow that you'll see in simple installations: +This is the basic architecture of Airflow that you'll see in simple installations. + +Note that even in the simple installation, Airflow Webserver does not access the DAG files directly +(the code you see in the Code tab of the UI is synchronized via the metadata database). + +This is to allow for a more secure deployment, where the webserver has never the need or even +possibility to executed code submitted by DAG authors, the only custom code that webserver can +execute are the plugins and those should be only installed and deployed by the Airflow Deployment Manager. Review Comment: @potiuk Sounds good. It's not my goal to keep things simple per se. However, I do think we should introduce complex topics more gradually, one concept at a time. A piece of text, or an image, becomes difficult to comprehend if multiple previously unmentioned concepts are introduced at the same time to the reader. So starting with a bare minimum diagram, and then saying "here's how a more complex setup might look like" is indeed a good approach to introduce the reader to a complex topic. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
