- 15 Dec, 2015 6 commits
-
-
Danny Auble authored
Conflicts: src/slurmctld/proc_req.c
-
Danny Auble authored
_slurm_rpc_complete_job_allocation. This is a rewrite of 438365ec which didn't catch the job_ptr wasn't in a lock so the memory issue could still of existed. This hopefully fixes all the spots the job_ptr wasn't in the lock. Fixes bug 2146
-
Danny Auble authored
This reverts commit 438365ec.
-
Brian Christiansen authored
Bug 2261
-
Danny Auble authored
15.08.5. Bug 2243
-
David Bigagli authored
bug 2171
-
- 14 Dec, 2015 3 commits
-
-
Morris Jette authored
Decrease parallelism in job cancel request to prevent denial of service when cancelling huge numbers of jobs. bug 2256
-
Morris Jette authored
Prevent triggering gang scheduling within a partition if configured with PreemptType=partition_prio and PreemptMode=suspend,gang. The essence of this fix is to change a "<=" to "<" in cons_res/job_test.c: - if ((p_ptr->part_ptr->priority <= jp_ptr->part_ptr->priority) && + if ((p_ptr->part_ptr->priority < jp_ptr->part_ptr->priority) && but logic was also added to insure that a partition configuration with PreemptMode did not override PreemptType != partition_prio. bug 2232
-
David Bigagli authored
-
- 11 Dec, 2015 8 commits
-
-
Tim Wickberg authored
No changes to logic
-
Morris Jette authored
-
Tim Wickberg authored
Previously an error() would be logged when the attempt to open the job script using the new directory format failed but the successive fallback to the old directory structure was successful, leading to confusion when troubleshooting. Move emitted warnings to debug(), and only error() after failing to open in both directory structures. Add a note about backwards compatibility to both functions - we cannot remove these fallbacks as directory structure for pending jobs does not change on Slurm version update, and people may need to chain multiple version update together to get to a current slurm version which would correctly update slurmctld state files but leave pending jobs in the old directory structure. Bug #2244.
-
Morris Jette authored
If a job is requeued while in the process of being launch, remove it's job ID from slurmd's record of active jobs in order to avoid generating a duplicate job ID error when launched for the second time (which would drain the node). bug 2240
-
Morris Jette authored
In slurmctld log file, log duplicate job ID found by slurmd. Previously was being logged as prolog/epilog failure. bug 2240
-
David Bigagli authored
-
Morris Jette authored
-
Morris Jette authored
-
- 10 Dec, 2015 7 commits
-
-
Danny Auble authored
The problem comes from situations where a node range is given, leaving the task count at NO_VAL is the desired behavior in this case.
-
Danny Auble authored
so we can set it correctly before tasks are set.
-
Tim Wickberg authored
-
Danny Auble authored
-
Tim Wickberg authored
messages like FAILURE: 1 scontrol show assoc_mgr did not show correct information (6 != 5). to happen erroneously.
-
David Bigagli authored
-
Tim Wickberg authored
The error messages associated with these are: /home/tim/slurm/src/common/callerid.c:337:21: warning: address of array 'entryp->d_name' will always evaluate to 'true' [-Wpointer-bool-conversion] else if (!entryp->d_name) ~~~~~~~~~^~~~~~ 1 warning generated. /home/tim/slurm/src/common/gres.c:831:19: warning: comparison of unsigned expression < 0 is always false [-Wtautological-compare] if ((tmp_uint64 < 0) || (tmp_uint64 >= NO_VAL)) { ~~~~~~~~~~ ^ ~ 1 warning generated. /home/tim/slurm/src/slurmctld/node_mgr.c:1684:26: warning: if statement has empty body [-Wempty-body] if (first_new == NULL); ^ /home/tim/slurm/src/slurmctld/node_mgr.c:1684:26: note: put the semicolon on a separate line to silence this warning /home/tim/slurm/src/slurmctld/node_mgr.c:1756:26: warning: if statement has empty body [-Wempty-body] if (first_new == NULL); ^ /home/tim/slurm/src/slurmctld/node_mgr.c:1756:26: note: put the semicolon on a separate line to silence this warning /home/tim/slurm/src/slurmctld/node_mgr.c:1830:26: warning: if statement has empty body [-Wempty-body] if (first_new == NULL); ^ /home/tim/slurm/src/slurmctld/node_mgr.c:1830:26: note: put the semicolon on a separate line to silence this warning /home/tim/slurm/src/plugins/switch/generic/switch_generic.c:837:27: warning: address of array 'if_rec->ifa_addr->sa_data' will always evaluate to 'true' [-Wpointer-bool-conversion] if (!if_rec->ifa_addr->sa_data) ~~~~~~~~~~~~~~~~~~~^~~~~~~
-
- 09 Dec, 2015 11 commits
-
-
Morris Jette authored
news
-
Danny Auble authored
-
Danny Auble authored
-
Danny Auble authored
-
Danny Auble authored
no matter who calls the function.
-
Danny Auble authored
-
Danny Auble authored
function to common.c from last commit.
-
Alejandro Sanchez authored
through sacctmgr.
-
Morris Jette authored
select/cray: Prevent NHC from running more than once per job or step. bug 2192
-
Tim Wickberg authored
-
Morris Jette authored
In both sched/basic and backfill: If a job can not be started due to some account/qos limit, then don't start other jobs which could delay jobs. The old logic would skip the job and start other jobs, which could delay the higher priority job. bug 2129
-
- 08 Dec, 2015 5 commits
-
-
Brian Christiansen authored
-
Tim Wickberg authored
This also corrects per user burst buffer space calculation
-
Brian Christiansen authored
-
Danny Auble authored
_calc_billable_tres that was missed there.
-
Danny Auble authored
-