[GitHub] [airflow] potiuk commented on a change in pull request #19686: Disclaimer in Kubernetes executor pod template example

[GitBox <gi...@apache.org>]

potiuk commented on a change in pull request #19686:
URL: https://github.com/apache/airflow/pull/19686#discussion_r752508035



##########
File path: docs/apache-airflow/executor/kubernetes.rst
##########
@@ -91,6 +91,14 @@ Example pod templates
 
 With these requirements in mind, here are some examples of basic ``pod_template_file`` YAML files.
 
+.. note::
+

Review comment:
       > I'd be in favor of stating these are for inspiration and that care should be taken to ensure all configuration is consistent across components in your deployment.
   
   This is precisely the first thing you see when you follow the "config" link: https://airflow.apache.org/docs/apache-airflow/stable/configurations-ref.html
   
   | Use the same configuration across all the Airflow components. While each component does not require all, some configurations need to be same otherwise they would not work as expected. A good example for that is secret_key which should be same on the Webserver and Worker to allow Webserver to fetch logs from Worker.
   




-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org