Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Temporarily skip lpmode test for some transceivers with known issue #16547

Merged
merged 1 commit into from
Jan 17, 2025
+14 −0
Merged

Temporarily skip lpmode test for some transceivers with known issue #16547

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions tests/platform_tests/api/test_sfp.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -258,6 +258,11 @@ class TestSfpApi(PlatformApiTestBase):
'supported_max_tx_power'
]

# xcvr to be skipped for lpmode test due to known issue
LPMODE_SKIP_LIST = [
{'manufacturer': 'Cloud Light', 'host_electrical_interface': '400GAUI-8 C2M (Annex 120E)'},
]

chassis_facts = None
duthost_vars = None

Expand Down Expand Up @@ -310,6 +315,15 @@ def is_xcvr_support_lpmode(self, xcvr_info_dict):
ext_identifier = xcvr_info_dict["ext_identifier"]
if ("QSFP" not in xcvr_type and "OSFP" not in xcvr_type) or "Power Class 1" in ext_identifier:
return False

# Temporarily add this logic to skip lpmode test for some transceivers with known issue
for xcvr_to_skip in self.LPMODE_SKIP_LIST:
if (xcvr_info_dict["manufacturer"].strip() == xcvr_to_skip["manufacturer"] and
xcvr_info_dict["host_electrical_interface"].strip() == xcvr_to_skip["host_electrical_interface"]):
logger.info("Temporarily skipping {} due to known issue".format(
xcvr_info_dict["manufacturer"]))
return False

return True

def is_xcvr_support_power_override(self, xcvr_info_dict):
Expand Down
Loading