- 16 Oct, 2017 10 commits
-
-
Danny Auble authored
-
Morris Jette authored
Note: This check is needed since the maximum file path length really is file system dependent and not something local to the Linux distro Coverity CID 45200
-
Morris Jette authored
Coverity CID 45343
-
Morris Jette authored
Really should not happen, but this adds checks Coverity CID 171449
-
Morris Jette authored
Coverity CID 45167
-
Morris Jette authored
-
Morris Jette authored
Coverity CID 44823
-
Morris Jette authored
-
Morris Jette authored
-
Morris Jette authored
Coverity CID 45160
-
- 14 Oct, 2017 8 commits
-
-
Felip Moll authored
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.
-
Tim Wickberg authored
deadline_time_limit is ensuring this is always set, but the optimizer appears to not be able to work out the flow here.
-
Tim Wickberg authored
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.)
-
Tim Wickberg authored
-
Tim Wickberg authored
This one's a bit confusing, and should be a false positive, but initializing the saveptr argument to strtok_r quiets things down.
-
Tim Wickberg authored
Treat default case as an error. Visible with GCC 6.3 and -01.
-
Morris Jette authored
-
Josh Samuelson authored
Pending job with administrator extended TimeLimit beyond partition's MaxTime remains pending with reason PartitionTimeLimit bug 4262
-
- 13 Oct, 2017 14 commits
-
-
Tim Wickberg authored
-
Morris Jette authored
-
Morris Jette authored
-
Brian Christiansen authored
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
-
Brian Christiansen authored
The perl api leaves undefined objects in the node_array returned by load_nodes() for any node that is hidden. But 4250
-
Tim Wickberg authored
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.
-
Morris Jette authored
Bug 4003
-
Morris Jette authored
as process exits Bug 4003
-
Tim Wickberg authored
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.)
-
Brian Christiansen authored
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.
-
Morris Jette authored
Coverity CID 44862, 44863
-
Morris Jette authored
Coverity CID 45160
-
Morris Jette authored
Coverity CID 45147
-
Morris Jette authored
Coverity CID 44874
-
- 12 Oct, 2017 8 commits
-
-
Brian Christiansen authored
2266778f
-
Marshall Garey authored
When creating a reservation, correct logic to ignoring job allocations on request. bug 4172
-
Marshall Garey authored
When creating a reservations, validate the CoreCnt specification matches the number of nodes listed. bug 4172
-
Morris Jette authored
Test for node count being too large Avoid printing two error messages in "scontrol create reservation" node count value is illigal.
-
Marshall Garey authored
Bug 4172
-
Brian Christiansen authored
was missed in 3e3b3c97
-
Brian Christiansen authored
which isn't allowed. Bug 4235
-
Felip Moll authored
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.
-