fix: AREX out and err need to exist before file integrity check

aldbr · Apr 22, 2024 · 43a9f5c · 43a9f5c
1 parent e39e36f
commit 43a9f5c
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)