Ticket 14299 - add slurm_conf option to pam_slurm_adopt for non default slurm.conf location
Summary: add slurm_conf option to pam_slurm_adopt for non default slurm.conf location
Status: OPEN
Alias: None
Product: Slurm
Classification: Unclassified
Component: Other (show other tickets)
Version: 23.02.x
Hardware: All Linux
: C - Contributions
Assignee: Tim Wickberg
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2022-06-12 08:16 MDT by Yair Yarom
Modified: 2022-09-06 12:46 MDT (History)
2 users (show)

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:
Target Release: ---
DevPrio: ---
Emory-Cloud Sites: ---

Attachments
pam_slurm_adopt: slurm_conf parameter (2.26 KB, patch)
2022-06-12 08:16 MDT, Yair Yarom 		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 Yair Yarom 2022-06-12 08:16:00 MDT 
Created attachment 25471 [details]
pam_slurm_adopt: slurm_conf parameter

Hi,

Is it possible to add a slurm_conf option to pam_slurm_adopt? E.g. when slurm.conf is in a non default location.

Attached is a patch that achieves this. The patch is against master (23.02), but I've only tested it on 21.08 (which is what we currently have).

Thanks,
    Yair.
Comment 1 Ruixing Wang 2022-08-23 23:25:07 MDT 
I am also interested in this patch.

I am currently using pam_slurm_adopt with compute node slurmd running in configless mode. With configless mode, I expect slurm.conf is not needed in compute nodes. However, pam_slurm_adopt still wants to read slurm.conf at the default location, which doesn't exist, instead of the one that slurmd pulled from slurmctld (saved at /var/spool/slurm/d/conf-cache/slurm.conf). This patch could be useful also in this case.

hope this will get noticed and merged in future releases. 

Thanks.