Merge pull request #7588 from aldbr/v8.0_FIX_RemoteRunner-outputs2

[v8.0] fix: AREX out and err need to exist before file integrity check
DIRACGrid · Apr 22, 2024 · e57a310 · e57a310
2 parents e39e36f + 43a9f5c
commit e57a310
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/DIRAC/Resources/Computing/AREXComputingElement.py b/src/DIRAC/Resources/Computing/AREXComputingElement.py
@@ -898,10 +898,8 @@ def getJobOutput(self, jobID, workingDirectory=None):
             if remoteOutput == f"{stamp}.out":
                 with open(localOutput) as f:
                     stdout = f.read()
-                os.unlink(localOutput)
             if remoteOutput == f"{stamp}.err":
                 with open(localOutput) as f:
                     stderr = f.read()
-                os.unlink(localOutput)
 
         return S_OK((stdout, stderr))
diff --git a/src/DIRAC/WorkloadManagementSystem/Utilities/RemoteRunner.py b/src/DIRAC/WorkloadManagementSystem/Utilities/RemoteRunner.py
@@ -128,6 +128,8 @@ def execute(self, command, workingDirectory=".", numberOfProcessors=1, cleanRemo
         self.log.info("The output has been retrieved and declared complete")
 
         # Clean up the job (local files not needed anymore)
+        os.remove(f"{stamp}.out")
+        os.remove(f"{stamp}.err")
         os.remove(self.checkSumOutput)
         os.remove(self.executable)