Ticket 4500 - Switch to standard pidfiles on Cray systems
Summary: Switch to standard pidfiles on Cray systems
Status: RESOLVED FIXED
Alias: None
Product: Slurm
Classification: Unclassified
Component: Contributions (show other tickets)
Version: 17.11.0
Hardware: Linux Linux
: 4 - Minor Issue
Assignee: Danny Auble
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2017-12-11 10:43 MST by David Gloe
Modified: 2017-12-11 13:32 MST (History)
0 users

See Also:
Site: CRAY
Slinky Site: ---
Alineos Sites: ---
Atos/Eviden Sites: ---
Confidential Site: ---
Coreweave sites: ---
Cray Sites: Cray Internal
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: 17.11.1 18.08.0-0pre1
Target Release: ---
DevPrio: ---
Emory-Cloud Sites: ---

Attachments
Patch to switch to standard pid files on Cray systems (3.80 KB, patch)
2017-12-11 10:43 MST, David Gloe 		Details | Diff
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 David Gloe 2017-12-11 10:43:39 MST 
Created attachment 5714 [details]
Patch to switch to standard pid files on Cray systems

The pid files in slurm.conf and the systemd service files must match, or systemd will time out looking for the wrong pid file. Currently, the Cray slurm.conf template has different pid files for slurmctld and slurmd than the service files.

There's no reason for us to use these nonstandard pid files, and it will save us some headaches to switch over.

I've attached a patch which changes the slurm.conf template to put the pid files in /var/run, matching the systemd service files. I've also removed some init.d handling that's no longer needed.
Comment 1 Danny Auble 2017-12-11 13:32:04 MST 
Thanks David, this has been added in commit 5342d9791fc15e312.