Skip to content

⬆ [pre-commit.ci] pre-commit autoupdate#1901

Merged
tatiana merged 2 commits into
mainfrom
pre-commit-ci-update-config
Aug 6, 2025
Merged

⬆ [pre-commit.ci] pre-commit autoupdate#1901
tatiana merged 2 commits into
mainfrom
pre-commit-ci-update-config

Conversation

@pre-commit-ci
Copy link
Copy Markdown
Contributor

@pre-commit-ci pre-commit-ci Bot commented Aug 4, 2025

@pre-commit-ci
⬆ [pre-commit.ci] pre-commit autoupdate
81fed88 
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.12.5 → v0.12.7](astral-sh/ruff-pre-commit@v0.12.5...v0.12.7)
- [github.com/pre-commit/mirrors-mypy: v1.17.0 → v1.17.1](pre-commit/mirrors-mypy@v1.17.0...v1.17.1)
@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 4, 2025 •
edited
Loading

✅ Deploy Preview for sunny-pastelito-5ecb04 canceled.

Name Link
🔨 Latest commit 66979c9
🔍 Latest deploy log https://app.netlify.com/projects/sunny-pastelito-5ecb04/deploys/689321a8582be80008ee95a9

@tatiana
Copy link
Copy Markdown
Collaborator

tatiana commented Aug 5, 2025

@pankajkoti do you have any idea why GH Actions are raising Error: Process completed with exit code 127? Could it be due to a security restriction we added?

@pankajkoti
Copy link
Copy Markdown
Contributor

pankajkoti commented Aug 5, 2025

@pankajkoti do you have any idea why GH Actions are raising Error: Process completed with exit code 127? Could it be due to a security restriction we added?

@tatiana haven't looked into the reason yet. I just tried re-running the failed test, it failed again with that error. Wondering if it has do anything with the security restriction as it happening only on this specific matrix entry.

Let me try clearing up the GitHub Action caches and re-run the failed job once.

@pankajkoti
Copy link
Copy Markdown
Contributor

pankajkoti commented Aug 5, 2025

Let me try clearing up the GitHub Action caches and re-run the failed job once.

I cleared the caches, re-triggered, same error. But this does not seem to be related to the security fixes. Somehow Airflow is getting uninstalled as part of our setup scripts for the integration tests https://github.com/astronomer/astronomer-cosmos/actions/runs/16728536342/job/47350568265#step:7:1287

@tatiana
Copy link
Copy Markdown
Collaborator

tatiana commented Aug 5, 2025

Let me try clearing up the GitHub Action caches and re-run the failed job once.

I cleared the caches, re-triggered, same error. But this does not seem to be related to the security fixes. Somehow Airflow is getting uninstalled as part of our setup scripts for the integration tests https://github.com/astronomer/astronomer-cosmos/actions/runs/16728536342/job/47350568265#step:7:1287

It seems Airflow apache-airflow==3.0.3 was installed in that job :(

Please, could you log a ticket for this - I believe we should not be merging any other Cosmos PRs until we sort this out

@tatiana tatiana mentioned this pull request Aug 5, 2025
Merged
tatiana added a commit that referenced this pull request Aug 6, 2025
@tatiana
Fix broken CI tests.py3.11-2.8-1.9:test-integration-setup (#1902)
ae453d5 
Installing `openlineage-airflow` and `airflow-provider-duckdb` without
restricting the version of Airflow was leading to AF3 to be installed
instead of AF2.8 when running

```bash
hatch run tests.py3.11-2.8-1.9:test-integration-setup
```

As can be seen in this GH action CI job run:

https://github.com/astronomer/astronomer-cosmos/actions/runs/16731639151/job/47426561634?pr=1901

Which led other PRs to fail:
#1901

Closes: #1905
@tatiana tatiana merged commit a407ccc into main Aug 6, 2025
89 of 91 checks passed
@tatiana tatiana deleted the pre-commit-ci-update-config branch August 6, 2025 09:35
@tatiana tatiana mentioned this pull request Aug 7, 2025
Merged
tatiana added a commit that referenced this pull request Aug 8, 2025
@tatiana
Fix broken CI tests.py3.11-2.8-1.9:test-integration-setup (#1902)
26fa573 
Installing `openlineage-airflow` and `airflow-provider-duckdb` without
restricting the version of Airflow was leading to AF3 to be installed
instead of AF2.8 when running

```bash
hatch run tests.py3.11-2.8-1.9:test-integration-setup
```

As can be seen in this GH action CI job run:

https://github.com/astronomer/astronomer-cosmos/actions/runs/16731639151/job/47426561634?pr=1901

Which led other PRs to fail:
#1901

Closes: #1905
(cherry picked from commit ae453d5)
tatiana added a commit that referenced this pull request Aug 8, 2025
@pre-commit-ci @tatiana
⬆ [pre-commit.ci] pre-commit autoupdate (#1901)
6174d42 
<!--pre-commit.ci start-->
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.12.5 →
v0.12.7](astral-sh/ruff-pre-commit@v0.12.5...v0.12.7)
- [github.com/pre-commit/mirrors-mypy: v1.17.0 →
v1.17.1](pre-commit/mirrors-mypy@v1.17.0...v1.17.1)
<!--pre-commit.ci end-->

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Tatiana Al-Chueyr <tatiana.alchueyr@gmail.com>
(cherry picked from commit a407ccc)
tatiana added a commit that referenced this pull request Aug 8, 2025
@tatiana @pankajkoti
Release 1.10.2 (#1911)
59b71ae 
**Bug Fixes**

* Fix task instance ``try_number`` attribute for Airflow 3 compatibility
by @pankajkoti in #1781
* Fix rendered template override logic when
``should_store_compiled_sql=False`` to restore pre-refactor behaviour by
@pankajkoti in #1777
* Fix ``ProfileConfig`` in GCP Cloud Run job execution mode by
@ramonvermeulen in #1783
* Fix dbt Docs page height by @1cadumagalhaes in #1793
* Add support to base64 encoded pem in Snowflake profiles by
@brunocmartins in #1801
* Allow to disable owner inheritance from dbt into airflow DAG owners by
@CorsettiS in #1787
* Fix Kubernetes Pod Operator conversion of ``container_resources`` to
``resources`` by @johnhoran in #1821
* Fix ``dbt deps`` with project level variables by @AlexandrKhabarov in
#1822
* Fix source freshness warnings in kubernetes execution mode by
@Pawel-Drabczyk in #1859
* Fix: Harden DbtNode against null config/meta by @pankajkoti in #1877
* Fix cache behaviour when DAG name contains "." by @tatiana in #1908

**Documentation**

* Fix ``contributing.rst`` docs by @tatiana in #1785
* Fix docs rendering in Airflow 3 Compatibility by @pankajastro in #1790
* Fix typo in ``selecting-excluding.rst`` by @msshroff in #1814
* Update testing behavior file with ``ExecutionMode.KUBERNETES`` by
@LuigiCerone in #1813
* Add step to fork repo in contributing guide by @pankajastro in #1808
* Fix ``depends_on`` attribute by @benedikt-buchert in #1837
* Fix character name by @ThePsyjo in #1860
* Update suggested MWAA startup script by @jaklan in #1884
* Make implementation & docs consistent regarding
``use_dataset_airflow3_uri_standard`` by @Anti0ff in #1878

**Others**

* Set retries to 0 in example DAGs by @pankajkoti in #1782
* Fix ``test_async_example_dag_without_setup_task`` tests by
@pankajastro in #1788
* Fix test hash value for Darwin when using Py 3.12.10 by @tatiana in
#1786
* Upgrade Python and Airflow used to run MyPy checks by @tatiana in
#1796
* Assert example DAGs' ``DagRunState`` and fix issues by @pankajkoti and
@tatiana in #1778
* Update the conflict matrix to include AF 2.10, 2.11 & 3.0 and dbt 1.9
& 1.10 by @tatiana in #1820
* Fix broken CI due to Pydantic conflicts by @tatiana in #1809
* Drop Python 3.8 Support by @pankajastro in #1852
* Add Airflow 2.11 to the test matrix by @tatiana in #1807
* Require Authorize for all jobs on pull requests from external
contributors in CI by @pankajkoti in #1861
* Leverage Trusted Publisher Management when publishing PyPI package by
@tatiana in #1862
* CI: Add back accidentally deleted python-version matrix for running
unit tests by @pankajkoti in #1872
* Remove commented code and fix mypy failures by @pankajkoti in #1876
* Add Zizmor analysis GitHub action by @pankajkoti in #1870
* Catch FlushError on Datasets for Airflow 2.11 dags test by @pankajkoti
in #1880
* Add deprecation warning for ``LoadMode.CUSTOM`` parser by
@duongphannamhung in #1885
* CI: Add GitHub CodeQL analysis workflow (codeql.yml) by @pankajkoti in
#1871
* Resolve 'credential persistence through GitHub Actions artifacts'
warnings from Zizmor by @pankajkoti in #1890
* Resolve 'overly broad permissions' warnings from Zizmor by @pankajkoti
in #1889
* Resolve Zizmor error alerts for unpinned image references; mark alert
for pull_request_target ignored by @pankajkoti in #1888
* Fix broken CI ``tests.py3.11-2.8-1.9:test-integration-setup`` by
@tatiana in #1902
* Add dbt-core 1.10 to test matrix by @tatiana in #1767
* Pin package dbt-databricks by @pankajastro in #1909
* Enable matrix test entry for dbt-1.9, python-3.9 and airflow-3.0 tests
in CI by @pankajastro in #1900
* Pre-commit updates: #1779, #1795, #1800, #1857, #1863, #1869, #1892,
#1901
* Dependabot updates: #1904

Co-authored-by: Pankaj Koti <pankajkoti699@gmail.com>
@pankajkoti pankajkoti mentioned this pull request Aug 8, 2025
Merged
tatiana added a commit that referenced this pull request Aug 8, 2025
@pankajkoti @tatiana
Release 1.10.2 (#1912)
f698166 
**Bug Fixes**

* Fix task instance ``try_number`` attribute for Airflow 3 compatibility
by @pankajkoti in #1781
* Fix rendered template override logic when
``should_store_compiled_sql=False`` to restore pre-refactor behaviour by
@pankajkoti in #1777
* Fix ``ProfileConfig`` in GCP Cloud Run job execution mode by
@ramonvermeulen in #1783
* Fix dbt Docs page height by @1cadumagalhaes in #1793
* Add support to base64 encoded pem in Snowflake profiles by
@brunocmartins in #1801
* Allow to disable owner inheritance from dbt into airflow DAG owners by
@CorsettiS in #1787
* Fix Kubernetes Pod Operator conversion of ``container_resources`` to
``resources`` by @johnhoran in #1821
* Fix ``dbt deps`` with project level variables by @AlexandrKhabarov in
#1822
* Fix source freshness warnings in kubernetes execution mode by
@Pawel-Drabczyk in #1859
* Fix: Harden DbtNode against null config/meta by @pankajkoti in #1877
* Fix cache behaviour when DAG name contains "." by @tatiana in #1908

**Documentation**

* Fix ``contributing.rst`` docs by @tatiana in #1785
* Fix docs rendering in Airflow 3 Compatibility by @pankajastro in #1790
* Fix typo in ``selecting-excluding.rst`` by @msshroff in #1814
* Update testing behavior file with ``ExecutionMode.KUBERNETES`` by
@LuigiCerone in #1813
* Add step to fork repo in contributing guide by @pankajastro in #1808
* Fix ``depends_on`` attribute by @benedikt-buchert in #1837
* Fix character name by @ThePsyjo in #1860
* Update suggested MWAA startup script by @jaklan in #1884
* Make implementation & docs consistent regarding
``use_dataset_airflow3_uri_standard`` by @Anti0ff in #1878

**Others**

* Set retries to 0 in example DAGs by @pankajkoti in #1782
* Fix ``test_async_example_dag_without_setup_task`` tests by
@pankajastro in #1788
* Fix test hash value for Darwin when using Py 3.12.10 by @tatiana in
#1786
* Upgrade Python and Airflow used to run MyPy checks by @tatiana in
#1796
* Assert example DAGs' ``DagRunState`` and fix issues by @pankajkoti and
@tatiana in #1778
* Update the conflict matrix to include AF 2.10, 2.11 & 3.0 and dbt 1.9
& 1.10 by @tatiana in #1820
* Fix broken CI due to Pydantic conflicts by @tatiana in #1809
* Drop Python 3.8 Support by @pankajastro in #1852
* Add Airflow 2.11 to the test matrix by @tatiana in #1807
* Require Authorize for all jobs on pull requests from external
contributors in CI by @pankajkoti in #1861
* Leverage Trusted Publisher Management when publishing PyPI package by
@tatiana in #1862
* CI: Add back accidentally deleted python-version matrix for running
unit tests by @pankajkoti in #1872
* Remove commented code and fix mypy failures by @pankajkoti in #1876
* Add Zizmor analysis GitHub action by @pankajkoti in #1870
* Catch FlushError on Datasets for Airflow 2.11 dags test by @pankajkoti
in #1880
* Add deprecation warning for ``LoadMode.CUSTOM`` parser by
@duongphannamhung in #1885
* CI: Add GitHub CodeQL analysis workflow (codeql.yml) by @pankajkoti in
#1871
* Resolve 'credential persistence through GitHub Actions artifacts'
warnings from Zizmor by @pankajkoti in #1890
* Resolve 'overly broad permissions' warnings from Zizmor by @pankajkoti
in #1889
* Resolve Zizmor error alerts for unpinned image references; mark alert
for pull_request_target ignored by @pankajkoti in #1888
* Fix broken CI ``tests.py3.11-2.8-1.9:test-integration-setup`` by
@tatiana in #1902
* Add dbt-core 1.10 to test matrix by @tatiana in #1767
* Pin package dbt-databricks by @pankajastro in #1909
* Enable matrix test entry for dbt-1.9, python-3.9 and airflow-3.0 tests
in CI by @pankajastro in #1900
* Pre-commit updates: #1779, #1795, #1800, #1857, #1863, #1869, #1892,
#1901
* Dependabot updates: #1904

Co-authored-by: Tatiana Al-Chueyr <tatiana.alchueyr@gmail.com>
@tatiana tatiana added this to the Cosmos 1.10.2 milestone Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tatiana tatiana tatiana approved these changes

@pankajkoti pankajkoti pankajkoti approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

Cosmos 1.10.2

Development

Successfully merging this pull request may close these issues.

2 participants

@tatiana @pankajkoti