From 6d6cf8ebaf4e74152ccd5c60f5f09af3de0fd01f Mon Sep 17 00:00:00 2001
From: gardar <gardar@users.noreply.github.com>
Date: Sun, 21 May 2023 03:00:17 +0000
Subject: [PATCH] skip_changelog: unify automatic and manual ansible role tests
 with labels

Signed-off-by: gardar <gardar@users.noreply.github.com>
---
 .github/workflows/ansible-ci.yml | 55 ++++++++++++++++++++++++--------
 1 file changed, 41 insertions(+), 14 deletions(-)

diff --git a/.github/workflows/ansible-ci.yml b/.github/workflows/ansible-ci.yml
index 790259e5..d9435751 100644
--- a/.github/workflows/ansible-ci.yml
+++ b/.github/workflows/ansible-ci.yml
@@ -58,8 +58,41 @@ jobs:
               "skip_changelog": "skip_changelog"
               }
 
+  role-label:
+    runs-on: ubuntu-latest
+    needs: pr-label
+    if: github.event.pull_request.labels.length == 0
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: Get changed roles
+        id: changed-roles
+        uses: tj-actions/changed-files@v35
+        with:
+          path: "roles"
+          diff_relative: "true"
+          files: "**"
+          dir_names: "true"
+          dir_names_max_depth: "1"
+          since_last_remote_commit: "true"
+
+      - name: Add changed roles labels
+        uses: actions/github-script@v6
+        if: |
+          steps.changed-roles.outputs.all_changed_and_modified_files
+        with:
+          script: |
+            const labels = '${{ steps.changed-roles.outputs.all_changed_and_modified_files }}'.split(' ');
+            github.rest.issues.addLabels({
+              issue_number: context.issue.number,
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              labels: labels
+            })
+
   ansible-lint:
     runs-on: ubuntu-latest
+    needs: pr-label
     steps:
       - uses: actions/checkout@v3
 
@@ -68,6 +101,7 @@ jobs:
 
   discover-ansible-versions:
     runs-on: ubuntu-latest
+    needs: pr-label
     outputs:
       versions: ${{ steps.supported-ansible-versions.outputs.result }}
     steps:
@@ -91,7 +125,10 @@ jobs:
 
   discover-ansible-tests:
     runs-on: ubuntu-latest
-    needs: ansible-lint
+    needs:
+      - pr-label
+      - role-label
+      - ansible-lint
     outputs:
       molecule-tests: ${{ steps.set-molecule-tests.outputs.tests }}
       integration-tests: ${{ steps.set-integration-tests.outputs.tests }}
@@ -100,22 +137,12 @@ jobs:
         with:
           fetch-depth: 0
 
-      - name: Get changed roles
-        id: changed-roles
-        uses: tj-actions/changed-files@v35
-        with:
-          path: "roles"
-          diff_relative: "true"
-          files: "**"
-          dir_names: "true"
-          dir_names_max_depth: "1"
-          since_last_remote_commit: "true"
-
       - name: Discover role tests
         id: set-molecule-tests
+        env:
+          LABELS: ${{ toJson(github.event.pull_request.labels.*.name) }}
         run: |
-          roles=${{ steps.changed-roles.outputs.all_changed_and_modified_files }}
-          roles+=${{ github.event.label.name }}
+          roles=$(echo $LABELS | jq -r '.[]')
           echo tests="[`for role in $roles; do
             for test in $(find tests/integration/targets -maxdepth 1 -mindepth 1 -type d -iname "molecule-${role}-*" -printf "%f\n"); do
               echo '{"test":\"'"${test}"'\","name":\"'"${test#*-}\"'"}';