Merge pull request #441 from GATEOverflow/mlperf-inference

Fixes for get-platform-details

.github/workflows/test-nvidia-mlperf-implementation.yml

@@ -2,7 +2,7 @@ name: MLPerf Inference Nvidia implementations
 
 on:
   schedule:
-    - cron: "23 20 * * *" #to be adjusted
+    - cron: "25 21 * * *" #to be adjusted
 
 jobs:
   build_nvidia:

script/get-generic-sys-util/run.sh

@@ -1,6 +1,19 @@
-#!/bin/bash
+# Safe execution of a command stored in a variable
+cmd="${CM_SYS_UTIL_INSTALL_CMD}"
+echo "$cmd"
 
-cmd=${CM_SYS_UTIL_INSTALL_CMD}
-echo $cmd
-eval $cmd
-test $? -eq 0 || exit $?
+# Execute the command and capture the exit status directly
+if ! eval "$cmd"; then
+    echo "Command failed with status $?"
+    if [[ "${CM_TMP_FAIL_SAFE}" == 'yes' ]]; then
+        # Exit safely if fail-safe is enabled
+        echo "Fail-safe is enabled, exiting with status 0"
+        exit 0
+    else
+        # Otherwise exit with the actual error status
+        exit $?
+    fi
+else
+    #echo "Command succeeded"
+    exit 0
+fi

script/get-platform-details/_cm.json

@@ -66,7 +66,10 @@
           "linux"
         ]
       },
-      "tags": "get,sys-util,generic,_linux-tools"
+      "tags": "get,sys-util,generic,_linux-tools",
+      "env": {
+        "CM_TMP_FAIL_SAFE": "yes"
+      }
     }
   ],
   "tags": [

script/get-platform-details/run.sh

@@ -79,7 +79,7 @@ echo "------------------------------------------------------------" >> $OUTPUT_F
 
 echo "14. cpupower frequency-info" >> $OUTPUT_FILE
 eval "cpupower frequency-info" >> $OUTPUT_FILE
-test $? -eq 0 || exit $?
+test $? -eq 0 || echo "FAILED: cpupower frequency-info" >> $OUTPUT_FILE
 echo "------------------------------------------------------------" >> $OUTPUT_FILE
 
 echo "15. sysctl" >> $OUTPUT_FILE
@@ -120,7 +120,7 @@ echo "------------------------------------------------------------" >> $OUTPUT_F
 echo "21. dmidecode" >> $OUTPUT_FILE
 if [[ ${CM_SUDO_USER} == "yes" ]]; then
     eval "${CM_SUDO} dmidecode" >> $OUTPUT_FILE
-    test $? -eq 0 || exit $?
+    test $? -eq 0 || echo "FAILED: dmidecode" >> $OUTPUT_FILE
 else
     echo "Requires SUDO permission" >> $OUTPUT_FILE
 fi
@@ -129,7 +129,7 @@ echo "------------------------------------------------------------" >> $OUTPUT_F
 echo "22. BIOS" >> $OUTPUT_FILE
 if [[ ${CM_SUDO_USER} == "yes" ]]; then
     eval "${CM_SUDO} dmidecode -t bios" >> $OUTPUT_FILE
-    test $? -eq 0 || exit $?
+    test $? -eq 0 || echo "FAILED: dmidecode -t bios" >> $OUTPUT_FILE
 else
     echo "Requires SUDO permission" >> $OUTPUT_FILE
 fi