-
Notifications
You must be signed in to change notification settings - Fork 124
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 running dbt tasks in AWS EKS #944
Conversation
✅ Deploy Preview for sunny-pastelito-5ecb04 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #944 +/- ##
==========================================
+ Coverage 95.67% 95.72% +0.05%
==========================================
Files 59 60 +1
Lines 2890 2926 +36
==========================================
+ Hits 2765 2801 +36
Misses 125 125 ☔ View full report in Codecov by Sentry. |
ad79eca
to
d60ea0c
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.
This is a great contribution @VolkerSchiewe . I think we're close and soon can merge this.
Would it be possible to also paste a snapshot of a successful DAG run using AWS EKS operator?
Hope this is what you mean @pankajkoti (the operator naming is still before the renaming though) |
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.
LGTM. Appreciate your contribution @VolkerSchiewe 👏🏽
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.
This looks great, thanks a lot for iterating and improving it so quickly, @VolkerSchiewe !
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.
looks great 👍
Co-authored-by: Pankaj Koti <[email protected]>
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