-
Notifications
You must be signed in to change notification settings - Fork 188
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
Gracefully error when users set imcompatible RenderConfig.dbt_deps
and operator_args
install_deps
#1505
Conversation
✅ Deploy Preview for sunny-pastelito-5ecb04 canceled.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1505 +/- ##
=======================================
Coverage 97.31% 97.31%
=======================================
Files 79 79
Lines 4616 4618 +2
=======================================
+ Hits 4492 4494 +2
Misses 124 124 ☔ View full report in Codecov by Sentry. |
Deploying astronomer-cosmos with
|
Latest commit: |
7178870
|
Status: | ✅ Deploy successful! |
Preview URL: | https://e3dcb20a.astronomer-cosmos.pages.dev |
Branch Preview URL: | https://issue-1457.astronomer-cosmos.pages.dev |
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. Thanks for the discussion together! Happy to be merged once CI passes.
…and operator_args install_deps
Customer is facing this error when
![Image](https://private-user-images.githubusercontent.com/272048/402574216-495e46c8-419d-48d3-9bb4-e8955ce4c43e.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkxNzAyOTAsIm5iZiI6MTczOTE2OTk5MCwicGF0aCI6Ii8yNzIwNDgvNDAyNTc0MjE2LTQ5NWU0NmM4LTQxOWQtNDhkMy05YmI0LWU4OTU1Y2U0YzQzZS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjEwJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIxMFQwNjQ2MzBaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1lOWIxYjBjZWQzZmNhMmE2NTg4YmVmNTc4ZTQ5MzAwOWZiNGExYzMwODA5MzFlYmZjZDE3YzQxODk3MzVmYTc3JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.QV1lPVX0pX19fKlJOOPj4IA8BmLcTiwxJCiBrmZ3GY4)
RenderConfig(dbt_deps=True)
andoperator_args={"install_deps": False}
:This issue does not happen if both of them are
False
.Closes: #1458
Closes: #1457