adoroszlai commented on code in PR #8679: URL: https://github.com/apache/ozone/pull/8679#discussion_r2161940890
########## .github/workflows/intermittent-test-check.yml: ########## @@ -95,54 +94,11 @@ jobs: with: repo: ${{ github.event.inputs.ratis-repo || format('{0}/ratis', github.repository_owner) }} ref: ${{ github.event.inputs.ratis-ref }} - find-tests: - if: ${{ always() && github.event.inputs.submodule == 'AUTO' }} - needs: - - prepare-job - runs-on: ubuntu-24.04 - outputs: - modules: ${{ steps.modules.outputs.modules }} - steps: - - uses: actions/checkout@v4 - with: - ref: ${{ github.event.inputs.ref }} - - name: Cache for maven dependencies - uses: actions/cache/restore@v4 - with: - path: | - ~/.m2/repository/*/*/* - !~/.m2/repository/org/apache/ozone - key: maven-repo-${{ hashFiles('**/pom.xml') }} - restore-keys: | - maven-repo- - - name: Setup java - uses: actions/setup-java@v4 - with: - distribution: 'temurin' - java-version: ${{ github.event.inputs.java-version }} - - name: Find tests - run: | - # find tests to be run in splits by running them with very short timeout - hadoop-ozone/dev-support/checks/junit.sh -DexcludedGroups="native|slow|unhealthy" -DskipShade \ - -Dtest="$TEST_CLASS,Abstract*Test*\$*" \ - -Dsurefire.fork.timeout=1 -Dmaven-surefire-plugin.version=${{ env.SUREFIRE_VERSION }} \ Review Comment: > So IIUC we cannot use new version because of the fork not getting terminated correctly and can't use the old version as it does not print the module correctly for some reason? Yes. -- 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...@ozone.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org For additional commands, e-mail: issues-h...@ozone.apache.org