[airflow] 02/02: Disable Helm tests when branch is not `main`

[po...@apache.org]

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

potiuk pushed a commit to branch v2-1-test
in repository https://gitbox.apache.org/repos/asf/airflow.git

commit 429e66313d8d12fb5e6e75abec58cda9291469fd
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Fri Aug 6 09:10:02 2021 +0200

    Disable Helm tests when branch is not `main`
    
    We are preparing Helm chart from `main` branch only and we never
    run it from airflow version branches (similarly as providers)
    
    This change disables Helm Chart tests in case default branch
    is different than main.
---
 scripts/ci/selective_ci_checks.sh | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/scripts/ci/selective_ci_checks.sh b/scripts/ci/selective_ci_checks.sh
index ee3b506..6dff38b 100755
--- a/scripts/ci/selective_ci_checks.sh
+++ b/scripts/ci/selective_ci_checks.sh
@@ -213,16 +213,19 @@ function needs_ui_tests() {
 
 if [[ ${DEFAULT_BRANCH} == "main" ]]; then
     ALL_TESTS="Always API Core Other CLI Providers WWW Integration"
+    MIGHT_NEED_HELM_TESTS="true"
 else
     # Skips Provider tests in case current default branch is not main
     ALL_TESTS="Always API Core Other CLI WWW Integration"
+    # Skips Helm tests in case current default branch is not main
+    MIGHT_NEED_HELM_TESTS="false"
 fi
 readonly ALL_TESTS
 
 function set_outputs_run_everything_and_exit() {
     needs_api_tests "true"
     needs_api_codegen "true"
-    needs_helm_tests "true"
+    needs_helm_tests ${MIGHT_NEED_HELM_TESTS}
     needs_javascript_scans "true"
     needs_python_scans "true"
     run_tests "true"
@@ -405,7 +408,11 @@ function check_if_helm_tests_should_be_run() {
     if [[ $(count_changed_files) == "0" ]]; then
         needs_helm_tests "false"
     else
-        needs_helm_tests "true"
+        if [[ ${MIGHT_NEED_HELM_TESTS} == "true" ]]; then
+            needs_helm_tests "true"
+        else
+            needs_helm_tests "false"
+        fi
     fi
     start_end::group_end
 }