Remove run

Signed-off-by: maipbui <[email protected]>

Author: maipbui

device/mellanox/x86_64-mlnx_msn2700-r0/plugins/sfputil.py

@@ -158,7 +158,7 @@ def get_low_power_mode(self, port_num):
         lpm_cmd = ["docker", "exec", "syncd", "python", "/usr/share/sonic/platform/plugins/sfplpmget.py", str(port_num)]
 
         try:
-            output = subprocess.run(lpm_cmd, universal_newlines=True, capture_output=True, check=True).stdout
+            output = subprocess.check_output(lpm_cmd, universal_newlines=True)
             if 'LPM ON' in output:
                 return True
         except subprocess.CalledProcessError as e:
@@ -182,7 +182,7 @@ def set_low_power_mode(self, port_num, lpmode):
 
         # Set LPM
         try:
-            subprocess.run(lpm_cmd, check=True, universal_newlines=True, capture_output=True).stdout
+            subprocess.check_output(lpm_cmd, universal_newlines=True)
         except subprocess.CalledProcessError as e:
             print("Error! Unable to set LPM for {}, rc = {}, err msg: {}".format(port_num, e.returncode, e.output))
             return False
@@ -197,7 +197,7 @@ def reset(self, port_num):
         lpm_cmd = ["docker", "exec", "syncd", "python", "/usr/share/sonic/platform/plugins/sfpreset.py", str(port_num)]
 
         try:
-            subprocess.run(lpm_cmd, check=True, universal_newlines=True, capture_output=True).stdout
+            subprocess.check_output(lpm_cmd, universal_newlines=True)
             return True
         except subprocess.CalledProcessError as e:
             print("Error! Unable to set LPM for {}, rc = {}, err msg: {}".format(port_num, e.returncode, e.output))
@@ -269,7 +269,7 @@ def _read_eeprom_specific_bytes_via_ethtool(self, port_num, offset, num_bytes):
         eeprom_raw = []
         ethtool_cmd = ["ethtool", "-m", sfpname, "hex", "on", "offset", str(offset), "length", str(num_bytes)]
         try:
-            output = subprocess.run(ethtool_cmd, check=True, universal_newlines=True, capture_output=True).stdout
+            output = subprocess.check_output(ethtool_cmd, universal_newlines=True)
             output_lines = output.splitlines()
             first_line_raw = output_lines[0]
             if "Offset" in first_line_raw:

platform/mellanox/mlnx-platform-api/tests/test_chassis.py

@@ -276,12 +276,12 @@ def test_revision_permission(self):
         #Override the dmi file
         sonic_platform.chassis.DMI_FILE = "/tmp/dmi_file"
         new_dmi_file = sonic_platform.chassis.DMI_FILE
-        subprocess.run(["touch", new_dmi_file])
-        subprocess.run(["chmod", "-r", new_dmi_file])
+        subprocess.call(["touch", new_dmi_file])
+        subprocess.call(["chmod", "-r", new_dmi_file])
         chassis = Chassis()
         rev = chassis.get_revision()
         sonic_platform.chassis.DMI_FILE = old_dmi_file
-        subprocess.run(["rm", "-f", new_dmi_file])
+        subprocess.call(["rm", "-f", new_dmi_file])
         assert rev == "N/A"
 
     def test_get_port_or_cage_type(self):