Revert "[Mellanox]Check dmi file permission before access (#11309)"

This reverts commit 187f351.
sonic-net · Jul 6, 2022 · 296b21e · 296b21e
1 parent 7d06c7f
commit 296b21e
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 16 deletions.
diff --git a/platform/mellanox/mlnx-platform-api/sonic_platform/chassis.py b/platform/mellanox/mlnx-platform-api/sonic_platform/chassis.py
@@ -632,9 +632,6 @@ def _parse_dmi(self, filename):
         """
         result = {}
         try:
-            if not os.access(filename, os.R_OK):
-                return result
-
             with open(filename, "rb") as fileobj:
                 data = fileobj.read()
 

diff --git a/platform/mellanox/mlnx-platform-api/tests/test_chassis.py b/platform/mellanox/mlnx-platform-api/tests/test_chassis.py
@@ -267,16 +267,3 @@ def test_module(self):
         module_list = chassis.get_all_modules()
         assert len(module_list) == 3
         assert chassis.module_initialized_count == 3
-
-    def test_revision_permission(self):
-        old_dmi_file =  sonic_platform.chassis.DMI_FILE
-        #Override the dmi file
-        sonic_platform.chassis.DMI_FILE = "/tmp/dmi_file"
-        new_dmi_file = sonic_platform.chassis.DMI_FILE
-        os.system("touch " + new_dmi_file)
-        os.system("chmod -r " + new_dmi_file)
-        chassis = Chassis()
-        rev = chassis.get_revision()
-        sonic_platform.chassis.DMI_FILE = old_dmi_file
-        os.system("rm -f " + new_dmi_file)
-        assert rev == "N/A"