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 7326.

Bug 7877.

Bug 7758.

Continuation of b2e3bb06.

Bug 6605

Bug 7326.

Signed-off-by: Nate Rini <nate@schedmd.com>

Continuation of 92058c54



Signed-off-by: Jason Booth <jbooth@schedmd.com>

Bug 7891

Partially reverts e12742f2

Bug 7868

Continuation of e12742f2

Bug 7868

Bug 7679

If a reservation with a name of "" (empty string) is created, then
salloc, sbatch, and srun can't submit jobs to that reservation. If
empty string is provided as the reservation name, generate a reservation
name instead.

Bug 7854.

Bug 7666.

Bug 7680.

pthread_create is not a async-signal-safe function, since it can't be
used in a fork (according to POSIX). Calling it may end-up with deadlock
in glibc.

Bug 7666.

Bug 7779

Bug 7779

Add some slurm.conf validation to make sure that typed GRES and
untyped GRES don't mix (e.g. do not allow gres=gpu:1,gpu:tesla:1).

Bug 7517

If a configured GPU matches a GPU on the system, match them together.
If a configured GPU has mismatched Cores or Links with the system GPU,
then omit that configured GPU from the final list.

Bug 7517

Before, data from slurm.conf was only partially merged if there was
no corresponding entry in gres.conf.
Now, Slurm tries to match up gres.conf records to what is defined in
slurm.conf.

Bug 7517

Bug 7169

Increase the maximum array len large to be packed/unpacked with one order of
magnitude, since the current value demonstrated it is not enough when an MPI
program spawns a considerable amount of tasks over a big set of nodes.

This limit was introduced in 627928f4.

Bug 7495

There was never any security to allow for this,
so we are just removing it.

Bug 7765

Admin/Operator users were not able to skip MaxQueryTimeRange when trying
to show/fix runaway jobs.
This commit uses _validate_operator() instead of
_validate_slurm_user() in _get_jobs_cond() as well as check for operators
in _fix_runaway_jobs().

Bug 7765

Bug 7708

Don't remove jobs from preemptee_candidates List.

Bug 7708

Since this happens inside a the user process, this can inadvertently
cause the user's job to die by running out of file descriptors.

Bug 7814.

Co-authored-by: William Arndt <warndt@lbl.gov>

Bug 7499

Regression introduced in fb26b706.

Bug 7675

Now the signaling of the batch step and the handeling of the flags is totally
handled in _kill_all_active_steps() in slurmd, and _handle_signal_container()
in stepd to ensure that:
- if KILL_JOB_BATCH then only batch container is signaled
- if KILL_FULL_JOB then batch script and its children are also signaled
- if both of the above then only the batch script and its children are signaled

We do not relay anymore on proctrack_g_signal() to handle the batch step
signaling anymore, therefore it works the same for all proctrack plugins.

This commit also includes minor related fixes in other code handling such
signaling flags, and documentation improvement.

Bug 7282

Signed-off-by: Tim Wickberg <tim@schedmd.com>

Bug 7697

1cd43fce

Bug 7630

Bug 7727

This was missed in commit 6ac4ce84.

An incorrect logic with the variables holding available cores in the
gres_plugin_job_core_filter3() function lead to a potential infinite
"while (avail_cores_tot > req_cores)" loop, leaving slurmctld unresponsive.

Bug 7685.