jedcunningham commented on code in PR #38951: URL: https://github.com/apache/airflow/pull/38951#discussion_r1562649946
########## airflow/providers/cncf/kubernetes/utils/pod_manager.py: ########## @@ -42,7 +42,7 @@ from airflow.exceptions import AirflowException, AirflowProviderDeprecationWarning from airflow.providers.cncf.kubernetes.callbacks import ExecutionMode, KubernetesPodOperatorCallback -from airflow.providers.cncf.kubernetes.pod_generator import PodDefaults +from airflow.providers.cncf.kubernetes.utils.xcom_sidecar import PodDefaults Review Comment: No, these are deployment wide defaults. A little more historical context - this is one area where we've instructed users to modify defaults in the past, before the ability to control the image for the sidecar directly in other ways existed (like in the connect). -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
