Coverity CID 44823

Coverity CID 45160

Setting -Werror tricks the test into failing on the

"error: In the GNU C Library, "major" is defined by <sys/sysmacros.h>.
 For historical compatibility, it is  currently defined by <sys/types.h>
 as well, but we plan to remove this soon. To use "major", include
 <sys/sysmacros.h> directly. If you did not intend to use a system-defined
 macro "major", you should undefine it after including <sys/types.h>."

error. Since the normal Slurm build uses -Werror, this warning on including
both headers will then cause the build itself to fail.

Bug 3982.

deadline_time_limit is ensuring this is always set, but the optimizer
appears to not be able to work out the flow here.

Move test above the accounting limit check. Change the test
to verify first_job_ptr instead of pack_job_id to avoid warning
about a possible dereference of an uninitialized variable lower down.
(Note that both pack_job_id and first_job_ptr are set at the same time,
so this test is functionally equivalent but able to convince the compiler
that things will be okay.)

This one's a bit confusing, and should be a false positive, but
initializing the saveptr argument to strtok_r quiets things down.

Treat default case as an error.
Visible with GCC 6.3 and -01.

Pending job with administrator extended TimeLimit beyond partition's
MaxTime remains pending with reason PartitionTimeLimit
bug 4262

The contoller will return node records with a NULL name for nodes that
are hidden. This is so that you can map a partition_info's nodes --
using it's node_inx[] -- to a node record in the returned node_array
from slurm_load_node().

Previously the perl api would leave an undefined object in the the
node_array if the hidden nodes were found before a real node in the
node_array and any hidden nodes at the end of the array from the
controller wouldn't be counted for in the perl node_array. This patch
adds empty hashes for hidden nodes and preserves the record_count and
node_array from the slurmctld.

Bug 4250

The perl api leaves undefined objects in the node_array returned by
load_nodes() for any node that is hidden.

But 4250

With our current packaging approach this appears to be
the preferrable format. We only include a release version
number in the tarball when != 1. It's a rare occurance, but
the spec file needs to be flexible enough to handle it without
modifications.

Bug 4003

as process exits

Bug 4003

Needed when building on non-RHEL/SuSE platforms that do not have
the systemd macros defined. Note that installing such packages is
not recommended and may fail.

(Building in such a manner is necessary for test30.1.)

CID 177947

by removing redundant checks for WAIT_ states. If the job was already
released then the state_reason won't have the WAIT_HELD states.

Coverity CID 44862, 44863

Coverity CID 45160

Coverity CID 45147

Coverity CID 44874

2266778f

When creating a reservation, correct logic to ignoring job allocations on
    request.
bug 4172

When creating a reservations, validate the CoreCnt specification matches
    the number of nodes listed.
bug 4172

Test for node count being too large
Avoid printing two error messages in "scontrol create reservation"
  node count value is illigal.

Bug 4172

was missed in 3e3b3c97

which isn't allowed.

Bug 4235

Starting from glibc-2.25 the macros major and minor are only available
from sys/sysmacros.h. This patch uses an autoconf macro to detect the
location and includes the header accordingly.

Bug 3982.

to prev commit 33e352a6

Before:
When a reservation goes away the reservation pointer is removed from
every pending job and the job could run outside the requested
reservation.

Now:
When the reservation goes away, pending jobs requesting the reservation
are put into a held state unless the reservation has the
NO_HOLD_JOBS_AFTER_END flag.

Bug 4214

In a job allocation spanning KNL and non-KNL nodes and requiring a reboot,
    do not attempt to set default NUMA or MCDRAM modes on non-KNL nodes.
bug 4205

The "sort" command output ordering depends upon the node names.
This change makes the test work with any node names.