From d534d41db4237b018f18a3063c4ceb86b91fbe42 Mon Sep 17 00:00:00 2001 From: Michael Jennings Date: Tue, 21 Dec 2021 19:02:58 -0700 Subject: [PATCH] helpers: Recognize & ignore new "planned" state Slurm 21.08 has added a *very* welcome new feature called the "planned" state; this state is used to indicate nodes which would otherwise appear "idle" but are being held for a large/high-priority job. This is a frequent source of user confusion and angst (because they would look at the node counts and wonder why their job wasn't running even though enough nodes appeared to be empty), so having this is a big win for user services/consultant staff and scheduler admins everywhere! As with the handling of the `reboot` states, full recognition of this state will be available in 1.4.4 and 1.5, but for now, the helpers will just ignore this state completely. --- helpers/node-mark-offline | 2 +- helpers/node-mark-online | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/helpers/node-mark-offline b/helpers/node-mark-offline index 568ccc4..ef9b1b9 100644 --- a/helpers/node-mark-offline +++ b/helpers/node-mark-offline @@ -63,7 +63,7 @@ elif [[ "$NHC_RM" == "slurm" ]]; then OLD_NOTE_LEADER="${LINE[1]}" OLD_NOTE="${LINE[*]:2}" case "$STATUS" in - *'@'*|*'#'*|boot*) + *'@'*|*'#'*|boot*|*-*|plnd*) # These states aren't handled yet. echo "$0: State \"$STATUS\" not yet handled; ignoring." exit 0 diff --git a/helpers/node-mark-online b/helpers/node-mark-online index 407a136..4af85ad 100644 --- a/helpers/node-mark-online +++ b/helpers/node-mark-online @@ -60,7 +60,7 @@ elif [[ "$NHC_RM" == "slurm" ]]; then # Slurm does not run the HealthCheckProgram on nodes in the DOWN state, # but if someone runs NHC by hand, we want to be able to do the right thing. case "$STATUS" in - *'@'*|*'#'*|boot*) + *'@'*|*'#'*|boot*|*-*|plnd*) # These states aren't handled yet. echo "$0: State \"$STATUS\" not yet handled; ignoring." exit 0