mumrah commented on code in PR #16936:
URL: https://github.com/apache/kafka/pull/16936#discussion_r1723796433


##########
.github/workflows/pr.yml:
##########
@@ -68,35 +50,17 @@ jobs:
       fail-fast: false
       matrix:
         java: [ 11, 17 ]
+    if: contains(github.ref, 'gh-')  # For now, only run tests on "gh-*" PRs
     name: JUnit tests Java ${{ matrix.java }}
     steps:
-      - name: Checkout code
-        uses: actions/checkout@v4
-        with:
-          persist-credentials: false
-      - name: Setup Java
-        uses: actions/setup-java@v4
-        with:
-          distribution: temurin
-          java-version: ${{ matrix.java }}
       - name: Setup Gradle
-        uses: gradle/actions/setup-gradle@v4
-        env:
-          GRADLE_BUILD_ACTION_CACHE_DEBUG_ENABLED: true
+        uses: ./.github/actions/setup-gradle
         with:
-          gradle-version: wrapper
-          develocity-access-key: ${{ secrets.GE_ACCESS_TOKEN }}
-          # Only write to the cache from trunk
-          #cache-read-only: ${{ github.ref != 'refs/heads/trunk' }}
-          cache-read-only: false
-          # Cache downloaded JDKs in addition to the default directories.
-          gradle-home-cache-includes: |
-            caches
-            notifications
-            jdks
-          cache-cleanup: on-success
-          cache-overwrite-existing: true
+          java-version: ${{ matrix.java }}
+          gradle-cache-read-only: true
       - name: Test
+        # Publish build scans for now. This will only work on PRs from 
apache/kafka, not public forks.

Review Comment:
   I think all our PRs come from forks normally. Leaving this in here does give 
committers a way to get a build scan from a non-trunk code base.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to