Cast function return values to (void) to eliminate error

In the scenario where a batch job is submitted from remote cluster and
runs on another cluster, the cluster running the job doesn't need to
send back it's contact information if the srun is coming from the same
cluster.

Missed in 9b5c97b9

Ported from 7a4aa7f2.

Continuation of:
49d756da

and make it so the slurmd formats it correctly when it comes in.

Continuation of commit 7db90c8a

This is needed to make it so we can run pmix with multi-slurmd.

to slurm_allocation_lookup(). Previous commit removed unused
slurm_allocation_lookup().

Since it's not used and slurm_allocation_lookup_lite() is, we're going
to rename slurm_allocation_allocation_lite() to
slurm_allocation_allocation(). The rename is in the next commit.

Was sending the response_rec for every job on a normal cluster when it
only needs to send when the allocation was from a different cluster.

The response_cluster_rec was pointing to the conf's cluster_name which
gets free'd and reallocated on a reconfig. So the cluster_name was
pointing to garbage.

Bug 3446

bug 3446

so that it will only run the tests if you have check.

If size is 0, it will allocate 0 + 2. 2 for the magic and the size. But
xmalloc will then retun the pointer pointing past the newly allocated
memory (p[2]) which can cause a segfault.

burst_buffer/cray - Support default pool which is not the first pool
    reported by DataWarp and log in Slurm when pools that are added or removed
    from DataWarp.
bug 3453

Refactor the loading in of plugins.

Instead of creating static functions everytime.

This reverts commit fd690a9c.