Changed sacct -p option to -P to fix issue #24

neilmunday · Sep 14, 2021 · 04f6d4b · 04f6d4b
1 parent 04d4320
commit 04f6d4b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,14 @@
 Change Log
 ==========
 
+Version 2.4
+-----------
+
+Date: 2021-09-14
+
+* Changed `sacct` command option `-p` to `-P` to handle pipe symbol in job names (issue #24)
+. Thanks to @jitkang for the fix
+
 Version 2.3
 -----------
 

diff --git a/bin/slurm-send-mail.py b/bin/slurm-send-mail.py
@@ -213,7 +213,7 @@ def process_spool_file(f: pathlib.Path, first_job_id: int, state: str):
         field_str = ",".join(fields)
 
         # Get job info from sacct
-        cmd = (f"{sacct_exe} -j {first_job_id} -p -n --fields={field_str}")
+        cmd = (f"{sacct_exe} -j {first_job_id} -P -n --fields={field_str}")
         rc, stdout, stderr = run_command(cmd)
         if rc != 0:
             logging.error("Failed to run {0}".format(cmd))