Bug 4529

instead of the normal comment.  Also send it to the database if needed.

Bug 4529

as well as updating the system_comment.

Bug 4529

before returning.

Bug 4529

Bug 4529

Bug 4529

Bug 4529

Bug 4529

No real code change.

Bug 4529

Bug 4529

# Conflicts:
#	src/plugins/task/cgroup/task_cgroup_cpuset.c

This mostly reverts commit 8325e9f8.

Bug 4478

We decided to not do this by default for the slurmd.  See bug 4478 for
more info/reasoning.

We also didn't want to rely on #include <linux/oom.h> since that might
not exist everywhere.

Format for inserted X11 cookies is now hostname/unix:display, rather
than localhost:display. This seems to resolve some issues for different
setups while not causing any issues in my test environments. This also
helps avoid a conflict between multiple nodes in a larger job if the
same display number is allocated on multiple nodes simultaneously.

Bug 4420.

Bug 4483

Signed-off-by: Boris Karasev <karasev.b@gmail.com>

Bug 4541

The slurm and pmix RPMs both attempt to install a libpmi. While Slurm's
version should be preferred, the best way to handle this is to have both
pmix and slurm packages split this off into a separate RPM that can
conflict with each other, giving the admin a way to resolve the conflict
manually.

Bug 4511.

The cluster name could have been free'd before it would be strdup'ed.
Happens when a federated persisent connection is established and the
cluster needs to sync up with the remote cluster.

Bug 4503

Remove the origin job when a remote sibling job was cancelled while the
origin was down.

e.g.
job originated on cluster1
job runs on cluster2
cluster1 goes down
job is cancelled on cluster2
cluster1 comes backup. The origin job should be removed if the sibling
job was cancelled.

Replace some references to "pack job" with "heterogenous job".
bug 4546

The user_name is only populated if LaunchParameters=send_gids is set.
Without this field set, the pam_setup call will get NULL instead of the
user_name, leading to batch job launch failures. (The step launch code
already handles this same issue separately.)

Bug 4412.

Previously written to "admin_comment" in commit 540d0e5c
bug 4529

This reverts commit 36a86e50.

Turns out this needs more work than expected.  hash_part_inx is used
afterwards and HASH_FCN no longer uses/sets it.

For future reference it looks like in the new code you would just
call HASH_TO_BKT after calling HASH_FCN.  I didn't want to spend
time testing it, so just rolled back the code.  As this is the only
place in the entirity of Slurm we currently use it I am not super
concerned.

NOTE: the macro HASH_FCN (which is really HASH_JEN) is identical in
both places outside the already explained differences.

On a job [pack]allocation RPC request, if the allocation succeed but the
send response message back to the client failed (i.e. srun was killed before
it could receive the response), then modify the job_record pointer so that
the job_state is set to FAILED, the exit_code as if the job got a SIGTERM
signal and the state_reason to FAIL_LAUNCH. Then users when querying
the job with sacct can discern that something bad happend for this scenario,
instead of STATE being showed as COMPLETED and the ExitCode as 0:0.

Bug 4513.

Use FREE_NULL_BUFFER instead, otherwise we could attempt to
free_buffer this a second time if we jump to the rwfail label.

Bug 4484.

When printing fields in sacct with user specified units (--units), the
nnodes field showed an incorrect string. This commit reverts a65fa572
and avoids the unit conversion, which does not make sense outside the
context of a Blue Gene systems (deprecated) anyways.

Bug 4490.

Slurm may generate empty manifest files depending on configuration
and library availability. Disable the new empty manifest check to
allow builds to proceed with rpm 4.13+ / Fedora 25+.

Bug 4453.

(Fixing tim@schedmd.com's mistake on prior commit.)

Bug 4467.