Ticket 4118

Summary:	Test 7.11 leaves plugstack.conf with incorrect permissions
Product:	Slurm	Reporter:	David Gloe <david.gloe>
Component:	Test Suite	Assignee:	Tim Wickberg <tim>
Status:	RESOLVED FIXED	QA Contact:
Severity:	4 - Minor Issue
Priority:	---
Version:	17.02.7
Hardware:	Linux
OS:	Linux
Site:	CRAY	Alineos Sites:	---
Atos/Eviden Sites:	---	Confidential Site:	---
Coreweave sites:	---	Cray Sites:	Cray Internal
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:	17.11.13 18.08.4 19.05.0-pre2	Target Release:	---
DevPrio:	---	Emory-Cloud Sites:	---

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.