Cleanups for mlperf inference get-platform-details

script/app-mlperf-inference/_cm.yaml

@@ -111,6 +111,17 @@ deps:
 
 posthook_deps:
   - tags: get,mlperf,sut,description #populate system meta information like framework
+  - tags: get,platform,details
+    enable_if_any_env:
+      CM_SUDO_USER:
+        - yes
+      CM_GET_PLATFORM_DETAILS:
+        - yes
+    skip_if_env:
+      CM_MLPERF_LOADGEN_MODE:
+      - accuracy
+    env: 
+      CM_PLATFORM_DETAILS_FILE_PATH: '<<<CM_MLPERF_OUTPUT_DIR>>>/system_info.txt'
 
 # Order of variations for documentation
 variation_groups_order:
@@ -1052,14 +1063,6 @@ variations:
       - mlperf-accuracy-script
       - imagenet-accuracy-script
       tags: run,accuracy,mlperf,_imagenet
-    - tags: get,platform,details
-      enable_if_any_env:
-        CM_SUDO_USER:
-          - yes
-        CM_GET_PLATFORM_DETAILS:
-          - yes
-      env: 
-        CM_PLATFORM_DETAILS_FILE_PATH: '<<<CM_MLPERF_OUTPUT_DIR>>>/system_info.txt'
 
   onnxruntime:
     group: backend

script/get-platform-details/customize.py

@@ -16,18 +16,16 @@ def preprocess(i):
     if os_info['platform'] == "windows":
         return {'return':1, 'error':'get-platform-details script not yet supported in windows!'}
 
-    print(env['CM_HOST_OS_KERNEL_VERSION'])
-
     if not check_installation("numactl",os_info):
         env['CM_INSTALL_NUMACTL'] = 'True'
 
     #if not check_installation("cpupower",os_info):
     env['CM_INSTALL_CPUPOWER'] = 'True'
 
-   if env.get('CM_PLATFORM_DETAILS_FILE_PATH', '') == '':
-       env['CM_GET_PLATFORM_DETAILS_FILE_PATH'] = os.path.join(os.getcwd(), "system_info.txt")
+    if env.get('CM_PLATFORM_DETAILS_FILE_PATH', '') == '':
+       env['CM_PLATFORM_DETAILS_FILE_PATH'] = os.path.join(os.getcwd(), "system_info.txt")
 
-   return {'return':0}
+    return {'return':0}
 
 
 def postprocess(i):

script/get-platform-details/run.sh

@@ -1,8 +1,9 @@
 #!/bin/bash
 
-OUTPUT_FILE="system_info.txt"
-
-echo "WARNING: sudo permission is needed to some packages for measuring the platform details"
+OUTPUT_FILE="$CM_PLATFORM_DETAILS_FILE_PATH"
+set -e
+#echo $OUTPUT_FILE
+echo "WARNING: sudo permission is needed for some of the below commands"
 
 if [[ ${CM_HOST_OS_FLAVOR} == "macos" ]]; then
     echo "WARNING: To be done for the mac os"