Also remove some redundant logic

- avoid `abort()` when collective is failed
- added logging of coll details for fail cases

Bug 5067

- avoid `abort()` when collective is failed
- added logging of coll details for fail cases

Bug 5067

Note, this is setting it up so we can use defunct functions.  It will
probably need to be properly fixed in a future version so we don't
do this.

This change is associated with commit 6be109d9

gres_per_socket requires sockets-per-node count specification
gres_per_task requires task count specification
these restrictions are required in order for cons_res to
  support these options in a finite amount of time/code

Bug 5098.

with preemption or when job requests a specific list of hosts.

Bug 5293.

Coverity CID 186992