Bug 4038

Summary: job_submit.lua: output of slurm.log_user() should be returned to user even on "return slurm.SUCCESS".
Product: Slurm Reporter: Paolo Palazzi <paolo.palazzi>
Component: User CommandsAssignee: Alejandro Sanchez <alex>
Status: RESOLVED FIXED QA Contact:
Severity: 5 - Enhancement    
Priority: --- CC: ahkumar, alex, brian.gilmer, brian, tim
Version: 17.02.1   
Hardware: Linux   
OS: Linux   
See Also: https://bugs.schedmd.com/show_bug.cgi?id=14539
Site: CRAY Alineos Sites: ---
Atos/Eviden Sites: --- Confidential Site: ---
Coreweave sites: --- Cray Sites: AWI
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: 17.11.0pre3 Target Release: 17.11
DevPrio: 3 - High Emory-Cloud Sites: ---

Description Paolo Palazzi 2017-07-31 01:54:50 MDT
job_submit.lua command 'slurm.log_user("Some Text")' writes output to the user's session only when the script ends with 'return slurm.ERROR'.

A way to output to the user's session even when job_submit.lua ends with 'return slurm.SUCCESS' would be welcome.
Comment 1 Paolo Palazzi 2017-07-31 01:56:30 MDT
Note: the same request is found in https://groups.google.com/forum/#!topic/slurm-devel/Rd_k6qGNd_w
Comment 2 Brian F Gilmer 2017-07-31 01:57:11 MDT
I am out of office until August 7
Comment 3 Paolo Palazzi 2017-07-31 01:57:26 MDT
This is an automatic reply.

I am on vacation, back at work on 31st July.

I will have no email access from 10th to 25th July.

--
                     Paolo
Comment 6 Paolo Palazzi 2017-08-15 13:06:32 MDT
This is an automatic reply.

15th August is a public holiday in Italy,
16th August is a contract holiday.

I will have only sporadic access to my e-mail until the 17th.

--
                     Paolo
Comment 7 Paolo Palazzi 2017-09-26 07:32:59 MDT
The Customer has requested an update.

Is this enhancement still on track to appear in release 17.11?

Thanks.
Comment 8 Alejandro Sanchez 2017-09-26 07:34:19 MDT
This is on track. We'll try to get this in for 17.11.
Comment 24 Alejandro Sanchez 2017-10-11 03:17:04 MDT
Hi,

this functionality has been added through the following pile of commits, which are included since Slurm 17.11.0pre3:

60c9f30644
e22835f139
2de45efb66
84a79b011b
c917bf06e1
ceb449b01d
747fcb74f5
6117a30778
d3354d0d56
6767567f90
4635533712
be481bdd30
473dc22b9d
83b0dea63a
60468f3eac
64fe6e06db

Closing this bug. Please, reopen if you encounter further issues.

Thanks