From 966e7969328188a6f054148c03b2da3f67fd04ad Mon Sep 17 00:00:00 2001 From: NiniOak Date: Mon, 18 Sep 2023 12:30:18 -0700 Subject: [PATCH] Update base ref property name --- .../scripts/filter_changed_files_go_test.sh | 46 ++++++++++--------- .github/workflows/test-integrations.yml | 2 + 2 files changed, 26 insertions(+), 22 deletions(-) diff --git a/.github/scripts/filter_changed_files_go_test.sh b/.github/scripts/filter_changed_files_go_test.sh index 1366cc2eeb266..efeb54ecd598c 100755 --- a/.github/scripts/filter_changed_files_go_test.sh +++ b/.github/scripts/filter_changed_files_go_test.sh @@ -2,36 +2,38 @@ # Copyright (c) HashiCorp, Inc. # SPDX-License-Identifier: BUSL-1.1 +set -euo pipefail # Get the list of changed files -files_to_check=$(git diff --name-only origin/$GITHUB_BASE_REF) +files_to_check=$(git diff --name-only origin/$BRANCH) # Define the directories to check -skipped_directories=("docs/" "ui/" "website/" "grafana/") +skipped_directories=("docs/" "ui/" "website/" "grafana/" ".github/") # Initialize a variable to track directories outside the skipped ones other_directories="" trigger_ci=true -# # Loop through the changed files and find directories/files outside the skipped ones -# for file_to_check in $files_to_check; do -# file_is_skipped=false -# for dir in "${skipped_directories[@]}"; do -# if [[ "$file_to_check" == "$dir"* ]] || [[ "$file_to_check" == *.md && "$dir" == *"/" ]]; then -# file_is_skipped=true -# break -# fi -# done -# if [ "$file_is_skipped" = "false" ]; then -# other_directories+="$(dirname "$file_to_check")\n" -# trigger_ci=true -# echo "Non doc file(s) changed - triggered ci: $trigger_ci" -# echo -e $other_directories -# echo "trigger-ci=$trigger_ci" >>"$GITHUB_OUTPUT" -# exit 0 ## if file is outside of the skipped_directory exit script -# fi -# done +# Loop through the changed files and find directories/files outside the skipped ones +for file_to_check in $files_to_check; do + file_is_skipped=false + for dir in "${skipped_directories[@]}"; do + if [[ "$file_to_check" == "$dir"* ]] || [[ "$file_to_check" == *.md && "$dir" == *"/" ]]; then + file_is_skipped=true + break + fi + done + if [ "$file_is_skipped" = "true" ]; then + trigger_ci=false + other_directories+="$(dirname "$file_to_check")\n" + echo -e $other_directories + echo "Only doc file(s) changed - triggered ci: $trigger_ci" + echo "trigger-ci=$trigger_ci" >>"$GITHUB_OUTPUT" + exit 0 ## if file is in list of the skipped_directory, exit script + fi +done -# echo "Only doc file(s) changed - triggered ci: $trigger_ci" -echo "Doc file(s) change detection is currently disabled - triggering ci" +other_directories+="$(dirname "$file_to_check")\n" +echo -e $other_directories +echo "Non doc file(s) changed - triggered ci: $trigger_ci" echo "trigger-ci=$trigger_ci" >>"$GITHUB_OUTPUT" diff --git a/.github/workflows/test-integrations.yml b/.github/workflows/test-integrations.yml index 70315390e3b02..179c0204d03c4 100644 --- a/.github/workflows/test-integrations.yml +++ b/.github/workflows/test-integrations.yml @@ -41,6 +41,8 @@ jobs: fetch-depth: 0 - name: Get changed files id: read-files + env: + BRANCH: ${{ github.head_ref || github.ref_name }} run: ./.github/scripts/filter_changed_files_go_test.sh setup: