From 74c059fade69153a93a103ee99b1e55b8bcc83ee Mon Sep 17 00:00:00 2001 From: Xuanqi He Date: Thu, 30 Apr 2026 18:21:44 -0400 Subject: [PATCH] Refer SLURM_SCONTROL_RESUME_FAILED_STATE in is_bootstrap_failure comment Avoid hard-coding the state flags in the inline comment so future changes to the constant do not require a comment update. --- src/slurm_plugin/slurm_resources.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/slurm_plugin/slurm_resources.py b/src/slurm_plugin/slurm_resources.py index 8c0aae67..896b8e8f 100644 --- a/src/slurm_plugin/slurm_resources.py +++ b/src/slurm_plugin/slurm_resources.py @@ -734,7 +734,7 @@ def is_bootstrap_failure(self, ec2_instance_missing_max_count, nodes_without_bac self.state_string, ) return True - # Dynamic node in DOWN+CLOUD+POWERED_DOWN+NOT_RESPONDING state + # Dynamic node in SLURM_SCONTROL_RESUME_FAILED_STATE. elif self.is_bootstrap_timeout(): # We need to check if nodeaddr is set to avoid counting powering up nodes as bootstrap failure nodes during # cluster start/stop.