Bug 5602

fa38c431 was the commit this was added in where s_p_get_float
wasn't added until e1d2a0c1.

The conditional for handling --overcommit was not sufficient to prevent
other option combinations from using that code path, resulting in
unexpected resource allocation behavior. Correct that to check that
overcommit is enabled.

Bug 5345.

Bug 5379.

This does change a network RPC after 18.08.0rc1, which is best to be
avoided. But other recent changes have already broken RPC compatibility
between what will be 18.08.0, so we might as well land this now.

Bug 5443.

and when reconfigured.

Bug 5602

The socket count argument was added to
gres_plugin_node_config_validate()
in commit 63b95516

And add into appropriate pack and unpack functions.

And set it in launch_prolog().

Previous logic would re-use altered node and core bitmaps,
which would tend to fail.
Change some data types for consistency

Cores on additional sockets may be used, but we want to insure some
cores on the identify sockets are included in the job allocation

In cases when you have a range first and then a single afterwards:

nid00[446-447],nid00392

It only worked correctly if it were reversed like so:

nid00392,nid00[446-447]

No functional change.

No function change here, identical to 17.11 protocol block.

Bug 5524

Basically count was never getting over 3 so none of the "end" code
was ever happening.

Make test suite aware of how GPUs are bound to sockets

Introduced in commit d382a477

Modify a gpu/socket binding test to recognize newly available
information from "scontrol show node" in commit 63b95516

# Conflicts:
#	src/slurmctld/fed_mgr.c