Taragolis commented on code in PR #28236:
URL: https://github.com/apache/airflow/pull/28236#discussion_r1044242506
##########
airflow/providers/amazon/aws/hooks/eks.py:
##########
@@ -596,3 +597,6 @@ def fetch_access_token_for_cluster(self, eks_cluster_name:
str) -> str:
# remove any base64 encoding padding:
return "k8s-aws-v1." + base64_url.rstrip("=")
+
+ def get_waiter(self, waiter_name):
+ return EksBotoWaiter(client=self.conn).waiter(waiter_name)
Review Comment:
Just one additional thing by default there are limited files which included
in resulting wheel/sdist package, and json files do not add by default.
So we also need updates in this file:
https://github.com/apache/airflow/blob/5fad8a4a450de8388e4488523670d5f10f4f35b1/dev/provider_packages/MANIFEST_TEMPLATE.in.jinja2#L25-L30
For avoid the situation which happen with k8s provider:
https://github.com/apache/airflow/pull/27451
--
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]
