From f6c5302b4dc57a9ac0449e1a4716a881c826a079 Mon Sep 17 00:00:00 2001
From: David Bigagli <david@schedmd.com>
Date: Tue, 1 Sep 2015 14:53:20 +0200
Subject: [PATCH] If a node is down don't set it in power suspend mode.
---
NEWS | 2 ++
src/slurmctld/power_save.c | 2 +-
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/NEWS b/NEWS
index eef7efe6201..311cbe0aa67 100644
--- a/NEWS
+++ b/NEWS
@@ -72,6 +72,8 @@ documents those changes that are of interest to users and administrators.
-- Correct RebootProgram logic when executed outside of a maintenance
reservation.
-- Requeue job if possible when slurmstepd aborts.
+ -- 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.8
==========================
diff --git a/src/slurmctld/power_save.c b/src/slurmctld/power_save.c
index 05c11db5b34..26e92adb019 100644
--- a/src/slurmctld/power_save.c
+++ b/src/slurmctld/power_save.c
@@ -181,7 +181,7 @@ 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)) &&
--
GitLab