fix: deadlock storcli inventory #253

Merged
cyrinux merged 1 commit from fix/cle/storcli-inventory into master 2022-11-30 09:52:03 +01:00

View file

@ -21,14 +21,17 @@ def storecli(sub_command):
stdout=subprocess.PIPE, stdout=subprocess.PIPE,
stderr=subprocess.STDOUT stderr=subprocess.STDOUT
) )
p.wait()
stdout = p.stdout.read().decode("utf-8") stdout, stderr = p.communicate()
if p.returncode != 0: if stderr:
mesg = "Failed to execute command '{}':\n{}".format( mesg = "Failed to execute command '{}':\n{}".format(
" ".join(command), stdout " ".join(command), stdout
) )
raise StorcliControllerError(mesg) raise StorcliControllerError(mesg)
stdout = stdout.decode("utf-8")
data = json.loads(stdout) data = json.loads(stdout)
controllers = dict([ controllers = dict([
( (
c['Command Status']['Controller'], c['Command Status']['Controller'],