Skip to content

Revert "[202412] [Mellanox] platform: Enable cache on init"#1178

Merged
r12f merged 1 commit into202412from
revert-1145-202412_enable_cmis_cache
May 28, 2025
Merged

Revert "[202412] [Mellanox] platform: Enable cache on init"#1178
r12f merged 1 commit into202412from
revert-1145-202412_enable_cmis_cache

Conversation

@r12f
Copy link

@r12f r12f commented May 28, 2025

Reverts #1145

The original PR is closed and change is causing image issues:

$ show int st
Traceback (most recent call last):
  File "/usr/local/bin/intfutil", line 900, in <module>
    main()
  File "/usr/local/bin/intfutil", line 880, in main
    interface_stat.display_intf_status()
  File "/usr/local/bin/intfutil", line 452, in display_intf_status
    self.get_intf_status()
  File "/usr/local/lib/python3.11/dist-packages/utilities_common/multi_asic.py", line 156, in wrapped_run_on_all_asics
    func(self,  *args, **kwargs)
  File "/usr/local/bin/intfutil", line 539, in get_intf_status
    self.table += self.generate_intf_status()
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/intfutil", line 483, in generate_intf_status
    port_oper_speed_get(self.db, key),
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/intfutil", line 204, in port_oper_speed_get
    return appl_db_port_status_get(db, intf_name, PORT_SPEED)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/intfutil", line 169, in appl_db_port_status_get
    optics_type = port_optics_get(appl_db, intf_name, PORT_OPTICS_TYPE)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/intfutil", line 226, in port_optics_get
    if is_rj45_port(intf_name):
       ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/utilities_common/platform_sfputil_helper.py", line 306, in is_rj45_port
    platform_chassis = sonic_platform.platform.Platform().get_chassis()
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/sonic_platform/platform.py", line 35, in __init__
    CmisApi.set_cache_enabled(True)
    ^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: type object 'CmisApi' has no attribute 'set_cache_enabled'

@r12f r12f requested a review from lguohan as a code owner May 28, 2025 18:57
@r12f r12f merged commit 09a5352 into 202412 May 28, 2025
4 checks passed
mssonicbld added a commit that referenced this pull request May 29, 2025
…03 (#1181)

```<br>* 3829186 - (HEAD -> 202503) Merge branch '202412' of https://github.com/Azure/sonic-buildimage-msft into 202503 (2025-05-29) [Sonic Automation]
* 09a5352 - (origin/202412, 202412) Revert "[202412] [Mellanox] platform: Enable cache on init" (#1178) (2025-05-28) [Riff]
* 1b3d9eb - [TACACS] Fix build issue caused by missing patch file. (#1177) (2025-05-28) [DavidZagury]
* f771865 - [action] [PR:22639] [platform][arista] Fix NVMe sensor chip address in sensors.conf for 7060X6-64PE-B (#1165) (2025-05-28) [mssonicbld]<br>```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants