dimas-b commented on code in PR #1897: URL: https://github.com/apache/polaris/pull/1897#discussion_r2153005510
########## .github/workflows/gradle.yml: ########## @@ -33,65 +33,99 @@ on: branches: [ "main" ] jobs: + + # TODO remove this job in a future PR build: runs-on: ubuntu-latest permissions: contents: read + steps: + - name: Always succeeds + run: echo "Success!" + unit-tests: + name: Unit Tests + runs-on: ubuntu-latest + permissions: + contents: read steps: - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4 - name: Set up JDK 21 uses: actions/setup-java@c5195efecf7bdfc987ee8bae7a71cb8b11521c00 # v4 with: java-version: '21' distribution: 'temurin' - - # Configure Gradle for optimal use in GiHub Actions, including caching of downloaded dependencies. - # See: https://github.com/gradle/actions/blob/main/setup-gradle/README.md - name: Setup Gradle uses: gradle/actions/setup-gradle@ac638b010cf58a27ee6c972d7336334ccaf61c96 # v4 with: - # The setup-gradle action fails, if the wrapper is not using the right version or is not present. - # Our `gradlew` validates the integrity of the `gradle-wrapper.jar`, so it's safe to disable this. validate-wrappers: false - - - name: Code style checks and tests - run: ./gradlew --continue check - - - name: Check Maven publication - run: ./gradlew publishToMavenLocal sourceTarball - + - name: Run unit tests + run: | + ./gradlew check sourceTarball distTar distZip publishToMavenLocal \ + -x :polaris-quarkus-service:test \ + -x :polaris-quarkus-admin:test \ + -x intTest --continue - name: Archive test results uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4 if: always() with: - name: upload-test-artifacts + name: upload-unit-test-artifacts path: | **/build/test-results/** - - name: Stop Gradle daemons - run: ./gradlew --stop - - # Ensure that the build works properly when building against the "latest greatest" Java version - - name: Set up JDK 23 + quarkus-tests: + name: Quarkus Tests + runs-on: ubuntu-latest + permissions: + contents: read + steps: + - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4 + - name: Set up JDK 21 uses: actions/setup-java@c5195efecf7bdfc987ee8bae7a71cb8b11521c00 # v4 with: - java-version: '23' + java-version: '21' distribution: 'temurin' - - name: Show Java version - run: java -version - - name: Clean - run: ./gradlew clean - - name: Build - run: ./gradlew compileAll - - name: Run selected tests - run: ./gradlew :polaris-quarkus-service:intTest + - name: Setup Gradle + uses: gradle/actions/setup-gradle@ac638b010cf58a27ee6c972d7336334ccaf61c96 # v4 + with: + validate-wrappers: false + - name: Run Quarkus tests + run: | + ./gradlew \ + :polaris-quarkus-service:test \ Review Comment: I'd assume `intTest` takes the longest time in general, so it might be preferable to break it down if the goal is to reduce CI time :thinking: -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org