Fix node_is_fr to handle empty string instead

GoogleCloudPlatform · Dec 9, 2024 · 3ce09c3 · 3ce09c3
1 parent a897961
commit 3ce09c3
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/...ity/modules/scheduler/schedmd-slurm-gcp-v6-controller/modules/slurm_files/scripts/util.py b/...ity/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))