eladkal commented on code in PR #27120:
URL: https://github.com/apache/airflow/pull/27120#discussion_r998725827


##########
airflow/providers/cncf/kubernetes/CHANGELOG.rst:
##########
@@ -32,6 +32,11 @@ Breaking changes
 
 Previously KubernetesPodOperator considered some settings from the Airflow 
config's ``kubernetes`` section.  Such consideration was deprecated in 4.1.0 
and is now removed.  If you previously relied on the Airflow config, and you 
want client generation to have non-default configuration, you will need to 
define your configuration in an Airflow connection and set KPO to use the 
connection.  See kubernetes provider documentation on defining a kubernetes 
Airflow connection for details.
 
+Features
+~~~~~~~~
+
+Previously, ``namespace`` and ``name`` were required arguments for 
KubernetesPodOperator. Now, if ``name`` is not supplied, ``task_id`` will be 
used.  And if ``namespace`` is not supplied, and the task is running in a 
cluster, then the task pod's namespace will be used.

Review Comment:
   mmm was name mandatory for KPO? I think it was optional because it could 
have been defined in pod template



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

Reply via email to