Ticket 524 - AuthInfo
Summary: AuthInfo
Status: RESOLVED FIXED
Alias: None
Product: Slurm
Classification: Unclassified
Component: Other (show other tickets)
Version: 14.03.x
Hardware: All Linux
: 4 - Minor Issue
Assignee: Moe Jette
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2013-11-15 10:32 MST by Don Lipari
Modified: 2013-11-15 10:57 MST (History)
1 user (show)

See Also:
Site: LLNL
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
AuthInfo: Adds the ability to specify an alternate MUNGE socket. (8.81 KB, application/octet-stream)
2013-11-15 10:32 MST, Don Lipari
Details

Note You need to log in before you can comment on or make changes to this ticket.
Description Don Lipari 2013-11-15 10:32:57 MST
Created attachment 514 [details]
AuthInfo:  Adds the ability to specify an alternate MUNGE socket.

It turns out, the sys admins really need on BG/Q the MungeSocket functionality that Phil added to our v2.3.3 here:
https://github.com/chaos/slurm/commit/7d31af59f89b689408d12d607c6e5c4e31a8f2d8

The attached patch is a port of the above to the latest 13.12 development branch.  It follows Danny's request to use AuthInfo instead of MungeSocket for consistency sake.

I tested the patch and it works on a simulated machine.  I have not had the chance to port this to v2.6 and run it on a real machine.

The attached patch includes all of Phil's original patch (substituting AuthInfo) plus some additional changes I though would be required.  I used the existing code supporting authtype to locate possible additions for authinfo.

There were some files I opted not to change.  For example, I did not create an auth_info analog to slurm_get_auth_type() and slurm_set_auth_type().  I also judged that the auth_type code in src/slurmctld/read_config.c did not have to be expanded to include auth_info.

If you feel these sections need to include auth_info, let me know and I'll make the patch match up with auth_type exactly.

Again, it seemed to work on my test system.  And Phil's original MungeSocket patch has been working fine on our v2.3.3 systems.
Comment 1 Moe Jette 2013-11-15 10:57:23 MST
Slight variation of Phil's patch committed here: 

https://github.com/SchedMD/slurm/commit/581bd1d8cba665daade738c1fc10746f3285c785