potiuk commented on code in PR #36513: URL: https://github.com/apache/airflow/pull/36513#discussion_r1438907524
########## 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: OK. Let me know @BasPH and @jscheffl what you think now. We should - of course - tweak it still a bit (better icons as discussed below and possibly play a bit with graphviz/diagram layout to add some margins and get it a bit more readable. Plus I can add a few links to other parts of the documentation when I am mentioning other pieces - to get it all nicely hyperlinked. But I **think** this one might achieve good balance between the simplicity and being able to explain concepts and architecture in "gradually more complex" ways - without overloading the users who just want to install airflow and use it locally. -- 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