[GitHub] [airflow] kaxil commented on a diff in pull request #27116: Make namespace optional for KPO

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

kaxil commented on code in PR #27116:
URL: https://github.com/apache/airflow/pull/27116#discussion_r1002047251


##########
airflow/providers/cncf/kubernetes/CHANGELOG.rst:
##########
@@ -35,7 +35,9 @@ Previously KubernetesPodOperator considered some settings from the Airflow confi
 Features
 ~~~~~~~~
 
-Previously, ``name`` was a required argument for KubernetesPodOperator (when also not supplying pod template or full pod spec). Now, if ``name`` is not supplied, ``task_id`` will be used.
+* Previously, ``name`` was a required argument for KubernetesPodOperator (when also not supplying pod template or full pod spec). Now, if ``name`` is not supplied, ``task_id`` will be used.
+* KubernetsPodOperator argument ``namespace`` is now optional.  If not supplied, we'll check the airflow conn,

Review Comment:
   Should we document this priority?



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