You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@helix.apache.org by ji...@apache.org on 2021/01/07 07:46:31 UTC

[helix] branch master updated: Fix the CI script definition so the file list is passed to the steps that execute unit test. (#1604)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new adc0136  Fix the CI script definition so the file list is passed to the steps that execute unit test. (#1604)
adc0136 is described below

commit adc01364c31f860b8aa5cbdfe2f373b28c905b67
Author: Jiajun Wang <jj...@linkedin.com>
AuthorDate: Wed Jan 6 23:46:22 2021 -0800

    Fix the CI script definition so the file list is passed to the steps that execute unit test. (#1604)
    
    Fix the CI script definition so the file list is passed to the steps that execute unit test.
---
 .github/workflows/Helix-PR-CI.yml | 64 ++++++++++++++++++++-------------------
 1 file changed, 33 insertions(+), 31 deletions(-)

diff --git a/.github/workflows/Helix-PR-CI.yml b/.github/workflows/Helix-PR-CI.yml
index a8b959b..5a0a420 100644
--- a/.github/workflows/Helix-PR-CI.yml
+++ b/.github/workflows/Helix-PR-CI.yml
@@ -11,34 +11,36 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v2
-    - name: Set up JDK 1.8
-      uses: actions/setup-java@v1
-      with:
-        java-version: 1.8
-    - name: Get Changed Files
-      run: git fetch origin master > /dev/null 2>&1; git diff --name-only origin/master HEAD
-      id: find-diff
-    - name: Build with Maven
-      run: mvn clean install -Dmaven.test.skip.exec=true
-    - name: Test metrics-common
-      run: cd metrics-common; mvn -q test
-      if: ${{ contains(steps.find-diff.outputs, 'metrics-common') && (success() || failure()) }}
-    - name: Test metadata-store-directory-common
-      run: cd metadata-store-directory-common; mvn -q test
-      if: ${{ contains(steps.find-diff.outputs, 'metadata-store-directory-common') && (success() || failure()) }}
-    - name: Test zookeeper-api
-      run: cd zookeeper-api; mvn -q test
-      if: ${{ contains(steps.find-diff.outputs, 'zookeeper-api') && (success() || failure()) }}
-    - name: Test helix-common
-      run: cd helix-common; mvn -q test
-      if: ${{ contains(steps.find-diff.outputs, 'helix-common') && (success() || failure()) }}
-    - name: Test helix-lock
-      run: cd helix-lock; mvn -q test
-      if: ${{ contains(steps.find-diff.outputs, 'helix-lock') && (success() || failure()) }}
-    - name: Test helix-rest
-      run: cd helix-rest; mvn -q test
-      if: ${{ contains(steps.find-diff.outputs, 'helix-rest') && (success() || failure()) }}
-    - name: Test helix-core
-      run: cd helix-core; mvn -q test
-      if: ${{ contains(steps.find-diff.outputs, 'helix-core') && (success() || failure()) }}
+      - uses: actions/checkout@v2
+      - name: Set up JDK 1.8
+        uses: actions/setup-java@v1
+        with:
+          java-version: 1.8
+      - name: Get Changed Files
+        run: |
+          git fetch origin master
+          echo "::set-output name=files::$(git diff --name-only origin/master HEAD)"
+        id: find-diff
+      - name: Build with Maven
+        run: mvn clean install -Dmaven.test.skip.exec=true
+      - name: Test metrics-common
+        run: cd metrics-common; mvn -q test
+        if: ${{ contains(steps.find-diff.outputs.files, 'metrics-common') && (success() || failure()) }}
+      - name: Test metadata-store-directory-common
+        run: cd metadata-store-directory-common; mvn -q test
+        if: ${{ contains(steps.find-diff.outputs.files, 'metadata-store-directory-common') && (success() || failure()) }}
+      - name: Test zookeeper-api
+        run: cd zookeeper-api; mvn -q test
+        if: ${{ contains(steps.find-diff.outputs.files, 'zookeeper-api') && (success() || failure()) }}
+      - name: Test helix-common
+        run: cd helix-common; mvn -q test
+        if: ${{ contains(steps.find-diff.outputs.files, 'helix-common') && (success() || failure()) }}
+      - name: Test helix-lock
+        run: cd helix-lock; mvn -q test
+        if: ${{ contains(steps.find-diff.outputs.files, 'helix-lock') && (success() || failure()) }}
+      - name: Test helix-rest
+        run: cd helix-rest; mvn -q test
+        if: ${{ contains(steps.find-diff.outputs.files, 'helix-rest') && (success() || failure()) }}
+      - name: Test helix-core
+        run: cd helix-core; mvn -q test
+        if: ${{ contains(steps.find-diff.outputs.files, 'helix-core') && (success() || failure()) }}