From f774aa9453bdd0e22280da05e5595c1d3499d5d8 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..605283c5bb 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 bool(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))