Merge pull request #12217 from amaltaro/fix-12216

Populate WMCMSSWSubprocess section for failed jobs too
dmwm · Jan 7, 2025 · ede4d00 · ede4d00
2 parents f44dd28 + a5ce72c
commit ede4d00
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/src/python/WMCore/WMSpec/Steps/Executors/CMSSW.py b/src/python/WMCore/WMSpec/Steps/Executors/CMSSW.py
@@ -303,6 +303,12 @@ def execute(self, emulator=None):
         envOverride['PYTHONPATH'] = pythonPath
         os.environ.update(envOverride)
 
+        try:
+            # Update the job report with the relevant WMCMSSWSubprocess metrics
+            self.report.updateSubprocessInfo(sysTime, userTime, startTime, endTime)
+        except Exception as ex:
+            logging.error("Error updating job report with WMCMSSWSubprocess metrics: %s", str(ex))
+
         if returnCode != 0:
             argsDump = {'arguments': args}
             msg = "Error running cmsRun\n%s\n" % argsDump
@@ -322,9 +328,6 @@ def execute(self, emulator=None):
             self._setStatus(returnCode, returnMessage)
 
         try:
-            # update FJR report info with subprocess info
-            self.report.updateSubprocessInfo(sysTime, userTime, startTime, endTime)
-
             # parse job report XML
             self.report.parse(jobReportXML, stepName=self.stepName)
         except Exception as ex: