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

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


The following commit(s) were added to refs/heads/master by this push:
     new 560d017bc7 HDDS-12500. Do not skip JUnit tests in post-commit runs 
(#8024)
560d017bc7 is described below

commit 560d017bc767e85d5927151d35f5e5e6ac0ccd95
Author: Doroszlai, Attila <[email protected]>
AuthorDate: Wed Mar 26 02:00:55 2025 +0100

    HDDS-12500. Do not skip JUnit tests in post-commit runs (#8024)
---
 hadoop-ozone/dev-support/checks/_post_process.sh | 8 ++++----
 hadoop-ozone/dev-support/checks/junit.sh         | 8 +++++---
 2 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/hadoop-ozone/dev-support/checks/_post_process.sh 
b/hadoop-ozone/dev-support/checks/_post_process.sh
index ac05e2c000..7501050f34 100644
--- a/hadoop-ozone/dev-support/checks/_post_process.sh
+++ b/hadoop-ozone/dev-support/checks/_post_process.sh
@@ -26,13 +26,13 @@
 # - $REPORT_FILE should be defined
 # - Maven output should be saved in $REPORT_DIR/output.log
 
-# script failed, but report file is empty (does not reflect failure)
-if [[ ${rc} -ne 0 ]] && [[ ! -s "${REPORT_FILE}" ]]; then
-  # do we know what to look for?
+if [[ ! -s "${REPORT_FILE}" ]]; then
+  # check if there are errors in the log
   if [[ -n "${ERROR_PATTERN:-}" ]]; then
     grep -m25 "${ERROR_PATTERN}" "${REPORT_DIR}/output.log" > "${REPORT_FILE}"
   fi
-  if [[ ! -s "${REPORT_FILE}" ]]; then
+  # script failed, but report file is empty (does not reflect failure)
+  if [[ ${rc} -ne 0 ]] && [[ ! -s "${REPORT_FILE}" ]]; then
     echo "Unknown failure, check output.log" > "${REPORT_FILE}"
   fi
 fi
diff --git a/hadoop-ozone/dev-support/checks/junit.sh 
b/hadoop-ozone/dev-support/checks/junit.sh
index 46f1fd77f2..7c25293c5d 100755
--- a/hadoop-ozone/dev-support/checks/junit.sh
+++ b/hadoop-ozone/dev-support/checks/junit.sh
@@ -40,7 +40,7 @@ fi
 if [[ "${FAIL_FAST}" == "true" ]]; then
   MAVEN_OPTIONS="${MAVEN_OPTIONS} --fail-fast 
-Dsurefire.skipAfterFailureCount=1"
 else
-  MAVEN_OPTIONS="${MAVEN_OPTIONS} --fail-at-end"
+  MAVEN_OPTIONS="${MAVEN_OPTIONS} --fail-never"
 fi
 
 # apply module access args (for Java 9+)
@@ -50,8 +50,10 @@ if [[ -f 
hadoop-ozone/dist/src/shell/ozone/ozone-functions.sh ]]; then
   ozone_java_setup
 fi
 
+mvn ${MAVEN_OPTIONS} clean
+
 if [[ ${ITERATIONS} -gt 1 ]] && [[ ${OZONE_REPO_CACHED} == "false" ]]; then
-  mvn ${MAVEN_OPTIONS} -DskipTests clean install
+  mvn ${MAVEN_OPTIONS} -DskipTests install
 fi
 
 REPORT_DIR=${OUTPUT_DIR:-"$DIR/../../../target/${CHECK}"}
@@ -105,5 +107,5 @@ if [[ "${OZONE_WITH_COVERAGE}" == "true" ]]; then
   mvn -B -N jacoco:merge -Djacoco.destFile=$REPORT_DIR/jacoco-combined.exec 
-Dscan=false
 fi
 
-ERROR_PATTERN="\[ERROR\]"
+ERROR_PATTERN="BUILD FAILURE"
 source "${DIR}/_post_process.sh"


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to