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 ab9444e0c2 Switch committer builds to public runners (#39106) ab9444e0c2 is described below commit ab9444e0c26df1f2acfb83bb47bb64acefec437a Author: Jarek Potiuk <ja...@potiuk.com> AuthorDate: Thu Apr 18 12:14:43 2024 +0200 Switch committer builds to public runners (#39106) Following the discussion on devlist, we are switching committer buids to public runners. Discussion: * https://lists.apache.org/thread/f2kxdjm12nf2rh0tqxmv3rj49sjd6c3w --- .../src/airflow_breeze/utils/selective_checks.py | 5 +---- dev/breeze/tests/test_selective_checks.py | 20 ++++++++++---------- 2 files changed, 11 insertions(+), 14 deletions(-) diff --git a/dev/breeze/src/airflow_breeze/utils/selective_checks.py b/dev/breeze/src/airflow_breeze/utils/selective_checks.py index ee12f2881c..6b42d91346 100644 --- a/dev/breeze/src/airflow_breeze/utils/selective_checks.py +++ b/dev/breeze/src/airflow_breeze/utils/selective_checks.py @@ -1090,10 +1090,7 @@ class SelectiveChecks: if USE_SELF_HOSTED_RUNNERS_LABEL in self._pr_labels: # Forced self-hosted runners return RUNS_ON_SELF_HOSTED_RUNNER - if actor in COMMITTERS: - return RUNS_ON_SELF_HOSTED_RUNNER - else: - return RUNS_ON_PUBLIC_RUNNER + return RUNS_ON_PUBLIC_RUNNER return RUNS_ON_PUBLIC_RUNNER @cached_property diff --git a/dev/breeze/tests/test_selective_checks.py b/dev/breeze/tests/test_selective_checks.py index 099478ea9c..e2ee1f0a6b 100644 --- a/dev/breeze/tests/test_selective_checks.py +++ b/dev/breeze/tests/test_selective_checks.py @@ -1700,13 +1700,13 @@ def test_helm_tests_trigger_ci_build(files: tuple[str, ...], expected_outputs: d "apache/airflow", (), dict(), - '["self-hosted", "Linux", "X64"]', - "true", - "true", - "true", + '["ubuntu-22.04"]', + "false", "false", "true", "false", + "false", + "false", False, id="Pull request committer", ), @@ -1780,13 +1780,13 @@ def test_helm_tests_trigger_ci_build(files: tuple[str, ...], expected_outputs: d "apache/airflow", [], dict(), - '["self-hosted", "Linux", "X64"]', - "true", - "true", - "true", + '["ubuntu-22.04"]', + "false", "false", "true", "false", + "false", + "false", False, id="Pull request target committer", ), @@ -2065,7 +2065,7 @@ def test_mypy_matches( ("README.md",), { "is-committer-build": "true", - "runs-on-as-json-default": '["self-hosted", "Linux", "X64"]', + "runs-on-as-json-default": '["ubuntu-22.04"]', }, "potiuk", (), @@ -2075,7 +2075,7 @@ def test_mypy_matches( ("README.md",), { "is-committer-build": "false", - "runs-on-as-json-default": '["self-hosted", "Linux", "X64"]', + "runs-on-as-json-default": '["ubuntu-22.04"]', }, "potiuk", ("non committer build",),