diff --git a/device/celestica/x86_64-cel_seastone-r0/sonic_platform/helper.py b/device/celestica/x86_64-cel_seastone-r0/sonic_platform/helper.py index 2af4bc0412e9..f0415d856358 100644 --- a/device/celestica/x86_64-cel_seastone-r0/sonic_platform/helper.py +++ b/device/celestica/x86_64-cel_seastone-r0/sonic_platform/helper.py @@ -100,7 +100,7 @@ def ipmi_fru_id(self, id, key=None): if not key: p = subprocess.Popen( cmd1_args, universal_newlines=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE) - raw_data, err = p.communicate() + raw_data = p.communicate()[0] else: cmd2_args = ["grep", str(key)] with subprocess.Popen(cmd1_args, universal_newlines=True, stdout=subprocess.PIPE) as p1: @@ -108,7 +108,7 @@ def ipmi_fru_id(self, id, key=None): raw_data = p2.communicate()[0] if p2.returncode == 1: raise subprocess.CalledProcessError(returncode=p2.returncode, cmd=cmd, output=raw_data) - result = raw_data.strip() + result = raw_data.strip() except subprocess.CalledProcessError: status = False return status, result diff --git a/device/celestica/x86_64-cel_silverstone-r0/sonic_platform/helper.py b/device/celestica/x86_64-cel_silverstone-r0/sonic_platform/helper.py index d0447c2beec1..ef02f50e9f7a 100644 --- a/device/celestica/x86_64-cel_silverstone-r0/sonic_platform/helper.py +++ b/device/celestica/x86_64-cel_silverstone-r0/sonic_platform/helper.py @@ -65,7 +65,7 @@ def ipmi_raw(self, netfn, cmd): cmd = ["ipmitool", "raw", str(netfn), str(cmd)] p = subprocess.Popen( cmd, universal_newlines=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE) - raw_data, err = p.communicate() + raw_data = p.communicate()[0] if err == '': result = raw_data.strip() else: @@ -90,7 +90,7 @@ def ipmi_fru_id(self, id, key=None): raw_data = p2.communicate()[0] if p2.returncode == 1: raise subprocess.CalledProcessError(returncode=p2.returncode, cmd=cmd, output=raw_data) - result = raw_data.strip() + result = raw_data.strip() except subprocess.CalledProcessError: status = False return status, result