- 26 May, 2017 2 commits
-
-
Morris Jette authored
-
Gary authored
For jobs submited to multiple partitions, report the job's earliest start time for any partition. bug 3754
-
- 25 May, 2017 14 commits
-
-
Isaac Hartung authored
Burst buffer jobs cannot be run as root currently, change test to prevent that. Bug 3723.
-
Danny Auble authored
Bug 3756
-
Dominik Bartkiewicz authored
Bug 3756
-
Doug Jacobsen authored
fragmentation. Bug 3756
-
Dominik Bartkiewicz authored
Two jobs completing simultaneously leads to make_node_idle() returning before it has a chance to decrement node_ptr->owner_job_cnt, which can result in the node being "owned" by that user even through no jobs are running on it. Move the decrement block to the end at a fini label, and make sure all return paths pass through it. While moving that add a guard against node_ptr->owner_job_cnt underflowing. Bug 3771.
-
Dominik Bartkiewicz authored
If a job is considered on a partition with ExclusiveUser=YES then it would be marked as if it was submitted with the --exclusive flag, which would lead to delays launching it on ExclusiveUser=NO partitions, and cause lower-than-expected cluster usage. As a side effect, the job_ptr->part_ptr->flags need to be tested wherever WHOLE_NODE_USER is considered, instead of just job_ptr->details->whole_node. Bug 3771.
-
Tim Wickberg authored
Wrong author attributed by mistake. This reverts commit 9128476a.
-
Tim Wickberg authored
Wrong author attributed by mistake. This reverts commit a02d04f1.
-
Tim Wickberg authored
leaving the node owned. Two jobs completing simultaneously leads to make_node_idle() returning before it has a chance to decrement node_ptr->owner_job_cnt, which can result in the node being "owned" by that user even through no jobs are running on it. Move the decrement block to the end at a fini label, and make sure all return paths pass through it. While moving that add a guard against node_ptr->owner_job_cnt underflowing. Bug 3771.
-
Tim Wickberg authored
WHOLE_NODE_USER. If a job is considered on a partition with ExclusiveUser=YES then it would be marked as if it was submitted with the --exclusive flag, which would lead to delays launching it on ExclusiveUser=NO partitions, and cause lower-than-expected cluster usage. As a side effect, the job_ptr->part_ptr->flags need to be tested wherever WHOLE_NODE_USER is considered, instead of just job_ptr->details->whole_node. Bug 3771.
-
Alejandro Sanchez authored
_setup_assoc_cond_limits was using the table 'prefix' passed by argument in the where clause to select the where clause prefix.deleted=something. It turns out that _setup_assoc_cond_limits is called by these functions: as_mysql_modify_assocs as_mysql_remove_assocs as_mysql_get_assocs as_mysql_acct_no_users which set the prefix to 't2' before the call if a QOS is provided or if WithSubAccounts is provided. The 't2' prefix is fine for other where conditions in that case, but for choosing the deleted we need the t1 which is the table we're selecting the records off. Bug 3835
-
Alejandro Sanchez authored
-
Morris Jette authored
-
Tim Shaw authored
-
- 24 May, 2017 20 commits
-
-
Danny Auble authored
-
Danny Auble authored
-
Danny Auble authored
-
Danny Auble authored
-
Danny Auble authored
There isn't much we can do about this, it will always be misspelled until they fix it upstream. We could correct it, but then every time we run autogen.sh we would have to ignore the change which seems like more work than I would want to keep doing.
-
Tim Shaw authored
Bug 3821
-
Danny Auble authored
-
Danny Auble authored
do the correct thing. It turns out you can't += to *_SOURCES in a Makefile.am. The corresponding *_OBJECTS will not be made. So I added a HAVE_LUA define and handled things accordingly.
-
Danny Auble authored
typecasts.
-
Danny Auble authored
-
Danny Auble authored
# Conflicts: # src/common/slurm_acct_gather.c # src/plugins/sched/backfill/backfill.c # src/slurmctld/node_scheduler.c
-
Danny Auble authored
done in that commit, but only in the main scheduler instead of backfill.
-
Tim Wickberg authored
'scontrol update reservationname=foo duration=unlimited' sets INFINITE as the duration, which needs to be translated to a year as is done elsewhere. Otherwise it'll convert to 49710 days, which is definitely wrong. Bug 3836.
-
Brian Christiansen authored
-
Brian Christiansen authored
And handle memory leak. Continuation of d3ee28fa
-
Brian Christiansen authored
-
Alejandro Sanchez authored
build_all_frontend_info() could be called twice unintentionally.
-
Alejandro Sanchez authored
Bug 3443.
-
Alejandro Sanchez authored
_get_begin_next_month() may be called twice in succession as a side-effect of the MIN macro. Store in a local variable first.
-
Alejandro Sanchez authored
MAX() will re-evaluate the higher value argument; if this is a function is may be called twice over, leading to unintended side effects or a crash. Bug 3443.
-
- 23 May, 2017 4 commits
-
-
Brian Christiansen authored
fed.weight was removed in ae0c71d0.
-
Brian Christiansen authored
fed.weight was removed in ae0c71d0 but the packing of it wasn't removed in the case of a null slurmdb_cluster_rec_t.
-
Danny Auble authored
Otherwise the recently newly added '#include <lua.h>' condition will break the build on systems without any Lua installation. Follow on to commit 7cc4d0d8. Bug 3681.
-
Danny Auble authored
This also fixes the fed_mgr on the backup slurmctld to start backup correctly if the backup takes control more than once. Bug 3827
-