If bf_min_prio_reserve was not the last option listed, the setting would
not have been applied correctly, and an error message printed instead.

Bug 7242.

Bug 6221

Bug 6221

Put comments in the code to help figure out what is being packed.
No real code change.

Bug 7204

Bug 7204

Bug 6221

Return ESLURM_INVALID_TIME_MIN_LIMIT instead of ESLURM_INVALID_TIME_LIMIT
to provide a better error message back to the user.

Bug 6221

Bug 6221

Bug 7179.

bug 7203

Bug 6847
Bug 7190

For heterogeneous jobs, do not count the each component against the QOS or
association job limit multiple times.

bug 7190

bug 7197

bug 7197

bug 7197

Bug 6847

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.

JobHeldUser state reason was properly set for user held jobs during the
course of job_allocate() execution until calling _select_nodes_parts()
which could override it under certain conditions (i.e. when accounting
policy limit is hit). The fix restores it back when appropriate.

Continuation of 8712a254.

Bug 7136.

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

bug 7050

Bug 7146.

No functional changes to fsync_and_close(), but tag the file_type as
const and clean up some minor style issues while moving this.

Add include for fd.h header where needed as well.

Bug 7146.

This streamlines the code a bit and uses more common code.

Parsing logic failed to support underscore in plugin names, which
is now the standard for "cray_aries" configurations.