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.

Add logic in read_config.c to handle old config option name.

Bug 5916.

This disables the test of tracking GPU/MPS by Type.

bug 5840

Add and comment out some debug messages to accrue time accounting.

Bug 6096

Added a new DebugFlag=accrue to show debug messages related to accrue time
count or other related stuff.

Bug 6096

Make sure count used is that for a single GPU rather than a total MPS
over multiple GPUs

Coverity CID 190461

This would be triggered by the gres.conf on the compute node having a
higher count of GRES defined with "File" parameters than the node has
defined in slurm.conf.

This also adds documentation and some tests. The actual scheduling
logic is still not written.

No new fields added, just duplicated the save/restore for v19.05

# Conflicts:
#	doc/man/man5/slurm.conf.5
#	src/plugins/sched/backfill/backfill.c
#	src/slurmctld/job_scheduler.c

Coverity CID 190390, 190393, 190397

Enabling this alters the scheduling sorting algorithm so that hetjob
components belonging to the same hetjob are attempted to be scheduled
consecutively (not interleaved with other jobs or hetjob components).

Bug 5579.

AccrueTime limits.

Bug 6002


Co-authored-by: Felip Moll <felip.moll@schedmd.com>

Coverity CID 190390, 190393, 190397

Fix for Coverity CID 190461

Add small note to documentation to clarify how to set different SlurmUser for
slurmdbd and ctld.

Bug 6306

This fleshes out logic so scontrol can be used to update a node's GRES.

Previous logic was failing to report errors in node's GRES update
requests.

No change in logic, improve some comments and logging

This is only used to define the slurmctld_comm_addr symbol in allocate.h.
Variable definitions do not belong in headers. Rather than relocate the
struct declaration, replace the struct which only holds a single port number
with a static variable serving the same purpose in allocate.c, and the
extern in this same location so srun.c can find it as well.

Replaces the Linux SO_PEERCRED calls.