New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for EKS operator #944
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for sunny-pastelito-5ecb04 canceled.
|
19f1f26
to
c42f5ad
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution, @VolkerSchiewe !
Please, could you address the tests currently failing?
Hi @tatiana thanks for getting back to me! I was already working on it, but I ran into this issue: apache/airflow#39103 Seems to be affecting the latest version of the amazon airflow provider. I already tried with pinning an older version of |
Hi @VolkerSchiewe , just a head's up: we're wrapping up the Cosmos 1.4 release and either myself or @pankajkoti will support you on this next week! |
Looking in to the failing tests I'm not sure where the problems are coming from 🤔 Unit tests: Integration tests: Also let me know if the test coverage is enough. I tried to cover the parts that are special for the eks operator, but didn't want to copy everything from the KubernetesOperator since it's tested there already. |
It appears that earlier versions of Flask are not compatible with newer versions of Jinja2; and hence we have the failure in our tests These are the constraints for Airflow 2.3 which suggests Flask 1.1.2 and Jinja2 3.0.3, however dbt-core is upgrading it to jinja2-3.1.4 (Jinja2<4,>=3.1.3 (from dbt-core)) which is not supported on Flask 1.1.2 that comes with Airflow. |
Hey there, the only failing parts are the tests with Airflow 2.3 and python 3.9. For some reason the install step fails with
when installing the test requirements . I don't really understand why and how to solve this. Is this problem you ran into earlier already? |
a809dba
to
6faf960
Compare
Fixing boto3 in the runs with airflow 2.3 solved the problem 🚀 |
Description
We are using MWAA in combination with EKS so that all our dags in airflow are running in our EKS. We would like to use the same setup with cosmos.
What changes?
Related Issue(s)
Breaking Change?
No - only an additional feature
Checklist