Check for Memory Report: before reporting step

cms-sw · Sep 7, 2024 · 01b1403 · 01b1403
1 parent 3958b22
commit 01b1403
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/pr_testing/run-pr-comparisons b/pr_testing/run-pr-comparisons
@@ -402,7 +402,12 @@ for WF in ${WORKFLOWS_TO_COMPARE//,/ }; do
     $CMS_BOT_DIR/comparisons/compare-maxmem.py $WORKSPACE/data/PR-${PR_NUM}/$WF_DIR/maxmem_profile_${WF_NUMBER}.txt $WORKSPACE/data/$COMPARISON_RELEASE/$WF_DIR/maxmem_profile_${WF_NUMBER}.txt > $OUTPUT_DIR/${WF_NUMBER}.txt 2> $OUTPUT_DIR/${WF_NUMBER}.err || true
   fi
 done
-grep -q "exceeds threshold" $OUTPUT_DIR/*.err 2>/dev/null || echo "MAXMEM_COMPARISON${TEST_FLAVOR_STR};OK,max memory used ${UC_TEST_FLAVOR} comparison,See results,/SDT/jenkins-artifacts/${OUTPUT_DIR}/" >> ${RESULTS_FILE}
+if grep "exceeds threshold" $OUTPUT_DIR/*.err 2>/dev/null then
+    echo "MAXMEM_COMPARISON${TEST_FLAVOR_STR};OK,max memory used ${UC_TEST_FLAVOR} comparison,See results,/SDT/jenkins-artifacts/${OUTPUT_DIR}/" >> ${RESULTS_FILE}
+else
+    echo "MAXMEM_COMPARISON${TEST_FLAVOR_STR};OK,max memory used ${UC_TEST_FLAVOR} comparison,See results,/SDT/jenkins-artifacts/${OUTPUT_DIR}/" >> ${RESULTS_FILE}
+fi 
+
 # ----------------------------------------------------------------------------
 # Alternative Comparison
 # ----------------------------------------------------------------------------

diff --git a/run-ib-pr-matrix.sh b/run-ib-pr-matrix.sh
@@ -88,8 +88,10 @@ pushd "$WORKSPACE/matrix-results"
       memFile=maxmem_profile_$(echo $WF | cut -d_ -f1).txt
       pushd $WF
       for log in $(ls step*.log);do
-        echo ${log} | cut -d_ -f1 >> ${memFile}
-        grep "Memory Report: " $log | tail -5 >> ${memFile}
+	if grep -q "Memory Report: " $log 2>/dev/null; then
+            echo ${log} | cut -d_ -f1 >> ${memFile}
+            grep "Memory Report: " $log | tail -5 >> ${memFile}
+        fi
       done
       popd
     done