You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by as...@apache.org on 2021/01/07 13:53:55 UTC

[airflow] branch master updated: When "full tests needed" label is present, run all tests (#13538)

This is an automated email from the ASF dual-hosted git repository.

ash pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/master by this push:
     new ef2123c  When "full tests needed" label is present, run all tests (#13538)
ef2123c is described below

commit ef2123c671b985953707b056ef30b724f7f97d28
Author: Ash Berlin-Taylor <as...@firemirror.com>
AuthorDate: Thu Jan 7 13:53:43 2021 +0000

    When "full tests needed" label is present, run all tests (#13538)
    
    This label can be set manually, and the expectation is that it would run
    all tests, but in some cases for PRs that don't touch code, it would
    still skip most of the matrix and run minimal static checks only.
    
    This changes the behaviour so that if the label is found it will run all
    tests no matter what files have been changed -- making the label name
    more true :)
---
 scripts/ci/selective_ci_checks.sh | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/scripts/ci/selective_ci_checks.sh b/scripts/ci/selective_ci_checks.sh
index 314f59f..db37365 100755
--- a/scripts/ci/selective_ci_checks.sh
+++ b/scripts/ci/selective_ci_checks.sh
@@ -455,6 +455,10 @@ function run_all_tests_if_environment_files_changed() {
         echo "Important environment files changed. Running everything"
         set_outputs_run_everything_and_exit
     fi
+    if [[ ${FULL_TESTS_NEEDED_LABEL} == "true" ]]; then
+        echo "Full tests requested by label on PR. Running everything"
+        set_outputs_run_everything_and_exit
+    fi
     start_end::group_end
 }