Checking existence and then deleting is not atomic, so try to delete and

ignore FileNotFoundError instead

Author: hategan

src/psij/executors/batch/batch_scheduler_executor.py

@@ -541,8 +541,12 @@ def _clean_submit_script(self, job: Job) -> None:
             assert isinstance(self.config, BatchSchedulerExecutorConfig)
             if not self.config.keep_files:
                 submit_file_path = self.work_directory / (job.id + '.job')
-                if submit_file_path.exists():
+                try:
                     submit_file_path.unlink()
+                except FileNotFoundError:
+                    # this can reasonably happen for attached jobs when the main
+                    # job cleans up the script instead
+                    pass
         except Exception as ex:
             logger.warning('Job %s: failed clean submit script: %s', job.id, ex)