No change in logic

Move reused GRES parsing logic to a shared function
bug 5743

This shouldn't happen in practice.
bug 5749

Test1.113 was sometimes failing due to timing issues related to
the Cray Node Health Check running. Add small sleep to (mostly)
eliminate the problem.

The option was previously ignored with the cons_res plugin plus
topology/tree. With cons_tres, handle the option by ignoring the
topology and using the default node selection logic.

add logic to provide optimized resource allocation with respect
to topology/tree configuration. Mostly working
Fix max-node enforcement

Introduced in commit 3e727cc4
Coverity CID 188211

Problem introduced in commit a044d01b
Coverity CID 188212

They were being referenced prior to initialization

This better handles job submit failures and reduces the time spend
waiting for output files by overlapping the wait for all files.

without this the total GRES counter can keep getting incremented,
the _get_job_cnt() calculation can overflow and extra GRES be
allocated to the job

This reverts debugging accidentally committed in 48ec9695

There are a few anomalies with respect to front-end mode, which
test test changes address

Since we don't have any bluegene code this is pointless to send
NULL's over.

Bug 5698

this is related to commit 845e134b
when a step ends, flag that it is done on ALL nodes so that it's
memory can be released. Note that test1.19 would fail in front-end
mode when there was a default memory allocation. That test would
spawn 2 steps, but the first would not release its memory when
that step ended, so the second step would not start.

this is useful for testing. it is unclear why this was disabled

Bugs 5699, 5700, 5701

Bug 5698

Bug 5701

Bug 5701

Bug 5700

Bug 5700

Bug 5699

Bug 5699

Bug 5699

Bug 5699

Bug 5699

Bug 5699

Bug 5699

It doesn't do anything yet.