This patch removes the MemLimitEnforce parameter from Slurm and moves the
condition to enable the feature to the JobAcctGatherParam=OverMemoryKill.
Setting OverMemoryKill will now enable the mechanism that was before enabled
with MemLimitEnforce=yes, so activation of killing jobs or steps due to
exceeding memory will be done in the same place. Also reconfiguring or
starting slurmd/ctld with incorrect configuration will cause a fatal error.

Bug 5479

Redundant logic committed in 62e378a4
bug 6337

this fixes bug introduced in commit 4facada1 when a job was
submitted with a valid gres name but count of zero

We decided with AMAT to remove gres/mps percent specification.

Add a debug statement.

Bug 5823

When the backup slurmdbd is active, there is no need to print
connection errors every 10 minutes when the backup checks in on the
primary, at least at the 'error' level.

Prevent dbd clients, like sacctmgr, from printing a connection error
when running commands against the backup slurmdbd unless debug2.
Before, the clients would always print errors when trying to access
the primary dbd first.

Bug 5823

The GRES Links info was added to v18.08, but the unpack was
missing that field in v19.05.
bug 6337

Issuing job cancel on the wrong job ID. Also increase sleep in a
test to better try to overlap the jobs.

Support GRES types that include numbers (e.g. "--gres=gpu:123g:2").
Also change number suffix processing to use common function
See commit 396e82a167
bug 6345

See commit 396e82a167
bug 6345

See commit 396e82a167
bug 6345

Based on logic previously in src/plugins/burst_buffer/common/burst_buffer_common.c
bug 6345

If a Unix socket path exceeds the limit, emit errors and exit.
Fix memory leak on socket() failure.

Bug 5386

Fix issue introduced in commit 164d4878 that avoided a job to be modified
setting the current reservation to none.

Bug 6309

Need to easily detect real memory leaks

Bug 6217

Forgot to free a list

TRESBillingWeights and PriorityWeightTRES.

which caused negative values and values starting with a decimal
to be invalid for PriorityWeightTRES and TRESBillingWeight.

Bug 6217

The user can run multiple jobs on different nodes at the same time
or any number of non-gres/mps jobs on the same node. Different users
can also use MPS at the same time on any given node.

bug 6313

This is useful for testing and debugging.

Bug 6321

This reverts commit 4dc5839f.

Causes build failure on CentOS 7.6. Introduced in f7da0636.

Bug 6319.

Bug 5916.

As pthread_rwlock_t is a struct, we cannot use this syntax
to initialize all elements of the array to the same value
as we would if these were any basic type.

Instead, move the initialization to a dedicated loop to ensure
all locks are initialized correctly.

This does not seem to have caused any issues on Linux platforms,
but macOS at least does notice this mistake.

Bug 6319.