Skip to content
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

Release 1.5.1 #1094

Merged
merged 9 commits into from
Jul 17, 2024
Merged

Release 1.5.1 #1094

merged 9 commits into from
Jul 17, 2024

Conversation

tatiana
Copy link
Collaborator

@tatiana tatiana commented Jul 16, 2024

Bug fixes

Others

Copy link

netlify bot commented Jul 16, 2024

Deploy Preview for sunny-pastelito-5ecb04 canceled.

Name Link
🔨 Latest commit 2068f21
🔍 Latest deploy log https://app.netlify.com/sites/sunny-pastelito-5ecb04/deploys/6697c01352899800087ef9a3

Copy link

codecov bot commented Jul 17, 2024

Codecov Report

Attention: Patch coverage is 81.81818% with 2 lines in your changes missing coverage. Please review.

Project coverage is 96.06%. Comparing base (2994dba) to head (e0d45f3).
Report is 7 commits behind head on main.

Files Patch % Lines
cosmos/cache.py 71.42% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1094      +/-   ##
==========================================
- Coverage   96.12%   96.06%   -0.06%     
==========================================
  Files          62       62              
  Lines        3249     3255       +6     
==========================================
+ Hits         3123     3127       +4     
- Misses        126      128       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

pankajkoti and others added 7 commits July 17, 2024 13:29
New Features

* Speed up ``LoadMode.DBT_LS`` by caching dbt ls output in Airflow
Variable by @tatiana in #1014
* Support to cache profiles created via ``ProfileMapping`` by
@pankajastro in #1046
* Support for running dbt tasks in AWS EKS in #944 by @VolkerSchiewe
* Add Clickhouse profile mapping by @roadan and @pankajastro in #353 and
#1016
* Add node config to TaskInstance Context by @linchun3 in #1044

Bug fixes

* Support partial parsing when cache is disabled by @tatiana in #1070
* Fix disk permission error in restricted env by @pankajastro in #1051
* Add CSP header to iframe contents by @dwreeves in #1055
* Stop attaching log adaptors to root logger to reduce logging costs by
@glebkrapivin in #1047

Enhancements

* Support ``static_index.html`` docs by @dwreeves in #999
* Support deep linking dbt docs via Airflow UI by @dwreeves in #1038
* Add ability to specify host/port for Snowflake connection by @whummer
in #1063

Docs

* Fix rendering for env ``enable_cache_dbt_ls`` by @pankajastro in #1069

Others

* Update documentation for DbtDocs generator by @arjunanan6 in #1043
* Use uv in CI by @dwreeves in #1013
* Cache hatch folder in the CI by @tatiana in #1056
* Change example DAGs to use ``example_conn`` as opposed to
``airflow_db`` by @tatiana in #1054
* Mark plugin integration tests as integration by @tatiana in #1057
* Ensure compliance with linting rule D300 by using triple quotes for
docstrings by @pankajastro in #1049
* Pre-commit hook updates in #1039, #1050, #1064
* Remove duplicates in changelog by @jedcunningham in #1068

(cherry picked from commit 18d2c90)
<!--pre-commit.ci start-->
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.4.10 →
v0.5.0](astral-sh/ruff-pre-commit@v0.4.10...v0.5.0)
- [github.com/asottile/blacken-docs: 1.16.0 →
1.18.0](adamchainz/blacken-docs@1.16.0...1.18.0)
- [github.com/pre-commit/mirrors-mypy: v1.10.0 →
v1.10.1](pre-commit/mirrors-mypy@v1.10.0...v1.10.1)
<!--pre-commit.ci end-->

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
(cherry picked from commit 30a7e5f)
When Airflow is getting temporary AWS credentials by assuming role with
`role_arn` as only `Connection` parameter, this cause task to fail due
to missing credentials. This is due to the latest changes related to
profile caching. The `env_vars` are accessed before `profile` which, in
this case, means required values are not populated yet.

(cherry picked from commit 5a36651)
<!--pre-commit.ci start-->
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.5.0 →
v0.5.1](astral-sh/ruff-pre-commit@v0.5.0...v0.5.1)
<!--pre-commit.ci end-->

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
(cherry picked from commit 3ff8d10)
Cosmos 1.0 was released almost a year ago. I know at least 70 companies
are using it in production, and we are seeing nearly a million downloads
per month in PyPI. I believe it's safe to change the project status from
alpha to prod.

(cherry picked from commit 5b4c5d0)
Two Astronomer customers reported issues while trying to upgrade to
Cosmos 1.5:

```
No such file or directory "/usr/local/aiflow/dags/dbt/dbt_venv/bin/python"
```

The issue is that their dbt project folder had a symbolic link to a file
that no longer existed. When Cosmos attempted to iterate over that
directory to create a hash representing the cache version, it could not
open the file referenced by the symbolic link.

This PR reproduces the issue and fixes the behavior. Users are also
advised to check if there are any broken symlinks on their dbt folder
and delete them. We also discourage having Python virtual environments
inside the dbt project folder and/or the Airflow DAGs folder.

Closes: #1096
(cherry picked from commit 0bafe88)
@tatiana tatiana changed the title (WIP) Release 1.5.1 Release 1.5.1 Jul 17, 2024
@tatiana tatiana marked this pull request as ready for review July 17, 2024 12:35
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 17, 2024
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Jul 17, 2024
@tatiana
Copy link
Collaborator Author

tatiana commented Jul 17, 2024

Note: Github would like us to rebase from the main, but there are new features in the main that we should not add to this micro release.

@pankajkoti @pankajastro I suggest we:

  • release as it is
  • rebase on main
  • merge this PR
    WDYT?

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 17, 2024
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Jul 17, 2024
@tatiana tatiana merged commit d316b7a into main Jul 17, 2024
28 checks passed
@tatiana tatiana deleted the release-1.5.1 branch July 17, 2024 12:59
@tatiana tatiana added this to the Cosmos 1.5.1 milestone Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants