From 7042b36f29d21ca90cc7dd89c69015d5a192d352 Mon Sep 17 00:00:00 2001 From: Fran Mulero Date: Tue, 11 Jul 2023 13:06:13 +0200 Subject: [PATCH] [bitnami/containers] Run license check on modified files #17542 (#40523) * Run license check on modified files Signed-off-by: Fran Mulero * Fix Dockerfile filter Signed-off-by: Fran Mulero --------- Signed-off-by: Fran Mulero --- .github/workflows/license-headers.yml | 28 ++++++++++++++++++++++++++- 1 file changed, 27 insertions(+), 1 deletion(-) diff --git a/.github/workflows/license-headers.yml b/.github/workflows/license-headers.yml index 43c5de54101b..a131c74d1db0 100644 --- a/.github/workflows/license-headers.yml +++ b/.github/workflows/license-headers.yml @@ -25,5 +25,31 @@ jobs: with: ref: ${{ github.event.pull_request.head.ref }} repository: ${{ github.event.pull_request.head.repo.full_name }} + - id: get-modified-files + name: 'Get modified files' + env: + DIFF_URL: "${{github.event.pull_request.diff_url}}" + TEMP_FILE: "${{runner.temp}}/pr-${{github.event.number}}.diff" + run: | + # This request doesn't consume API calls. + curl -Lkso $TEMP_FILE $DIFF_URL + files_changed="$(sed -nr 's/[\-\+]{3} [ab]\/(.*)/\1/p' $TEMP_FILE | sort | uniq)" + dockerfiles=() + while read -r file_changed; do + # Avoid removed files + if [[ -f "${file_changed}" ]]; then + dockerfiles+=("${file_changed}") + fi + done <<< "$(echo "$files_changed" | grep -oE ".*/Dockerfile$" | sort | uniq || true)" + if [[ ${#dockerfiles[@]} -gt 0 ]]; then + # There are modifications on dockerfiles + export dockerfiles_json=$(printf "%s\n" "${dockerfiles[@]}" | jq -R . | jq -cs .) + # Overwrite configuration file to analyze only changed dockerfiles + yq -i '. | .header.paths=env(dockerfiles_json)' .licenserc.yaml + echo "result=success" >> $GITHUB_OUTPUT + else + echo "result=skip" >> $GITHUB_OUTPUT + fi - name: Check license Headers - uses: apache/skywalking-eyes/header@v0.4.0 \ No newline at end of file + uses: apache/skywalking-eyes/header@v0.4.0 + if: ${{ steps.get-modified-files.outputs.result == 'success' }} \ No newline at end of file