Ticket 3051

Summary: Prepare and export session directory for the application processes
Product: Slurm Reporter: Artem Polyakov <artpol84>
Component: ContributionsAssignee: Tim Wickberg <tim>
Status: RESOLVED FIXED QA Contact:
Severity: 4 - Minor Issue    
Priority: ---    
Version: 16.05.4   
Hardware: Linux   
OS: Linux   
Site: -Other- Slinky Site: ---
Alineos Sites: --- Atos/Eviden Sites: ---
Confidential Site: --- Coreweave sites: ---
Cray Sites: --- DS9 clusters: ---
Google sites: --- HPCnow Sites: ---
HPE Sites: --- IBM Sites: ---
NOAA SIte: --- NoveTech Sites: ---
Nvidia HWinf-CS Sites: --- OCF Sites: ---
Recursion Pharma Sites: --- SFW Sites: ---
SNIC sites: --- Tzag Elita Sites: ---
Linux Distro: --- Machine Name:
CLE Version: Version Fixed: 16.05.5 17.02-pre2
Target Release: --- DevPrio: ---
Emory-Cloud Sites: ---
Attachments: The patch
The patch with updated copyrights

Description Artem Polyakov 2016-09-04 23:40:36 MDT 
Created attachment 3461 [details]
The patch

This patch forces slurmstepd to create the session directory for the applications and provide the path to this directory for the app procs.

According to experimental measurements this may reduce the local jobstart time by 250-350 ms.
Comment 1 Artem Polyakov 2016-09-04 23:42:08 MDT 
P.S.
* please apply with "git am" to preserve the commit message.
* This patch is related (but not dependent) to the OMPI one: https://github.com/open-mpi/ompi/pull/2042.
Comment 2 Artem Polyakov 2016-09-06 23:46:31 MDT 
Created attachment 3469 [details]
The patch with updated copyrights

I fogrot to update copyrights, this patch is fixed.
Comment 3 Artem Polyakov 2016-09-19 10:41:19 MDT 
P.S.
I think it's a good time to add Mellanox to the list of contributing companies:
http://slurm.schedmd.com/team.html
Comment 4 Artem Polyakov 2016-09-29 08:17:51 MDT 
Any updates on this
Comment 5 Tim Wickberg 2016-09-29 10:28:13 MDT 
Committed to 16.05 as 901aeeb3. (Mellanox added to contributor list with 71a39bf4b.)

I made a few minor changes within the patch. Please watch for the 80-column limit. While we allow some occasional lines to pass beyond 80, 100 columns wide is too much especially in comment blocks.

I also added __func__ annotations to the error() calls to clarify where the message is being generated, as I didn't feel that the "chown:" message was specific enough to trace back to those locations.
Comment 6 Artem Polyakov 2016-09-29 19:38:14 MDT 
Thank you, Tim!