Ticket 11049

Summary:	Documentation of sacct --parsable should refer to --delimiter
Product:	Slurm	Reporter:	Ole.H.Nielsen <Ole.H.Nielsen>
Component:	Documentation	Assignee:	Ben Roberts <ben>
Status:	RESOLVED FIXED	QA Contact:
Severity:	4 - Minor Issue
Priority:	---
Version:	20.02.6
Hardware:	Linux
OS:	Linux
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:	---

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