Ticket 10236

Summary:	SLURM_JOB_QOS is not documented for prolog/epilog on website
Product:	Slurm	Reporter:	Chris Samuel (NERSC) <csamuel>
Component:	Documentation	Assignee:	Ben Roberts <ben>
Status:	RESOLVED FIXED	QA Contact:
Severity:	4 - Minor Issue
Priority:	---	CC:	alex, dmjacobsen
Version:	20.02.5
Hardware:	Linux
OS:	Linux
See Also:	https://bugs.schedmd.com/show_bug.cgi?id=9258
Site:	NERSC	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.3, 21.08.0pre1	Target Release:	---
DevPrio:	---	Emory-Cloud Sites:	---

Description Chris Samuel (NERSC) 2020-11-16 23:27:58 MST

Hi there,

When setting up idle timeouts for interactive shells I noticed that a way to make a decision based on the jobs QOS was not listed as an option on the web page about prologs and epilogs. I thought that was odd so went for a quick poke around in the source and found that there was a SLURM_JOB_QOS env var set for it.

SLURM_JOB_QOS does work for me in the task prolog, so I think it's just an oversight that it's not listed as an option on the webpage.

Hope this is useful!

All the best,
Chris

Comment 2 Ben Roberts 2020-11-17 12:25:58 MST

Hi Chris,

Thank you for bringing this to our attention.  It looks like there are some other variables that are also missing from the documentation.  I'll work on getting this addressed and let you know as there's progress.

Thanks,
Ben

Comment 3 Chris Samuel (NERSC) 2020-11-17 12:50:00 MST

Hi Ben,

No worries, thanks for this!

All the best,
Chris

Comment 8 Ben Roberts 2021-01-15 08:27:47 MST

Hi Chris,

We've spent some time updating the list of environment variables in the prolog and epilog guide.  This should be reflected on the website when 20.11.3 is released and you can see the commit here:
https://github.com/SchedMD/slurm/commit/bcc1f977c34a5fc099e283794e777b181f5ab01b

I'll go ahead and close this ticket, but let me know if you have any questions.

Thanks,
Ben

Comment 15 Ben Roberts 2021-01-21 08:12:06 MST

Hi Chris,

We reopened this ticket to do some minor corrections to the previous patch.  If you're interested the commits are:
ede15b09837896906b66fa9ebc7cb4d772f3577e
14bac7fcf319d20c07a71634c80c3a9d0c1138c5

Closing this ticket again.

Thanks,
Ben