Many cosmetic modifications to eliminate warning message from GCC version
4.6 compiler, mostly due to unused variables.

Avoid clearing a job's reason from JobHeldAdmin or JobHeldUser when it is
otherwise updated using scontrol or sview commands. Patch based upon work
by Phil Eckert (LLNL).

Do not remove the backup slurmctld's pid file when it assumes control, only
when it actually shuts down. Patch from Andriy Grytsenko (Massive Solutions
Limited).

variable wasn't initialized in the job structure making it so that job
wouldn't run.

conn_types in a block definition.

allow them on Q systems.

allocation with jobs running on blocks that don't exist in the static
setup.

and vice versa the node->base partition lists will be displayed if setup
in your .slurm/sviewrc file.

Fix bug which would crash slurmcld if job's owner (not root) tries to clear
a job's licenses by setting value to "".

If a job is deferred due to partition limits, then re-test those limits
after a partition is modified. Patch from Don Lipari.

blocks.

correct geometry of the system.

shutdown and then the system size changed.  This would probably only happen
if you were emulating a system.

when we are on a small block, not a regular one.

isn't sent to the slurmd where environment variables would be overwritten
incorrectly.

position in the block instead of absolute.

If salloc was run as interactive, with job control, reset the foreground
process group of the terminal to the process group of the parent pid before
exiting. Patch from Don Albert, Bull.

Add cray.conf parameter of SyncTimeout, maximum time to defer job
scheduling if SLURM node or job state are out of synchronization with ALPS.

_job_create() consistent with similar logic in select_nodes().

partition in the slurm.conf.

One of our testers created an illegal topology.conf file.

He has a config you probably wouldn't see in production, but can see in
testing when you are sometimes given a collection of miscellaneous
resources.

          |-- nodes
switch1 --|
          |-- switch2 -- nodes

He tried the topology.conf file below. Switch s1 is defined twice. Slurm
accepted this config, but wouldn't allocate nodes from both switches to
one job.

SwitchName=s1 Nodes=xna[14-26]
SwitchName=s2 Nodes=xna[41-43]
SwitchName=s1 Switches=s2

I believe slurm shouldn't allow the second definition of switch s1. The
attached patch checks for duplicate switch names.
Patch from Rod Schultz, Bull.

This reverts commit 350ef5dc.

next parallel step is ran on a sub block, SLURM won't over
subscribe cnodes.