GitHub Actions: make ./gradlew clean an independent pipeline step (#…

…2931) Motivation: Jakarta modules have some conflicts between `clean` and `check` tasks when they are executed for 2+ modules in a single run. Modification: - Run `clean` as an independent task; Result: We expect snapshot build pipelines to recover.
apple · May 14, 2024 · af629a8 · af629a8
1 parent f0e7aff
commit af629a8
Show file tree

Hide file tree

Showing 6 changed files with 24 additions and 12 deletions.
diff --git a/.github/workflows/ci-netty-snapshot.yml b/.github/workflows/ci-netty-snapshot.yml
@@ -27,11 +27,13 @@ jobs:
         env:
           ORG_GRADLE_PROJECT_nettyVersion: 4.1+
         run: ./gradlew :servicetalk-grpc-netty:dependencyInsight --configuration testRuntimeClasspath --dependency io.netty:netty-codec-http2 -PnettyVersion=4.1+ | grep "io.netty:netty-codec-http2"
+      - name: Clean Gradle project
+        run: ./gradlew --parallel clean
       - name: Build and Test
         env:
           JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8
           ORG_GRADLE_PROJECT_nettyVersion: 4.1.+
-        run:  sudo -E env "PATH=$PATH" bash -c "ulimit -l 65536 && ulimit -a && ./gradlew --no-daemon --parallel clean test"
+        run:  sudo -E env "PATH=$PATH" bash -c "ulimit -l 65536 && ulimit -a && ./gradlew --no-daemon --parallel test"
       - name: Publish Test Results
         if: always()
         uses: actions/upload-artifact@v4

diff --git a/.github/workflows/ci-prb.yml b/.github/workflows/ci-prb.yml
@@ -23,6 +23,8 @@ jobs:
         run: java -version
       - name: Make gradlew Executable
         run: chmod +x gradlew
+      - name: Clean Gradle project
+        run: ./gradlew --parallel clean
       # https://github.community/t/error-the-paging-file-is-too-small-for-this-operation-to-complete/17141
       - name: Configure Windows Pagefile
         if: ${{ runner.os == 'Windows' }}
@@ -35,12 +37,12 @@ jobs:
         if: runner.os == 'Linux'
         env:
           JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8
-        run:  sudo -E env "PATH=$PATH" bash -c "ulimit -l 65536 && ulimit -a && ./gradlew --no-daemon --parallel clean test"
+        run:  sudo -E env "PATH=$PATH" bash -c "ulimit -l 65536 && ulimit -a && ./gradlew --no-daemon --parallel test"
       - name: Build and Test (non-Linux)
         if: runner.os != 'Linux'
         env:
           JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8
-        run: ./gradlew --no-daemon --parallel clean test
+        run: ./gradlew --no-daemon --parallel test
       - name: Upload Test Results
         if: always()
         uses: actions/upload-artifact@v4

diff --git a/.github/workflows/ci-prq.yml b/.github/workflows/ci-prq.yml
@@ -24,8 +24,10 @@ jobs:
         run: chmod +x gradlew
       - name: Documentation Linter
         run: bash scripts/check-source-root.sh
+      - name: Clean Gradle project
+        run: ./gradlew --parallel clean
       - name: Build with Gradle
-        run: ./gradlew --parallel clean quality
+        run: ./gradlew --parallel quality
       - name: Upload CheckStyle Results
         if: always()
         uses: actions/upload-artifact@v4

diff --git a/.github/workflows/ci-release.yml b/.github/workflows/ci-release.yml
@@ -26,6 +26,8 @@ jobs:
         run: java -version
       - name: Make gradlew Executable
         run: chmod +x gradlew
+      - name: Clean Gradle project
+        run: ./gradlew --parallel clean
       - name: Build with Gradle
         env:
           ORG_GRADLE_PROJECT_signingKey: ${{ secrets.ORG_GRADLE_PROJECT_SIGNINGKEY }}
@@ -35,7 +37,7 @@ jobs:
         run: |
           # Build arguments to feed to the single gradlew publish command
           if [ "${{ matrix.java }}" = "8" ]; then
-            FIRST_GRADLE_TARGETS=" clean check"
+            FIRST_GRADLE_TARGETS=" check"
             SECOND_GRADLE_TARGETS=" --no-parallel publish"
           else
             FIRST_GRADLE_TARGETS=""
@@ -49,14 +51,14 @@ jobs:
               if [ "$javaTarget" = "1.9" ] || [ "$javaTarget" = "1.10" ] || [ "$javaTarget" -gt "8" ] 2> /dev/null
               then
                 currDir=$(echo "$line" | sed -e 's/^version:.* name: \(.*\)$/\1/g')
-                FIRST_GRADLE_TARGETS="$FIRST_GRADLE_TARGETS :$currDir:clean :$currDir:check"
+                FIRST_GRADLE_TARGETS="$FIRST_GRADLE_TARGETS :$currDir:check"
                 SECOND_GRADLE_TARGETS="$SECOND_GRADLE_TARGETS :$currDir:publish"
               fi
             done < <(./gradlew printJavaTargetCompatibility)
           fi
 
           # Execute the gradlew command to publish the build
-          sudo -E env "PATH=$PATH" bash -c "ulimit -l 65536 && ulimit -a && ./gradlew --no-daemon --parallel -PreleaseBuild=true$FIRST_GRADLE_TARGETS && ./gradlew --no-daemon -PreleaseBuild=true$SECOND_GRADLE_TARGETS"
+          sudo -E env "PATH=$PATH" bash -c "ulimit -l 65536 && ulimit -a && ./gradlew --no-daemon --parallel -PreleaseBuild=true$FIRST_GRADLE_TARGETS && ./gradlew --no-daemon --no-parallel -PreleaseBuild=true$SECOND_GRADLE_TARGETS"
       - name: Publish Test Results
         if: always()
         uses: scacap/action-surefire-report@a2911bd1a4412ec18dde2d93b1758b3e56d2a880

diff --git a/.github/workflows/ci-snapshot.yml b/.github/workflows/ci-snapshot.yml
@@ -27,6 +27,8 @@ jobs:
         run: java -version
       - name: Make gradlew Executable
         run: chmod +x gradlew
+      - name: Clean Gradle project
+        run: ./gradlew --parallel clean
       - name: Build with Gradle
         env:
           ORG_GRADLE_PROJECT_signingKey: ${{ secrets.ORG_GRADLE_PROJECT_SIGNINGKEY }}
@@ -36,7 +38,7 @@ jobs:
         run: |
           # Build arguments to feed to the single gradlew publish command
           if [ "${{ matrix.java }}" = "8" ]; then
-            FIRST_GRADLE_TARGETS=" clean check"
+            FIRST_GRADLE_TARGETS=" check"
             SECOND_GRADLE_TARGETS=" publish"
           else
             FIRST_GRADLE_TARGETS=""
@@ -50,14 +52,14 @@ jobs:
               if [ "$javaTarget" = "1.9" ] || [ "$javaTarget" = "1.10" ] || [ "$javaTarget" -gt "8" ] 2> /dev/null
               then
                 currDir=$(echo "$line" | sed -e 's/^version:.* name: \(.*\)$/\1/g')
-                FIRST_GRADLE_TARGETS="$FIRST_GRADLE_TARGETS :$currDir:clean :$currDir:check"
+                FIRST_GRADLE_TARGETS="$FIRST_GRADLE_TARGETS :$currDir:check"
                 SECOND_GRADLE_TARGETS="$SECOND_GRADLE_TARGETS :$currDir:publish"
               fi
             done < <(./gradlew printJavaTargetCompatibility)
           fi
 
           # Execute the gradlew command to publish the build
-          sudo -E env "PATH=$PATH" bash -c "ulimit -l 65536 && ulimit -a && ./gradlew --no-daemon --parallel$FIRST_GRADLE_TARGETS && ./gradlew --no-daemon$SECOND_GRADLE_TARGETS"
+          sudo -E env "PATH=$PATH" bash -c "ulimit -l 65536 && ulimit -a && ./gradlew --no-daemon --parallel$FIRST_GRADLE_TARGETS && ./gradlew --no-daemon --no-parallel$SECOND_GRADLE_TARGETS"
       - name: Publish Test Results
         if: always()
         uses: scacap/action-surefire-report@a2911bd1a4412ec18dde2d93b1758b3e56d2a880

diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
@@ -31,7 +31,9 @@ jobs:
         uses: github/codeql-action/init@v3
         with:
           languages: ${{ matrix.language }}
-      - name: Execute gradle build
-        run: ./gradlew --parallel clean assemble
+      - name: Clean Gradle project
+        run: ./gradlew --parallel clean
+      - name: Assemble Gradle project
+        run: ./gradlew --parallel assemble
       - name: Perform CodeQL Analysis
         uses: github/codeql-action/analyze@v3