bug 3979

Bug 3967

Bug 3979 and 3989

This reverts commit d49081df.

Bug 3979 and 3989

Otherwise we can end up printing Start times greater than End times,
leading to confusion when reading sacct output. 0 is displayed as Unknown.
Cosmetic change.

Bug 3940.

This behavior was introduced in bug 2504, commit 7fb0c981 and bug 2643
commit 988edf12 respectively.

The reasoning is that sysadmins who see nodes with Reason "Not Responding"
but they can manually ping/access the node end up confused. That reason
should only be set if the node is trully not responding, but not if the
HealthCheckProgram execution failed or returned non-zero exit code. For
that case, the program itself would take the appropiate actions, such
as draining the node and setting an appropiate Reason.

Bug 3931

burst_buffer logic modified to support sizes in both SI and EIC size units
    (e.g. M/MiB for powers of 1024, MB for powers of 1000).
bug 3922

This patch removes a window in which a message bound for the DBD
could be packed with the non-dbd packing.  This would result in a
packed msg_type, but nothing else.  When that message was given to
the DBD it would complain forever about an unpacking error.

Bug 3891 and 3939

done.

Pretty much remove 10cc6f93

Bug 3919

workflows through the slurmd.

Bug 3833

from the slurmctld make sure the step allocation is made aware of it.

Bug 3926

Also see commit b30139e5

Bug 3581.

Bug 3815

It would be nice to figure out a way to remove the check for version all
together but I (Danny) couldn't figure out how that would be done since we
need to know which libs/headers to use and on systems with multiple
installed and no 'lua' lib (Ubuntu) you have to use the PKG_CHECK_EXISTS
to set up the pkg name for PKG_CHECK_MODULES or you don't get things set up
correct when trying to link.

Bug 3919

more than 1 partition or when the partition is changed with scontrol.

Bug 3849

submitted to a QOS/association.

Bug 3849

Correct error message when ClusterName in configuration files does not match
    the name in the slurmctld daemon's state save file.

the requested value, instead of always setting one.

This would make --hint=multithread not work at all.

See Bug 3855 (commit 3c852da1)

Issue originated from commit 82a959a8.

bug 3447

Turns out if the extern step is created here and the job was killed long
before hand the step is made erroneously and can cause an assert just lines
later.

Bug 3898

set correctly.

Bug 3858

What this does is populate the node_hash_table as nodes are being read in
instead of after the node_record_table_ptr has been fully populated.

This speeds up a start of a slurmd with a system of 10000 nodes from
> 1 minute to less than a second.

In 17.11 we will remove the linear xstrcmp check as it should no longer be
needed.

Bug 3885

batch job takes longer than it takes to finish.

Bug 3833

time.

Bug 3833

An array was only being partially cleared due to bad logic
bug 3876

Bug 3874.

Improve selection of jobs to preempt when there are multiple partitions
    with jobs subject to preemption.
bug 3824

Prevent segfault from pointer dereference to the QOS that is
being deleted.

Fix to commit 3e8aa451.

Fix regression from commit c05dcb8a (bug 1923) that doesn't take
into consideration a blank char * as a valid option.

This fixes the scenario like

sacctmgr list associations user=''

which would only print account associations.
Bug 3862