Ticket 14299

Summary: add slurm_conf option to pam_slurm_adopt for non default slurm.conf location
Product: Slurm Reporter: Yair Yarom <irush>
Component: OtherAssignee: Tim Wickberg <tim>
Status: OPEN --- QA Contact:
Severity: C - Contributions    
Priority: --- CC: lyeager, ruixingw
Version: 23.02.x   
Hardware: All   
OS: Linux   
Site: -Other- Alineos Sites: ---
Atos/Eviden Sites: --- Confidential Site: ---
Coreweave sites: --- Cray Sites: ---
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: Target Release: ---
DevPrio: --- Emory-Cloud Sites: ---
Attachments: pam_slurm_adopt: slurm_conf parameter

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.