Whenever possible, avoid allocating nodes that require a reboot.
Previous logic failed to re-sort the job set table based upon
the need for rebooting to achieve the desired features (e.g. KNL
MCDRAM or CACHE mode).
bug 2726

Put header files in alphabetic order, No change in logic

# Conflicts:
#	src/slurmctld/controller.c

trying to verify the cluster name (which may try to /create/ files or
directories) *before* dropping privs results in a fatal error as
slurmctld tries to create items which ultimately fail.  Moving
this process until after the privs and uid have changed allows
the process to succeed.

Reported by Jon Nelson <jdnelson@dyn.com>

Bug 2728

Reject invalid step at submit time rather than leaving it queued.
Bug 2722 describes one of the use cases triggering the bug.

"Argument with 'nonnull' attribute passed null"

Minor update to commit 2fad3bcf
This leaves the files locked until file write completes.

Disable a job deadline test with no input time limit if the default
partition has a default time limit.

This partially restores commit 03b2cfb5
Logic was not closing file descriptor, which left the file locked and
leaked an open file descriptor.

tasks-per-node/nodes != tasks print warning and ignore
ntasks-per-node.

Bug 2520

On a Cray, the output file isn't being created the second time.

Make test_id in more tests be just the numeric value rather than "test#.#"
for consistency with the other tests.

Make test_id in test30.1 be just the numeric value rather than "test30.1"
for consistency with the other tests.

The account still had maxnodes=1 set preventing the qos grpnodes to take
affect. This showed up on slower machines because it takes a second for
the changes to get to the controller.

Test would originally try to start more jobs than default_queue_depth
in SchedulerParameters and fail

Job was failing on Cray/kachina due to timeout. Increase job time
  limit from 1 to 2 minutes.

g++ always internally has a '#define _GNU_SOURCE 1'. The type difference
between that and '#define _GNU_SOURCE' which is used throughout Slurm
causes a compiler warning about redefinition, which breaks the build.

Adding this one #ifndef back solves this, as this is appears to be the
only header defining it that needs to build both with g++ and gcc.

make it to the slurmctld when using message aggregation.

make sure we handle it correctly when the database comes back up.

ours in from the config.h which is now included.

Give test job an extra second to start. Test was failing by one
  second on kachina.

The Perl API didn't define HAVE_CONFIG, which prevented headers
from include config.h which has a different definition of the VERSION
macro than the Perl API build system. Add these two #undef's to
prevent build error and work around the namespace issue.

Get the maximum file pathname size from system include file rather
  than local #define. This was causing failures on kachina test.

strcasestr is a non-standard glibc extension.

Remove unused includes, and aliases from slurm_xlator.

Also remove local version of inet_pton/ntop functions hiding
behind HAVE_INET_NTOP/PTON. Those are both in POSIX1.2001.

Delete files, remove from Makefile.
Remove stray includes as well to keep build happy.