Skip to content

Commit

Permalink
helpers: Recognize & ignore new "planned" state
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
mej committed Dec 22, 2021
1 parent 1fbbaea commit d534d41
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion helpers/node-mark-offline
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
2 changes: 1 addition & 1 deletion helpers/node-mark-online
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit d534d41

Please sign in to comment.