[manuf] Change log file move method in orchestrator

shutil.move() is more robust than os.rename(). If the source and destination are on different partitions, os.rename() throws "OSError: [Errno 18] Invalid cross-device link". Signed-off-by: Noah Moroze <[email protected]>
lowRISC · Dec 3, 2024 · f22a505 · f22a505
1 parent cfa534a
commit f22a505
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/sw/host/provisioning/orchestrator/src/ot_dut.py b/sw/host/provisioning/orchestrator/src/ot_dut.py
@@ -6,6 +6,7 @@
 import logging
 import os
 import re
+import shutil
 import tempfile
 from dataclasses import dataclass
 
@@ -156,8 +157,8 @@ def run_cp(self) -> None:
             self.device_id.update_base_id(cp_device_id)
 
             self._make_log_dir()
-            os.rename(stdout_logfile, f"{self.log_dir}/cp_out.log.txt")
-            os.rename(stderr_logfile, f"{self.log_dir}/cp_err.log.txt")
+            shutil.move(stdout_logfile, f"{self.log_dir}/cp_out.log.txt")
+            shutil.move(stderr_logfile, f"{self.log_dir}/cp_err.log.txt")
 
         self.cp_data = chip_probe_data
         logging.info(f"CP logs saved to {self.log_dir}.")