This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 825a818eb5 Avoid upgrade-to-newer-dependencies when provider.yaml
changes (#33082)
825a818eb5 is described below
commit 825a818eb556e8d265f3b04b9476be55a77c9689
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.
---
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 3cdf3dc7bb..9bd83c15c8 100644
--- a/dev/breeze/tests/test_selective_checks.py
+++ b/dev/breeze/tests/test_selective_checks.py
@@ -926,7 +926,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",
),
