Coverity CID 179200

Coverity CID 179201

bug 4374

Do so even if pack-group 0 is completed, so long as not all components are completed
bug 4374

Specify "BackupController#=<hostname> and "BackupAddr#=<address>" to
  identify up to 9 backup servers in slurm.conf.
Output format of "scontrol ping" and the daemon status at the end of
  "scontrol status" is modified to report up status of the primary and all
  backup servers.
"scontrol takeover [#]" command can now identify the BackupController
  index number. Default value is "1" (the configured "BackupController" or
  "BackupController1" node).

In a prior incarnation of the patch that introduced it,
it was MaxQueryTimeLimit, and that was not updated with
the code base when changed.

Bug 4365.

Update slurm.spec and slurm.spec-legacy as well

Bug 4323.

This now matches the sinfo documentation.

Bug 4306.

The race condition this is avoiding has been fixed elsewhere.

This reverts commit 6c21c8bd.

Coverity 178912

Coverity CID 178913

Removed references to versions <=16.05 and adapted to new 17.11

launch/slurm plugin - Avoid using global variable for heterogeneous job
    steps, which could corrupt memory.
bug 4333

Ancient versions of OpenMPI and their derivatives (i.e. Cray MPI) are
dependent upon communication ports being assigned to them by Slurm. Such MPI
jobs will experience step launch failure if any component of a
heterogeneous job step is unable to acquire the allocated ports.
Non-heterogeneous job steps will retry step launch using a new set of
communication ports (no change in Slurm behavior).

NOTE: Correcting this would necessitate assigning the same set of ports
to all components of the heterogeneous job (not possible today) plus changes to
srun in order to better synchronize the step startup and error handling.

Same logic as done in commit fb296c70 done for energy.

Bug 4336

If heterogeneous job step is unable to acquire MPI reserved ports then
    avoid referencing NULL pointer.
bug 4333

Force tres change on a job to send data to the database.

This should be happening already, but this just makes it always happen.

This fixes the possibility of going into this loop when we hadn't
setup the tres_req_cnt.  The simple case Coverity reported is if the
job is already finished it goes here and we never set up tres_req_cnt.

Coverity CID 178897

This fixes the possibility of referencing a NULL pointer if the
reservation doesn't exist anymore when testing.

Coverity CID 178898

Bug 3647.

Bug 4353.

Also collapse a nested %{with cray} block leftover from earlier work.

Bug 4332.

Logic was inverted from the correct behavior.

Bug 4332.

Slurm package should not try to manage configs; leave this to the
admin to setup as they wish. This avoids an issue on RPM install if
/etc/slurm is a symlink to somewhere else.

Bug 4332.

Bug 4332.

Bug 4332.