Ticket 4118 - Test 7.11 leaves plugstack.conf with incorrect permissions
Summary: Test 7.11 leaves plugstack.conf with incorrect permissions
Status: RESOLVED FIXED
Alias: None
Product: Slurm
Classification: Unclassified
Component: Test Suite (show other tickets)
Version: 17.02.7
Hardware: Linux Linux
: 4 - Minor Issue
Assignee: Tim Wickberg
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2017-08-29 15:20 MDT by David Gloe
Modified: 2018-10-25 14:44 MDT (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.13 18.08.4 19.05.0-pre2
Target Release: ---
DevPrio: ---
Emory-Cloud Sites: ---

Attachments
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-08-29 15:20:06 MDT 
Test 7.11 runs a chmod 700 on the plugstack.conf file. After the test completes, the permissions cause sbatch to fail with:

sbatch: error: spank: Failed to open /etc/opt/slurm/plugstack.conf: Permission denied
sbatch: error: Failed to initialize plugin stack
Comment 1 Tim Wickberg 2017-08-29 16:28:31 MDT 
Fixed with commit 3547869bb.
Comment 2 David Gloe 2018-10-25 13:15:04 MDT 
It appears that this fix has been lost in Slurm 17.11/18.08. I just ran the regression suite on 18.08 and the plugstack.conf file was left with 0700 permissions.
Comment 3 Tim Wickberg 2018-10-25 14:44:57 MDT 
Sorry about that, fixed it again. It was inadvertently discarded on a merge commit before 17.11 was released.

commit abf5bcfd782293cb417f989629c306da5d0db52d
Author: Tim Wickberg <tim@schedmd.com>
Date:   Thu Oct 25 14:30:33 2018 -0600

    Revert "Add two chmod calls to test7.11 to make sure it runs when "
    
    This reverts commit 0581585c70933452a7efcab6423fdd45571169b8.
    
    Do not change permissions on files the testsuite does not "own".
    
    This is a duplicate of the previous 3547869bb76a commit, which
    was previously discarded by merge commit cf98dfc019f7b.
    
    Bug 4118.