Previously sched_nodes was set to the estimated nodes on the last
evaluated partition that was adding a reservation, instead of the
one offering the earliest estimated start time.

Natural continuation of fdae6a05

.

Bug 7344.

Signed-off-by: Dominik Bartkiewicz <bart@schedmd.com>

Bug 8014

Bug 7749

Bug 7995

Create generic function get_gres_count to get the node counts of any
GRES, not just GPUs.
Make get_gpu_count able to parse any combination of GRES names and
types.
Create get_gpu_count wrapper of get_gres_count for backwards
compatibility.
Expand the regex to not include newlines.
Rename variable gpu_count to gres_count.

Bug 7995

gres_node_config_load() requires gres_list to work properly after
fully merge slurm.conf with gres.conf logic added in 4d7df8b0.

Bug 7986

Bug 7490

If not enforcing QOS, it's possible to submit a job without a qos. If
submitting such a job to multiple partitions where at least one has a
qos, slurmctld would abort in a development build. A non-development
build didn't segfault only because _find_qos_part doesn't dereference
the NULL pointer. Prevent the abort.

Bug 7171

Bug 7171

Bug 7712

Now it will work with multiple GPU types. This currently affects tests
like a subset of the test39.* series and test1.62.

Bug 7884.

If there are less than 2 sockets per node, only the first part of the
test needs to be skipped. So the second half of the test was changed
to still run as long as 2+ GPUs are configured.

Also add headers to sub-tests while here.

Bug 7884.

Bug 7884.

Ensure gres_plugin_node_config_load() is called with a gres_list.

Bug 7884.

Otherwise we would be incorrectly reusing the previous to the
reconfiguration slurm.conf GRES value.

Continuation of previous commit.

Bug 7884.

Ensure that the stepd has slurm.conf GRES data. If the slurm.conf
input argument is NULL for gres_plugin_node_config_load(), then
gres_devices will end up being NULL, causing all GRES binding to not
work.

Bug 7884.

Previous logic would only call s_p_hashtbl_create() to create the hashtable
when the file acct_gather.conf could be successfully stat()'d. This lead to
a subsequent attempt to pack the non-created hashtable into a buffer which
triggered the abort.

This makes it so the hashtable is uncondtionally created no matter if the
file is missing.

Bug 7893.

gethostbyaddr() can potentially return a fully-qualified domain name,
which breaks backwards compatibility with the shortname AllocNodes
expected pre 19.05.

Bug 7653.

Fortunately the extra arguments were provided at the end, and thus ignored on
most common platforms.

Bug 7555.

This reverts commit e233ed11.

Fortunately the extra arguments were provided at the end, and thus ignored on
most common platforms.

Bug 7555.

Bug 6933

Bug 7870

Bug 7912

Bug 7374

Bug 7326.

Bug 7877.

Bug 7758.

No functional change.

Bug 7927.

Continuation of b2e3bb06.

Bug 6605

Instead of giving user as "nobody", give user numeric id. The RPC stats
last the lifetime of the slurmctld process and thus may have users that
have been removed.

There is no performance benefit from caching users in sdiag as it will
only lookup each user once. Caching adds qsort insert cost to every
user for no benefit. Swapped from non-cached call to direct user lookup.

Bug 7845.

Bug 7845.