Same logic was in both backfill plugin and slurmctld/job_scheduler.c
No change in logic

Bug 2130

Adopted processes didn't have access to the job's devices.

Bug 2130

Full revert of c2fbf88f, 13b64c35 had caught part of this, but
this will revert it completely.  The code just wasn't needed in modern
Slurm.  It appears the patch came from an older version of Slurm that
didn't handle this correctly.

This reverts commit 29f25688.

Conflicts:
	NEWS

Looks like this isn't needed, commit c2fbf88f doesn't appear to be needed
and is what is causing this issue.  c2fbf88f was added from an older
version of Slurm where this was already handled correctly in commit
815e5a44.

get messages like

slurmctld: error: _handle_assoc_tres_run_secs: job 33355: assoc 1 TRES node grp_used_tres_run_secs underflow, tried to remove 21 seconds when only 0 remained.

which are bad.

If a store fails, do not retry for 30 seconds

Cray job NHC delayed until after burst buffer released and epilog completes
    on all allocated nodes.
bugs 2099 and 2192

Add new buffer state of BB_STATE_POST_RUN indicating the post_run
  operation has been started
Add new bb_p_job_test_post_run function to test status of post_run
  operation
Execute post_run operation before the stage_out operation to opimize
  use of compute nodes (rather than burst buffer space)

Release a job's allocated licenses only after epilog runs on all nodes
    rather than at start of termination process.
bug 2192

sched/backfill - Delay backfill scheduler for completing jobs only if
    CompleteWait configuration parameter is set (make code match documentation).

moved code to common location.

Addition to b9668b2b

Bug 2030

Invalid user input could generate slurmctld error(), changed to info()

Eliminated redundant data structure
Minor performance improvements

correctly when PriorityFlags=CALCULATE_RUNNING is set.

Previously the slurmctld could seg fault if the tres_alloc_str is NULL.

Prevent slurmdbd divide by zero if no associations defined at rollup time.

else behind.

uint64_t.

removes the kill_extern_procs() function added in the referenced commit.

This appears to be the way we do it with other steps and seems to work
correctly.

as all the associations from the database will be lower case.

sched/backfill: This change moves some logic used to support the
   bf_min_age_reserve configuration parameter for slightly better
   performance.

Correct job task count calcuation if only node count and ntasks-per-node
    options supplied.
bug 2196

bug 2199

SchedulerParameters now accepts keyword of "max_rpc_count" in
  addition to the documented "max_rpc_cnt"