Fix memory leak in previous patch.

Bug 5479

Coverity 188464

The format for this field is "<inst>:<svc_id>:<comp_id>"; if inst or svc_id
are unset then they're blank, and the parsing code had assumed they would
always be blank as the systems slurmsmwd were developed on never used them.

Bug 5411.

Prevents job priorities from being lowered to 0 or from underflowing.

Bug 5165

Bug 5403

sinfo output in this test is grouping all nodes together in one
output line (e.g. "nid[000001-000099'), while the data used to
have one node per line. Adding the "-N" option to the test returns
the output to one node per line, as required for the test to work.

Bug 5750

# Conflicts:
#	src/common/slurmdbd_defs.c

Bug 5544

Bug 5544

Bug 5613

Surrounding with spaces ensures the closing paren isn't accidentally
included in the link.

Bug 5463.

Now slurmctld will fatal if the two incompatible memory enforcement mechanisms
are configured at the same time (JobAcctGather + task/cgroup). Also, a new info
message in slurmctld startup will indicate if there are not any configured
mechanism.

Bug 5479

MemLimitEnforce and setting NoOverMemoryKill kills the job, or the job step
respectivelly if the memory is exceeded at some point based on JobAcctGather
plugin information, in a JobAcctGatherFreq frequency basis.

Both are incompatible with task/cgroup memory enforcement. The defaults have
been changed to not setting any mechanism, and info/fatal messages added
accordingly to give useful info to the admin. Moreover NoOverMemoryKill has
been deprecated in favor of OverMemoryKill and a fix on the detection of job
memory confinement has been included.

Bug 5479

Bug 5574

Bug 5762

bug 5669

Remove hotel block while here.

Two tests were failing when using --enable-front-end configuration

hosts in each list.

Bug 5741

by integers.

We understand this reverts bc7b91cd, but in all our testing we have
been unsuccessful finding what that commit actually does.

Bug 5711
Bug 5746

This reverts commit 204da0b6.

# Conflicts:
#	NEWS

So this takes us back to 204da0b6.

See bug 5746

distribution.

Bug 5706

This reverts commit 450cd6c8.

Bug 5711

This will be re-reverted in 19.05 with other code changes we didn't feel
comfortable dealing with in 18.08.

to --cpu-bind

Continuation of a57fafb2

No longer just used when using profiling.

ic/ofed, fs/disk, fs/lustre, pages, vmem

Bug 5702

"Rebood" changed to "Reboot"

This throws 0c3133a2 out, as that'd already been handled here previously.
(And thus we do not add a separate NEWS entry for 18.08.1 on this issue.)

The backup slurmctld can end up without an agent thread if
read_slurm_conf() takes too long to complete. Forcing the agent_init()
here again ensures it is restarted if not currently running.

This was fixed upstream in c1a537db.

Bug 5633.

"Generalized Hypercube - A Topology Plugin" was accidentally given
a 50 minute slot instead of 30 minutes as it should have been. Carry
the fix forward through the schedule, and cede the extra time to Google.