Merge pull request #177 from mila-iqia/update-slurm-version-retrieving

Fix Slurm version retrieving through sacct

slurm_state/parsers/slurm_parser.py

@@ -49,17 +49,16 @@ def get_slurm_version(self):
             # return the value of the configuration
             return self.cluster["slurm_version"]
         else:
-            print("3")
             # Launch the sacct or sinfo command to get its version
             remote_command = f"{self.slurm_command_path} -V"
             response = self.launch_slurm_command(remote_command)
             assert len(response) == 1
             version_regex = re.compile(r"^slurm (\d+\.\d+\.\d+)$")
-            if m := version_regex.match(response):
+            if m := version_regex.match(response[0]):
                 return m.group(1)
             # If the version has not been identified, raise an error
             raise Exception(
-                f'The version "{response}" has not been recognized as a Slurm version.'
+                f'The version "{response[0]}" has not been recognized as a Slurm version.'
             )
 
     def launch_slurm_command(self, remote_command):