From 3ce09c3d58aeb403b72ccde7dab3468ece7b4205 Mon Sep 17 00:00:00 2001 From: abbas1902 Date: Mon, 9 Dec 2024 23:40:25 +0000 Subject: [PATCH] Fix node_is_fr to handle empty string instead --- .../modules/slurm_files/scripts/util.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/community/modules/scheduler/schedmd-slurm-gcp-v6-controller/modules/slurm_files/scripts/util.py b/community/modules/scheduler/schedmd-slurm-gcp-v6-controller/modules/slurm_files/scripts/util.py index c8b3fc9704..c434cc503b 100755 --- a/community/modules/scheduler/schedmd-slurm-gcp-v6-controller/modules/slurm_files/scripts/util.py +++ b/community/modules/scheduler/schedmd-slurm-gcp-v6-controller/modules/slurm_files/scripts/util.py @@ -1607,7 +1607,7 @@ def node_is_tpu(self, node_name=None): return self.cfg.nodeset_tpu.get(nodeset_name) is not None def node_is_fr(self, node_name:str) -> bool: - return self.node_nodeset(node_name).future_reservation is not None + return self.node_nodeset(node_name).future_reservation != "" def is_dormant_fr_node(self, node_name:str) -> bool: fr = self.future_reservation(self.node_nodeset(node_name))