Ticket 3051 - Prepare and export session directory for the application processes
Summary: Prepare and export session directory for the application processes
Status: RESOLVED FIXED
Alias: None
Product: Slurm
Classification: Unclassified
Component: Contributions (show other tickets)
Version: 16.05.4
Hardware: Linux Linux
: 4 - Minor Issue
Assignee: Tim Wickberg
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2016-09-04 23:40 MDT by Artem Polyakov
Modified: 2016-09-29 19:38 MDT (History)
0 users

See Also:
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 (7.77 KB, application/mbox)
2016-09-04 23:40 MDT, Artem Polyakov 		Details
The patch with updated copyrights (7.77 KB, patch)
2016-09-06 23:46 MDT, Artem Polyakov 		Details | Diff
Show Obsolete (1) 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 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!