Ticket 11049 - Documentation of sacct --parsable should refer to --delimiter
Summary: Documentation of sacct --parsable should refer to --delimiter
Status: RESOLVED FIXED
Alias: None
Product: Slurm
Classification: Unclassified
Component: Documentation (show other tickets)
Version: 20.02.6
Hardware: Linux Linux
: --- 4 - Minor Issue
Assignee: Ben Roberts
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2021-03-10 04:48 MST by Ole.H.Nielsen@fysik.dtu.dk
Modified: 2021-03-11 12:00 MST (History)
0 users

See Also:
Site: DTU Physics
Alineos Sites: ---
Atos/Eviden Sites: ---
Confidential Site: ---
Coreweave sites: ---
Cray Sites: ---
DS9 clusters: ---
HPCnow Sites: ---
HPE Sites: ---
IBM Sites: ---
NOAA SIte: ---
OCF Sites: ---
Recursion Pharma Sites: ---
SFW Sites: ---
SNIC sites: ---
Linux Distro: ---
Machine Name:
CLE Version:
Version Fixed: 20.11.5, 21.08.pre1
Target Release: ---
DevPrio: ---
Emory-Cloud Sites: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this ticket.
Description Ole.H.Nielsen@fysik.dtu.dk 2021-03-10 04:48:24 MST 
As discussed on the slurm-users mailing list, the sacct command options --parsable and --parsable2 may be modified by the --delimiter option.

Please add to the sacct man-page under --parsable and --parsable2 text like:

The '|' delimiter may be modified by the --delimiter option.

Thanks,
Ole
Comment 2 Ben Roberts 2021-03-10 10:07:36 MST 
Hi Ole,

Thank you for bringing this up.  I've put together a patch to reference the --delimiter option in the --parsable options of sacct.  I'll let you know when there is progress with this update to the docs.

Thanks,
Ben
Comment 3 Tim Wickberg 2021-03-10 18:25:29 MST 
Okay @ 20.11. Thanks Ben.
Comment 5 Ben Roberts 2021-03-11 08:41:01 MST 
Thanks again for bringing this up Ole.  The change has been checked in that refers users to the --delimiter option from the --parsable option.  This change will be reflected when 20.11.5 is released.  For your reference the commit can be found here.  

https://github.com/SchedMD/slurm/commit/897647ccc7e57034872695c7a073908909452b99

Thanks,
Ben
Comment 6 Ole.H.Nielsen@fysik.dtu.dk 2021-03-11 12:00:50 MST 
(In reply to Ben Roberts from comment #5)
> Thanks again for bringing this up Ole.  The change has been checked in that
> refers users to the --delimiter option from the --parsable option.  This
> change will be reflected when 20.11.5 is released.  For your reference the
> commit can be found here.  
> 
> https://github.com/SchedMD/slurm/commit/
> 897647ccc7e57034872695c7a073908909452b99

Thanks a lot Ben!

Ole