You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by ep...@apache.org on 2023/08/04 10:50:26 UTC
[airflow] 04/14: Avoid upgrade-to-newer-dependencies when provider.yaml changes (#33082)
This is an automated email from the ASF dual-hosted git repository.
ephraimanierobi pushed a commit to branch v2-7-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 74aeccd236a159ad60c89f7ef8530b805ce73f1c
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Thu Aug 3 21:17:11 2023 +0200
Avoid upgrade-to-newer-dependencies when provider.yaml changes (#33082)
The "upgrade-to-newer-dependencies" has been triggered when
provider.yaml changed, but this was completely unnecessary. The
real reason for upgrade is only when
generated/provider_dependencies.json changes - because that is
what changes dependencies used.
The provider_dependencies.json is automatically updated during
pre-commit based on provider.yaml files, so only using it to
trigger --upgrade-to-newer-dependencies is enough.
(cherry picked from commit 825a818eb556e8d265f3b04b9476be55a77c9689)
---
dev/breeze/src/airflow_breeze/utils/selective_checks.py | 1 -
dev/breeze/tests/test_selective_checks.py | 2 +-
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/dev/breeze/src/airflow_breeze/utils/selective_checks.py b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
index 84e92905ed..6f9fc51d88 100644
--- a/dev/breeze/src/airflow_breeze/utils/selective_checks.py
+++ b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
@@ -133,7 +133,6 @@ CI_FILE_GROUP_MATCHES = HashableDict(
r"^setup.cfg",
r"^setup.py",
r"^generated/provider_dependencies.json$",
- r"^airflow/providers/.*/provider.yaml$",
],
FileGroupForCi.DOC_FILES: [
r"^docs",
diff --git a/dev/breeze/tests/test_selective_checks.py b/dev/breeze/tests/test_selective_checks.py
index e3f9bacb98..6e31ee6aaa 100644
--- a/dev/breeze/tests/test_selective_checks.py
+++ b/dev/breeze/tests/test_selective_checks.py
@@ -925,7 +925,7 @@ def test_no_commit_provided_trigger_full_build_for_any_event_type(github_event):
pytest.param(
("airflow/providers/microsoft/azure/provider.yaml",),
{
- "upgrade-to-newer-dependencies": "true",
+ "upgrade-to-newer-dependencies": "false",
},
id="Provider.yaml changed",
),