- 04 Jun, 2018 14 commits
-
-
Morris Jette authored
-
Morris Jette authored
I was seeing rare failures on the test due to timing issues. This increased timeout seems to fix the issue for me.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
These calls to select_g_select_jobinfo_get() are a no-op on non-select/bluegene systems, so remove them. Since we're no longer modifying the job_desc through that call, remove the debug3 statements around this as well. The rest is removal of HAVE_BG blocks. Collapse the one else if that was broken across an ifndef block as well here.
-
Tim Wickberg authored
Remove the temporary variables and directly reference the structure values while here. Removes another select_g_select_jobinfo_get() block.
-
Tim Wickberg authored
Continue to remove select_g_select_jobinfo_get() calls.
-
Tim Wickberg authored
Continue removing select_g_alter_node_cnt() calls. The err_cpus return return for: select_g_select_nodeinfo_get(node_ptr->select_nodeinfo, SELECT_NODEDATA_SUBCNT, NODE_STATE_ERROR, &err_cpus); is always zero for non-select/bluegene plugins, so this whole function can be drastically simplified. Move the node_ptr->name check up front. Invert the remaining logic, which makes it clear any nodes in DRAIL/FAIL/DOWN are what will trigger the clusteracct_storage_g_node_down call.
-
Tim Wickberg authored
SELECT_GET_NODE_SCALING is always 1 for non-select/bluegene plugins. Pack it directly. On a future RPC layer this should be removed once the client commands have been updated as well.
-
Tim Wickberg authored
SELECT_APPLY_NODE_MAX_OFFSET is a no-op on non-bluegene plugins, so we can now just pack total_nodes directly. Similarly, update_part does not need to translate a min node count into a midplane count anymore.
-
Tim Wickberg authored
Just strip out all HAVE_BG code here to remove the select_g_alter_node_cnt() calls buried within.
-
Tim Wickberg authored
select_p_alter_node_cnt() is a no-op here for every non-bluegene select plugin.
-
Tim Wickberg authored
This is always zero on non-bluegene select plugins: select_g_select_nodeinfo_get(node_ptr->select_nodeinfo, SELECT_NODEDATA_SUBCNT, NODE_STATE_ERROR, &err_cpus); So all of this code can be collapsed down to three lines.
-
Tim Wickberg authored
Only used by select/bluegene, which is being removed. Still need to remove all calling paths into select_g_alter_node_cnt, so leave stubbed out for now.
-
- 02 Jun, 2018 4 commits
-
-
Brian Christiansen authored
srun would not return an exit code if a previous task exited before a latter task exited with a signal. If multiple tasks exit with a signal, srun returns the highest signal. Partially reverts commit 04b449e1 -- the setting of local_global_rc to NO_VAL as srun doesn't need to know whether it's been set or not anymore. srun always sets the signal if a task exited with a signal. Bug 5083
-
Brian Christiansen authored
-
Michael Hinton authored
-
Michael Hinton authored
NONE was not documented. Bug 5161
-
- 01 Jun, 2018 2 commits
-
-
Morris Jette authored
-
Morris Jette authored
Avoid left-over test input file
-
- 31 May, 2018 9 commits
-
-
Tim Wickberg authored
-
Tim Wickberg authored
src/plugins/ and src/common/assoc_mgr.c are still left to convert.
-
Morris Jette authored
-
Morris Jette authored
-
Morris Jette authored
-
Alejandro Sanchez authored
-
Danny Auble authored
No functional change. Bug 4999.
-
Alejandro Sanchez authored
-
Alejandro Sanchez authored
There were two code paths building an allocation response by calling its own static _build_alloc_msg() function: 1. src/slurmctld/proc_req.c 2. src/slurmctld/srun_comm.c These two functions diverged and both had members that were not filled in but were filled in the other. This patch makes it so we change the signature of the one in proc_req.c to make it extern and then in srun_comm.c we call this newly common function. Also added cpu_freq_[min|max|gov] members in the common one since these were the only members missing in proc_req.c function (the one in srun_comm.c had more members missing, like all the ntasks_per*, account, qos or resv_name). Bug 4999.
-
- 30 May, 2018 11 commits
-
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Update slurm.spec and slurm.spec-legacy as well
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Marshall Garey authored
Only trust MUNGE signed values, unless the RPC was signed by SlurmUser or root. CVE-2018-10995.
-
Tim Wickberg authored
-
Tim Wickberg authored
Already vetted by slurmctld/slurmd, no need to re-check here.
-
Tim Wickberg authored
-