Catch jobs with empty pack_job_id_sets.
Corruption bug fixed by previous commit.

Bug 6847

Rejected hetjobs were not correctly dump because pack_job_id_set was
not properly set in _slurm_rpc_allocate_pack() when submision was
rejected.

New helper function _create_pack_job_id_set() to avoid duplicated
code.

Bug 6847

Fix for test 21.39. If the job didn't start running before the
account/qos was removed, then the job would never start running, and the
test would fail.

Bug 6942.

Previous logic could hang indefinitely if srun did not return
It also performed unneeded test setup before confirming that the
system size was adequate to execute the test.

bug 7050

Remove excess tabs to properly align code
Make comment formats consistent

No functional changes.

Bug 7050.

Bug 7050.

Bug 7050.

Handles cases where last line of slurm.conf is not empty.

Bug 7050.

One part of this test can fail in a heterogeneous system. Do not run the
part of the test that would fail if the system is heterogeneous.

Bug 7051.

backport test7.20 changes from master to v18.08.

bug 7050

select/cons_res is recommended.

Bug 7138

Bug 6987.

Bug 6466.

Bug 7101.

The name variable hasn't been set yet, so this is always NULL. Print the
uid/gid instead. While here, treat uid/gid as uint32_t, and use strtoul()
rather than atoi() to avoid issues with high-number uid/gid values.

Fixes GCC 9 warning.

Bug 7101.

Bug 7101.

Use RETRY_DELAY per to mirror job complete delay but without a max retry
count for the time being.

Bug 6970.

Signed-off-by: Brian Christiansen <brian@schedmd.com>

Bug 6964

The reason was being set after the message was sent to the db. Also
clear the draing and reboot states before the message is sent so that
the event state will show DOWN.

Bug 6964

Bug 6964

so that new jobs can't get on the node.

Bug 6964

for completing job.

Bug 6927

Bug 6980

Bug 6945

Bug 6926

Continuation of 89b791bf.

Bug 7045.

To indicate that a job is dependent or has an invalid dependency.
Not used for now, just added and removed according to its meaning.

Bug 7045.

Bug 7080

Bug 7092.

Can happen if SrunPortRange has been set too small, especially on shared
login nodes launching multiple large-scale srun processes.

Bug 6916

Bug 6995

Job steps that run on cloud nodes and use the alias_list - in other
words, SlurmctldParameters=cloud_dns is not in slurm.conf - all talk
directly back to the slurmctld. To make that happen, we set the parent
tank of each stepd to -1. However, we also set the rank of each stepd to
0. this meant that when each stepd sent a REQUEST_STEP_COMPLETE RPC to
the slurmctld, they would tell slurmctld to clean up node 0 in the step
allocation. So, multi-node step allocations weren't cleaning up after
the steps completed and would cause subsequent job steps to hang. The
step allocations would only clean up properly at the end of the job.

Ensure that each stepd uses the correct rank so that job steps are
properly cleaned up after each step completes.

Bug 6467.

They were associated to these two commits:

b4d7de48
6871185a

Bug 5562.

Error reported by CLANG

Cherry pick to 18.08.

Bug 6996.

unlimited could get overwritten with default queue depth preventing the
whole queue from being looked at -- especially in a high-throughput
envrionment.

Bug 6822

Co-authored-by: Morris Jette <jette@schedmd.com>

Node memory overallocation wouldn't be properly detected since we would
just be interpreting the available memory as RealMemory - MemSpecLimit,
ignoring other job's memory usage.

Bug 5562.

This compares a job memory request against each selected node available
memory, interpreting the latter for now as RealMemory - MemSpecLimit.

Bug 5562.