Bug 1149566 - Review Request: slurm - Simple LinUx Resource Manager
Summary: Review Request: slurm - Simple LinUx Resource Manager
Keywords:
Status: CLOSED DUPLICATE of bug 1489668
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Adam Huffman
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-10-06 05:19 UTC by David Brown
Modified: 2017-09-11 15:29 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-09-11 15:29:28 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description David Brown 2014-10-06 05:19:17 UTC
Spec URL: https://github.com/dmlb2000/slurm-spec/raw/master/slurm.spec
SRPM URL: https://kojipkgs.fedoraproject.org//work/tasks/1774/7771774/slurm-14.03.8-1.fc21.src.rpm
Description:
Slurm is an open source, fault-tolerant, and highly
scalable cluster management and job scheduling system for Linux clusters.
Components include machine status, partition management, job management,
scheduling and accounting modules

Fedora Account System Username: dmlb2000

Comment 1 David Brown 2014-10-06 05:28:51 UTC
Here's the rpmlint output.

slurm.src: W: spelling-error %description -l en_US scalable -> salable, callable, calculable
slurm.src:19: W: mixed-use-of-spaces-and-tabs (spaces: line 19, tab: line 1)
slurm.x86_64: W: spelling-error %description -l en_US scalable -> salable, callable, calculable
slurm.x86_64: W: shared-lib-calls-exit /usr/lib64/libpmi2.so.0.0.0 exit.5
slurm.x86_64: E: missing-call-to-setgroups-before-setuid /usr/bin/salloc
slurm.x86_64: W: shared-lib-calls-exit /usr/lib64/libpmi.so.0.0.0 exit.5
slurm.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/libpmi.so.0.0.0 ['/usr/lib64']
slurm.x86_64: W: devel-file-in-non-devel-package /usr/lib64/slurm/src/srun/srun.wrapper.c
slurm.x86_64: E: incorrect-fsf-address /usr/share/doc/slurm/LICENSE.OpenSSL
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_freezer
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_memory
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_cpuset
slurm.x86_64: W: devel-file-in-non-devel-package /usr/lib64/slurm/src/sattach/sattach.wrapper.c
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/slurm.epilog.clean
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup.release_common.example
slurm-devel.x86_64: W: only-non-binary-in-usr-lib
slurm-lua.x86_64: W: spelling-error %description -l en_US proctrack -> protract
slurm-munge.x86_64: W: spelling-error Summary(en_US) crypto -> crypt, crypts, crypt o
slurm-munge.x86_64: W: spelling-error %description -l en_US crypto -> crypt, crypts, crypt o
slurm-pam_slurm.x86_64: E: binary-or-shlib-defines-rpath /lib64/security/pam_slurm.so ['/usr/lib64']
slurm-sjstat.x86_64: W: spelling-error %description -l en_US availablilty -> availability
slurm-sjstat.x86_64: W: spelling-error %description -l en_US sinfo -> info, s info, sinful
slurm-sjstat.x86_64: W: spelling-error %description -l en_US squeue -> queue, queues, s queue
slurm-sjstat.x86_64: W: spelling-error %description -l en_US scontrol -> control, controls, s control
slurm-sjstat.x86_64: W: spelling-error %description -l en_US utilites -> utilities, utilizes, utility
slurm-slurmdbd.x86_64: W: spelling-error %description -l en_US slurmctld -> slurred
slurm-slurmdb-direct.x86_64: W: non-etc-or-var-file-marked-as-conffile /usr/share/perl5/vendor_perl/config.slurmdb.pl
slurm-slurmdb-direct.x86_64: W: no-manual-page-for-binary moab_2_slurmdb
slurm-torque.x86_64: W: no-manual-page-for-binary qhold
slurm-torque.x86_64: W: no-manual-page-for-binary qstat
slurm-torque.x86_64: W: no-manual-page-for-binary qdel
slurm-torque.x86_64: W: no-manual-page-for-binary pbsnodes
slurm-torque.x86_64: W: no-manual-page-for-binary qalter
slurm-torque.x86_64: W: no-manual-page-for-binary qrerun
slurm-torque.x86_64: W: no-manual-page-for-binary mpiexec
slurm-torque.x86_64: W: no-manual-page-for-binary qsub
slurm-torque.x86_64: W: no-manual-page-for-binary qrls
slurm-torque.x86_64: W: no-manual-page-for-binary generate_pbs_nodefile
16 packages and 0 specfiles checked; 9 errors, 29 warnings.

Many of the errors are because slurm has a lot of script hooks that reside 
out in /etc. These intended to be modified by users for their own custom 
cluster needs.

$ rpm -qlp results_slurm/14.03.8/1.fc20/slurm-14.03.8-1.fc20.x86_64.rpm | grep '\.c$'
/usr/lib64/slurm/src/sattach/sattach.wrapper.c
/usr/lib64/slurm/src/srun/srun.wrapper.c

These are used for parallel debuggers like TotalView (shrug). Its the way
upstream packages the source.

Comment 2 David Brown 2014-10-06 06:16:15 UTC
I fixed some minor spelling and tab issues.

16 packages and 0 specfiles checked; 9 errors, 23 warnings.

Comment 3 Haïkel Guémar 2014-10-15 21:28:19 UTC
Few things:
* why is libmad-devel a build requirements for slurm ? I don't see the point in having a MP3 decoder to build a batch scheduler, and we don't ship libmad (rpmfusion does) so it's a NO-NO in Fedora. Looks like a typo to me.
Besides, it's inside a conditional that is always false "%if 0"
* All supported Fedora releases (oldest being 19) and EL7 have systemd and supports systemd macroized scriptlets.
%if 0%{?rhel} && 0%{?rhel} == 6
# sysV stuff
%else
# systemd stuff
%endif
* systemd BR & R are not correct, they should be:
Requires(post): systemd
Requires(preun): systemd
Requires(postun): systemd
BuildRequires: systemd
* subpackages should requires fully versionned base package or other "base" subpackages
https://fedoraproject.org/wiki/Packaging:Guidelines#Requiring_Base_Package

Comment 4 David Brown 2014-10-15 22:17:59 UTC
The first bit is a misspelling, it should be libibumad-devel and libibmad-devel. Those aren't supported in Fedora, not sure why, but they are supported in RHEL. I was going to look into why that's the case since I think ibverbs is supported in Fedora ... this is the whole funky infiniband network we like to run :)

I've fixed the other two issues, it cleans up things quite a bit, I'll have to apply those changes to my other packages...

The el6 version of the SYSV init script has a lot of issues. I'd like someone to look at them and figure out which are important to clean up. I can submit that back to slurm-devel and have them take a look. I'll post those later tonight.

Thanks,
- David Brown

Comment 5 David Brown 2014-10-16 05:15:35 UTC
Here's the epel-6 build of the same spec.

http://koji.fedoraproject.org/koji/taskinfo?taskID=7879588

slurm-slurmdbd.x86_64: E: no-status-entry /etc/rc.d/init.d/slurmdbd
slurm-slurmdbd.x86_64: W: no-reload-entry /etc/rc.d/init.d/slurmdbd
slurm-slurmdbd.x86_64: E: no-chkconfig-line /etc/rc.d/init.d/slurmdbd
slurm-slurmdbd.x86_64: E: subsys-not-used /etc/rc.d/init.d/slurmdbd
slurm-slurmdbd.x86_64: W: incoherent-init-script-name slurmdbd ('slurm-slurmdbd', 'slurm-slurmdbdd')

slurm.x86_64: E: no-status-entry /etc/rc.d/init.d/slurm
slurm.x86_64: W: no-reload-entry /etc/rc.d/init.d/slurm
slurm.x86_64: E: no-chkconfig-line /etc/rc.d/init.d/slurm
slurm.x86_64: E: subsys-not-used /etc/rc.d/init.d/slurm

Thanks,
- David Brown

Comment 6 David Brown 2014-10-16 15:40:04 UTC
Yup further investigation seems like the rpmlint stuff didn't find those lines but they are there in the init scripts... I'll have to look at the guidelines for what a SYSV init script is supposed to look like.

The ibmad and ibumad stuff is enabled in the build and the %if 0 thing was removed.

Comment 7 David Brown 2014-10-30 05:43:50 UTC
I assume package has been approved by the other maintainer?

Comment 8 David Brown 2014-10-30 05:54:47 UTC
New Package SCM Request
=======================
Package Name: slurm
Short Description: Simple LinUx Resource Manager
Upstream URL: http://www.schedmd.com
Owners: dmlb2000
Branches: f21 f20 el6 epel7

Comment 9 Thomas Spura 2014-10-30 08:43:35 UTC
(In reply to David Brown from comment #7)
> I assume package has been approved by the other maintainer?

Hmm... no :)

Haïkel didn't assign it yet to him or wrote that he approved the package...

Comment 10 David Brown 2014-10-30 15:25:01 UTC
I figured as much but its been a couple of weeks since I last poked at it.

Comment 11 David Brown 2014-10-30 16:15:42 UTC
Here's the updated upstream version rpmlint output.

slurm.src: W: spelling-error %description -l en_US scalable -> salable, callable, calculable
slurm.src: W: strange-permission slurm-14.03.9.tar.bz2 0640L
slurm.x86_64: W: spelling-error %description -l en_US scalable -> salable, callable, calculable
slurm.x86_64: W: shared-lib-calls-exit /usr/lib64/libpmi2.so.0.0.0 exit.5
slurm.x86_64: W: shared-lib-calls-exit /usr/lib64/libpmi.so.0.0.0 exit.5
slurm.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/libpmi.so.0.0.0 ['/usr/lib64']
slurm.x86_64: E: missing-call-to-setgroups-before-setuid /usr/bin/salloc
slurm.x86_64: W: devel-file-in-non-devel-package /usr/lib64/slurm/src/srun/srun.wrapper.c
slurm.x86_64: E: incorrect-fsf-address /usr/share/doc/slurm/LICENSE.OpenSSL
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_freezer
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_memory
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_cpuset
slurm.x86_64: W: devel-file-in-non-devel-package /usr/lib64/slurm/src/sattach/sattach.wrapper.c
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/slurm.epilog.clean
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup.release_common.example
slurm-devel.x86_64: W: only-non-binary-in-usr-lib
slurm-lua.x86_64: W: spelling-error %description -l en_US proctrack -> protract
slurm-pam_slurm.x86_64: E: binary-or-shlib-defines-rpath /lib64/security/pam_slurm.so ['/usr/lib64']
slurm-sjstat.x86_64: W: spelling-error %description -l en_US sinfo -> info, s info, sinful
slurm-sjstat.x86_64: W: spelling-error %description -l en_US squeue -> queue, queues, s queue
slurm-sjstat.x86_64: W: spelling-error %description -l en_US scontrol -> control, controls, s control
slurm-slurmdbd.x86_64: W: spelling-error %description -l en_US slurmctld -> slurred
slurm-slurmdb-direct.x86_64: W: no-manual-page-for-binary moab_2_slurmdb
slurm-torque.x86_64: W: no-manual-page-for-binary qhold
slurm-torque.x86_64: W: no-manual-page-for-binary qstat
slurm-torque.x86_64: W: no-manual-page-for-binary qdel
slurm-torque.x86_64: W: no-manual-page-for-binary pbsnodes
slurm-torque.x86_64: W: no-manual-page-for-binary qalter
slurm-torque.x86_64: W: no-manual-page-for-binary qrerun
slurm-torque.x86_64: W: no-manual-page-for-binary mpiexec
slurm-torque.x86_64: W: no-manual-page-for-binary qsub
slurm-torque.x86_64: W: no-manual-page-for-binary qrls
slurm-torque.x86_64: W: no-manual-page-for-binary generate_pbs_nodefile
16 packages and 0 specfiles checked; 9 errors, 24 warnings.

Same location for the spec.
https://github.com/dmlb2000/slurm-spec/raw/master/slurm.spec
http://koji.fedoraproject.org/koji/taskinfo?taskID=7982819

Comment 12 David Brown 2014-11-20 03:50:43 UTC
This is for an updated rawhide build for 14.11.0

slurm.src: W: spelling-error %description -l en_US scalable -> salable, callable, calculable
slurm.src: W: strange-permission slurm-14.11.0.tar.bz2 0640L
slurm.x86_64: W: spelling-error %description -l en_US scalable -> salable, callable, calculable
slurm.x86_64: W: invalid-url URL: http://www.schedmd.com timed out
slurm.x86_64: W: shared-lib-calls-exit /usr/lib64/libpmi2.so.0.0.0 exit.5
slurm.x86_64: W: shared-lib-calls-exit /usr/lib64/libpmi.so.0.0.0 exit.5
slurm.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/libpmi.so.0.0.0 ['/usr/lib64']
slurm.x86_64: E: missing-call-to-setgroups-before-setuid /usr/bin/salloc
slurm.x86_64: W: devel-file-in-non-devel-package /usr/lib64/slurm/src/srun/srun.wrapper.c
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_freezer
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_memory
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_cpuset
slurm.x86_64: W: devel-file-in-non-devel-package /usr/lib64/slurm/src/sattach/sattach.wrapper.c
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/slurm.epilog.clean
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup.release_common.example
slurm-devel.x86_64: W: only-non-binary-in-usr-lib
slurm-lua.x86_64: W: spelling-error %description -l en_US proctrack -> protract
slurm-pam_slurm.x86_64: E: binary-or-shlib-defines-rpath /lib64/security/pam_slurm.so ['/usr/lib64']
slurm-sjstat.x86_64: W: spelling-error %description -l en_US sinfo -> info, s info, sinful
slurm-sjstat.x86_64: W: spelling-error %description -l en_US squeue -> queue, queues, s queue
slurm-sjstat.x86_64: W: spelling-error %description -l en_US scontrol -> control, controls, s control
slurm-slurmdbd.x86_64: W: spelling-error %description -l en_US slurmctld -> slurred
slurm-slurmdb-direct.x86_64: W: no-manual-page-for-binary moab_2_slurmdb
slurm-torque.x86_64: W: no-manual-page-for-binary qhold
slurm-torque.x86_64: W: no-manual-page-for-binary qstat
slurm-torque.x86_64: W: no-manual-page-for-binary qdel
slurm-torque.x86_64: W: no-manual-page-for-binary pbsnodes
slurm-torque.x86_64: W: no-manual-page-for-binary qalter
slurm-torque.x86_64: W: no-manual-page-for-binary qrerun
slurm-torque.x86_64: W: no-manual-page-for-binary mpiexec
slurm-torque.x86_64: W: no-manual-page-for-binary qsub
slurm-torque.x86_64: W: no-manual-page-for-binary qrls
slurm-torque.x86_64: W: no-manual-page-for-binary generate_pbs_nodefile
16 packages and 0 specfiles checked; 8 errors, 25 warnings.

Same location for the spec.
https://github.com/dmlb2000/slurm-spec/raw/master/slurm.spec
http://koji.fedoraproject.org/koji/taskinfo?taskID=8190078

Comment 13 David Brown 2015-04-28 16:37:07 UTC
This is for an updated rawhide build for 14.11.6

slurm.src: W: spelling-error %description -l en_US scalable -> salable, callable, calculable
slurm.src: W: strange-permission slurm-14.11.6.tar.bz2 0640L
slurm.x86_64: W: spelling-error %description -l en_US scalable -> salable, callable, calculable
slurm.x86_64: W: shared-lib-calls-exit /usr/lib64/libpmi2.so.0.0.0 exit.5
slurm.x86_64: W: shared-lib-calls-exit /usr/lib64/libpmi.so.0.0.0 exit.5
slurm.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/libpmi.so.0.0.0 ['/usr/lib64']
slurm.x86_64: E: missing-call-to-setgroups-before-setuid /usr/bin/salloc
slurm.x86_64: W: devel-file-in-non-devel-package /usr/lib64/slurm/src/srun/srun.wrapper.c
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_freezer
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_memory
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_cpuset
slurm.x86_64: W: devel-file-in-non-devel-package /usr/lib64/slurm/src/sattach/sattach.wrapper.c
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/slurm.epilog.clean
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup.release_common.example
slurm-devel.x86_64: W: only-non-binary-in-usr-lib
slurm-lua.x86_64: W: spelling-error %description -l en_US proctrack -> protract
slurm-pam_slurm.x86_64: E: binary-or-shlib-defines-rpath /lib64/security/pam_slurm.so ['/usr/lib64']
slurm-sjstat.x86_64: W: spelling-error %description -l en_US sinfo -> info, s info, sinful
slurm-sjstat.x86_64: W: spelling-error %description -l en_US squeue -> queue, queues, s queue
slurm-sjstat.x86_64: W: spelling-error %description -l en_US scontrol -> control, controls, s control
slurm-slurmdbd.x86_64: W: spelling-error %description -l en_US slurmctld -> slurred
slurm-slurmdb-direct.x86_64: W: no-manual-page-for-binary moab_2_slurmdb
slurm-torque.x86_64: W: no-manual-page-for-binary qhold
slurm-torque.x86_64: W: no-manual-page-for-binary qstat
slurm-torque.x86_64: W: no-manual-page-for-binary qdel
slurm-torque.x86_64: W: no-manual-page-for-binary pbsnodes
slurm-torque.x86_64: W: no-manual-page-for-binary qalter
slurm-torque.x86_64: W: no-manual-page-for-binary qrerun
slurm-torque.x86_64: W: no-manual-page-for-binary mpiexec
slurm-torque.x86_64: W: no-manual-page-for-binary qsub
slurm-torque.x86_64: W: no-manual-page-for-binary qrls
slurm-torque.x86_64: W: no-manual-page-for-binary generate_pbs_nodefile
16 packages and 0 specfiles checked; 8 errors, 24 warnings.

https://github.com/dmlb2000/slurm-spec/raw/master/slurm.spec
http://koji.fedoraproject.org/koji/taskinfo?taskID=9587874

Comment 14 Christopher Brown 2015-06-19 10:48:10 UTC
Hi David,

We are interested in getting the packaged for epel.

I take it this is just awaiting review?

Anything I can do to assist?

Regards

Comment 15 David Brown 2015-06-19 15:13:46 UTC
Yes this package is still waiting for review. I'd love someone to review it and pass off on it.

I can get an update to the current version though there isn't very many changes to the spec or resulting RPMs.

Thanks,
- David Brown

Comment 16 Upstream Release Monitoring 2015-12-06 18:26:06 UTC
pbrobinson's scratch build of linux-user-chroot?#b7afe5173cbd31b029b027b6f8a14baa5e6ce87a for epel7-archbootstrap and git://pkgs.fedoraproject.org/linux-user-chroot?#b7afe5173cbd31b029b027b6f8a14baa5e6ce87a failed http://koji.fedoraproject.org/koji/taskinfo?taskID=12089939

Comment 17 David Brown 2015-12-07 00:43:38 UTC
Interesting scratch build, seems to have gotten on the wrong bug...

Thanks,
- David Brown

Comment 18 Upstream Release Monitoring 2015-12-07 02:11:47 UTC
dmlb2000's scratch build of slurm-15.08.4-1.fc24.src.rpm for f24 completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12095847

Comment 19 Adam Huffman 2016-04-10 09:03:15 UTC
Hi David,

If you can update this to 15.08.10, I'll try to find time to review it.

Recently I've been building it for my own use on Copr, using minimal changes to the upstream spec, but yours is much more Fedora-like.

Cheers,
Adam

Comment 20 David Brown 2016-04-12 13:26:28 UTC
Adam,

Here's the 15.08.10 scratch build of slurm.
http://koji.fedoraproject.org/koji/taskinfo?taskID=13635064
https://github.com/dmlb2000/slurm-spec/raw/master/slurm.spec

There's a few warnings that pop up for me that could be resolved... 

# this one seems new...
slurm-plugins.x86_64: W: manual-page-warning /usr/share/man/man5/burst_buffer.conf.5.gz 116: name expected (got `\ '): treated as missing

# this is for debuggers says in the file itself, not sure how to flag it for the debuginfo package...
slurm.x86_64: W: devel-file-in-non-devel-package /usr/lib64/slurm/src/srun/srun.wrapper.c

Thanks,
- David Brown

Comment 21 Adam Huffman 2016-04-12 13:58:07 UTC
Hi David,

Thanks for the update.

First observation is about the build part.

I and someone on the mailing list found that the following in %build fixed the FORTIFY warnings during compilation:

#Fix for build and linking failure
CFLAGS="$RPM_OPT_FLAGS -Wl,-z,lazy"
CXXFLAGS="$RPM_OPT_FLAGS -Wl,-z,lazy"

export CFLAGS
export CXXFLAGS

I see you've taken a different approach, which is fine, but your version still has the build warnings, which I'd prefer to be fixed before inclusion in Fedora.

Cheers,
Adam

Comment 22 Adam Huffman 2016-04-12 14:06:20 UTC
Hi David,

It's also worth mentioning that there are systemd unit files in the upstream source now. While you might want to patch them, you shouldn't need to include your own versions anymore.

Cheers,
Adam

Comment 23 David Brown 2016-04-12 15:40:30 UTC
Adam,

The issue I was trying to fix was when rpmlint errors about the following...

slurm.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/libpmi.so.0.0.0 ['/usr/lib64']
slurm-pam_slurm.x86_64: E: binary-or-shlib-defines-rpath /lib64/security/pam_slurm_adopt.so ['/usr/lib64']
slurm-pam_slurm.x86_64: E: binary-or-shlib-defines-rpath /lib64/security/pam_slurm.so ['/usr/lib64']
slurm-perlapi.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/perl5/vendor_perl/auto/Slurm/Slurm.so ['../../../../src/api/.libs', '/usr/lib']
slurm-perlapi.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/perl5/vendor_perl/auto/Slurmdb/Slurmdb.so ['../../../../src/db_api/.libs', '/usr/lib']

Now some of the rpaths in the libraries are redundant and some are plain dangerous... Not sure if you've ever looked at this but the perl bits give me the creeps with the relative rpath bits...

I've included the upstream systemd files with the new build as well.

Thanks,
- David Brown

Comment 24 David Brown 2016-04-12 15:48:40 UTC
I should mention that those errors are from me commenting out all my sed rpath commands in the spec posted.

Comment 25 Adam Huffman 2016-04-12 15:55:04 UTC
Hi David,

There's certainly no lack of things to clean up in the upstream build process...
With my Copr builds I did the minimum to make the upstream spec file work.

Perhaps you could add the FORTIFY_SOURCE fix on top of the rpath fix? 

Up to you how you wish to do it, but I'd certainly like to see those warnings fixed.

Cheers,
Adam

Comment 26 Adam Huffman 2016-04-12 16:03:28 UTC
I see you've already done that - thanks.

Comment 27 David Brown 2016-04-12 16:15:37 UTC
Adam,

Okay here's a smaller rpmlint output.

$ rpmlint results/15.08.10/1.el7/slurm-*.rpm 
slurm.x86_64: W: incoherent-version-in-changelog 15.08.10-1 ['15.08.10-1.el7.centos', '15.08.10-1.centos']
slurm.x86_64: W: shared-lib-calls-exit /usr/lib64/libpmi2.so.0.0.0 exit.5
slurm.x86_64: E: missing-call-to-setgroups /usr/bin/salloc
slurm.x86_64: W: shared-lib-calls-exit /usr/lib64/libpmi.so.0.0.0 exit.5
slurm.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/libpmi.so.0.0.0 ['/usr/lib64']
slurm.x86_64: W: devel-file-in-non-devel-package /usr/lib64/slurm/src/srun/srun.wrapper.c
slurm.x86_64: W: devel-file-in-non-devel-package /usr/lib64/slurm/src/sattach/sattach.wrapper.c
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_freezer
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_memory
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup/release_cpuset
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/slurm.epilog.clean
slurm.x86_64: E: executable-marked-as-config-file /etc/slurm/cgroup.release_common.example
slurm-pam_slurm.x86_64: E: binary-or-shlib-defines-rpath /lib64/security/pam_slurm_adopt.so ['/usr/lib64']
slurm-pam_slurm.x86_64: E: binary-or-shlib-defines-rpath /lib64/security/pam_slurm.so ['/usr/lib64']
slurm-slurmdbd.x86_64: W: only-non-binary-in-usr-lib
slurm-slurmdb-direct.x86_64: W: no-manual-page-for-binary moab_2_slurmdb
slurm-torque.x86_64: W: no-manual-page-for-binary qhold
slurm-torque.x86_64: W: no-manual-page-for-binary qstat
slurm-torque.x86_64: W: no-manual-page-for-binary qdel
slurm-torque.x86_64: W: no-manual-page-for-binary pbsnodes
slurm-torque.x86_64: W: no-manual-page-for-binary qalter
slurm-torque.x86_64: W: no-manual-page-for-binary qrerun
slurm-torque.x86_64: W: no-manual-page-for-binary mpiexec
slurm-torque.x86_64: W: no-manual-page-for-binary qsub
slurm-torque.x86_64: W: no-manual-page-for-binary qrls
slurm-torque.x86_64: W: no-manual-page-for-binary generate_pbs_nodefile
16 packages and 0 specfiles checked; 9 errors, 17 warnings.

Comment 28 David Brown 2016-04-12 16:22:08 UTC
Adam,

Some of these we can fix as package managers others (a lot of the errors) the slurm guys need to fix.

slurm.x86_64: E: missing-call-to-setgroups /usr/bin/salloc
Potential security issue as its calling something that is setuid and its not dropping all the users groups before it does so.

executable-marked-as-config-file
slurm has a lot of these, all the post and pre job scripts fall into this category. I'd like to see them handle this better by doing more of a global slurm job post script that loops over scripts in a space defined in /etc. That'd be a lot nicer from a system administration perspective.

Could make stub man pages for the torque commands... haven't done so yet.

Thanks,
- David Brown

Comment 29 Adam Huffman 2016-04-19 06:09:44 UTC
Hi David,

It's certainly acceptable to include patches in the RPM to fix these errors, while also submitting them upstream.

Here are a couple of things highlighted by fedora-review:
- If (and only if) the source package includes the text of the license(s)
  in its own file, then that file, containing the text of the license(s)
  for the package is included in %license.
  Note: License file COPYING is marked as %doc instead of %license
  See:
  http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text
- Large documentation must go in a -doc subpackage. Large could be size
  (~1MB) or number of files.
  Note: Documentation size is 9400320 bytes in 234 files.
  See:
  http://fedoraproject.org/wiki/Packaging/Guidelines#PackageDocumentation


Cheers,
Adam

Comment 30 Adam Huffman 2016-05-16 06:16:51 UTC
Hi David,

Could you update to 15.08.11?

Thanks,
Adam

Comment 31 Andrew Elwell 2016-10-17 08:05:01 UTC
Hi, is there any movement on this? If not I'll start a new package based on the current upsream release of 16.05.5 (http://slurm.schedmd.com/download.html)

Andrew

Comment 32 Adam Huffman 2016-11-12 08:08:50 UTC
Hi Andrew,

I'm happy to review your 16.05.x package if you submit it. 

Haven't heard from David in several months, so this does appear to be stalled at the moment.

Adam

Comment 33 Philip Kovacs 2017-09-03 17:09:44 UTC
Over the past month or so I've taken the time to completely rewrite the slurm spec, from scratch, in order to try and conform to Fedora packaging standards. I've been working with upstream on a few bugs and have included patches in my work to align with those upstream and to address build or Fedora-specific issues.  

I've got the mockbuilds working and all rpmlint errors are cleared, save a few minor warnings which I documented.  I added many little things for completeness, such as the creation of the slurm account, the state file directories (/var), the tmpfiles.d conf file, a logrotate.d conf file, etc.

I use slurm with openmpi and have also written an openmpi spec which builds against my new slurm-devel/slurm-libs packages, but that is a separate issue.

If anyone can sponsor me, I would be happy to maintain the new packages.

I'll wait for a message back, then provide the download info.

Philip

Comment 34 Philip Kovacs 2017-09-11 15:03:36 UTC
Please close this bug duplicate of bug #1489668.   This one is long stalled, the other is active.

Comment 35 Christopher Brown 2017-09-11 15:29:28 UTC

*** This bug has been marked as a duplicate of bug 1489668 ***


Note You need to log in before you can comment on or make changes to this bug.