diff --git a/NEWS b/NEWS index 8466df4dd6ac26a8245612cf61bd1aa65f019ce7..71c32afb62fef3207ba3ec1409af8b8abfaf5713 100644 --- a/NEWS +++ b/NEWS @@ -353,6 +353,11 @@ documents those changes that are of interest to users and administrators. has been modified to accept a "Pack" and "NoPack" option. These options can be used to override the CR_PACK_NODE configuration option. +* Changes in Slurm 14.11.10 +=========================== + -- If a node is down do not set it in power suspend mode, as it could be + resumed and defeat the purpose of being shut down. + * Changes in Slurm 14.11.9 ========================== -- Correct "sdiag" backfill cycle time calculation if it yields locks. A diff --git a/src/slurmctld/power_save.c b/src/slurmctld/power_save.c index 05c11db5b340c1b445d52a285a00209730e07914..262fd21fc39b04c09391878180353bc1e665c95f 100644 --- a/src/slurmctld/power_save.c +++ b/src/slurmctld/power_save.c @@ -181,10 +181,12 @@ static void _do_power_work(time_t now) if (run_suspend && (susp_state == 0) && ((suspend_rate == 0) || (suspend_cnt < suspend_rate)) && - (IS_NODE_IDLE(node_ptr) || IS_NODE_DOWN(node_ptr)) && + (IS_NODE_IDLE(node_ptr)) && (node_ptr->sus_job_cnt == 0) && (!IS_NODE_COMPLETING(node_ptr)) && (!IS_NODE_POWER_UP(node_ptr)) && + (!IS_NODE_DRAINED(node_ptr)) && + (!IS_NODE_DRAINING(node_ptr)) && (node_ptr->last_idle < (now - idle_time)) && ((exc_node_bitmap == NULL) || (bit_test(exc_node_bitmap, i) == 0))) {