xEviL commented on pull request #8150:
URL: https://github.com/apache/airflow/pull/8150#issuecomment-663499869


   > I'm gonna test if global pod_template_file setting would ruin 
KubernetesPodOperator or not. Hopefully not
   
   `KubernetesPodOperator` should not be affected by Airflow configuration, 
have a look at:
   
https://github.com/apache/airflow/blob/master/airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py#L365
   and 
https://github.com/apache/airflow/blob/458d7d4cb0ffb2a525f85e5c06de2852186507c8/airflow/kubernetes/pod_generator.py#L190
   Airflow `settings` are not imported anywhere, so only constructor arguments 
are relevant (for the operator).
   
   In our test setup we also ran `KubernetesPodOperator` which we already use 
since a few months, but now also from `KubernetesExecutor` - it works!


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to