dimberman commented on a change in pull request #12384:
URL: https://github.com/apache/airflow/pull/12384#discussion_r525381212
##########
File path: dev/provider_packages/refactor_provider_packages.py
##########
@@ -146,6 +146,13 @@ def rename_deprecated_modules(self) -> None:
for new, old in changes:
self.qry.select_module(new).rename(old)
+ def is_not_k8spodop(node: LN, capture: Capture, filename: Filename) ->
bool:
+ return not filename.endswith("/kubernetes_pod.py")
+
+
self.qry.select_module("airflow.providers.cncf.kubernetes.backcompat").filter(
+ callback=is_not_k8spodop
+ ).rename("airflow.kubernetes")
+
Review comment:
ohhh ok. I think I get it.
So basically the idea here is that we want to delete the backcompat files in
1.10 since we're not using them, though we were running into a bowler issue
because we weren't changing one import in `kubernetes_pod.py` (this was because
we needed to keep the original import for the converters. So adding that extra
query fixes the `pod_runtime_info_env` import and allows us to delete those
files.
----------------------------------------------------------------
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]
