-
Notifications
You must be signed in to change notification settings - Fork 177
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
Pin dbt-databricks version to < 1.9 #1376
Conversation
Deploying astronomer-cosmos with Cloudflare Pages
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1376 +/- ##
==========================================
+ Coverage 96.02% 96.06% +0.04%
==========================================
Files 67 67
Lines 4025 4042 +17
==========================================
+ Hits 3865 3883 +18
+ Misses 160 159 -1 ☔ View full report in Codecov by Sentry. |
dd76895
to
d9dc84f
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 fixing this and creating the follow up ticket, @pankajastro ! Happy for this to be merged once the checks pass.
Our CI is broken due to a
dbt-databricks
release with conflicting dependencies withAirflow>=2.8
.The latest release of
dbt-databricks
has pinned the Pydantic to"pydantic>=1.10.0, <2"
https://github.com/databricks/dbt-databricks/blob/main/pyproject.toml#L33-L33This change was added in PR: databricks/dbt-databricks#843 and released on 9 December as part of
dbt-databricks==1.9.0
(https://pypi.org/project/dbt-databricks/1.9.0/).The
debt-data bricks
conflicts with the neAirflow version, which that requires Pydantic 2.0.For Airflow < 2.8, things work as expected: https://github.com/astronomer/astronomer-cosmos/actions/runs/12259335118/job/34201293598.
In this PR, I'm pinning dbt-databricks version to < 1.9 to make CI green again.
Follow-up PR: #1379