[GitHub] [airflow] kaxil commented on a change in pull request #20573: Implement dry_run for KubernetesPodOperator

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

kaxil commented on a change in pull request #20573:
URL: https://github.com/apache/airflow/pull/20573#discussion_r776728308



##########
File path: airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py
##########
@@ -597,3 +607,57 @@ def __exit__(self, exctype, excinst, exctb):
             logger = logging.getLogger()
             logger.error(str(excinst), exc_info=True)
         return caught_error
+
+
+def _prune_dict(val: Any, mode='strict'):
+    """
+    Note: this is duplicated from ``airflow.utils.helpers.prune_dict``.  That one is tested

Review comment:
       It is an internal method so users should be using it so as soon as we make a change to make this provider require Airflow 2.3+ we should be able to remove it without even mentioning in `UPDATING.md` of this provider




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