diff --git a/tests/platform/test_sfp.py b/tests/platform/test_sfp.py index 7de20af0042..d6b057daff7 100644 --- a/tests/platform/test_sfp.py +++ b/tests/platform/test_sfp.py @@ -13,7 +13,11 @@ import pytest from platform_fixtures import conn_graph_facts +from loganalyzer import LogAnalyzer +pytestmark = [ + pytest.mark.disable_loganalyzer # disable automatic loganalyzer +] def parse_output(output_lines): """ @@ -56,9 +60,15 @@ def test_check_sfp_status_and_configure_sfp(testbed_devices, conn_graph_facts): * show interface transceiver eeprom * sfputil reset """ - ans_host = testbed_devices["dut"] + if ans_host.facts["asic_type"] in ["mellanox"]: + loganalyzer = LogAnalyzer(ansible_host=ans_host, marker_prefix='sfp_cfg') + loganalyzer.load_common_config() + + loganalyzer.ignore_regex.append("kernel.*Eeprom query failed*") + marker = loganalyzer.init() + cmd_sfp_presence = "sudo sfputil show presence" cmd_sfp_eeprom = "sudo sfputil show eeprom" cmd_sfp_reset = "sudo sfputil reset" @@ -114,6 +124,9 @@ def test_check_sfp_status_and_configure_sfp(testbed_devices, conn_graph_facts): assert len(intf_facts["ansible_interface_link_down_ports"]) == 0, \ "Some interfaces are down: %s" % str(intf_facts["ansible_interface_link_down_ports"]) + if ans_host.facts["asic_type"] in ["mellanox"]: + loganalyzer.analyze(marker) + def test_check_sfp_low_power_mode(testbed_devices, conn_graph_facts): """ @@ -126,6 +139,13 @@ def test_check_sfp_low_power_mode(testbed_devices, conn_graph_facts): """ ans_host = testbed_devices["dut"] + if ans_host.facts["asic_type"] in ["mellanox"]: + loganalyzer = LogAnalyzer(ansible_host=ans_host, marker_prefix='sfp_lpm') + loganalyzer.load_common_config() + + loganalyzer.ignore_regex.append("Eeprom query failed") + marker = loganalyzer.init() + cmd_sfp_presence = "sudo sfputil show presence" cmd_sfp_show_lpmode = "sudo sfputil show lpmode" cmd_sfp_set_lpmode = "sudo sfputil lpmode" @@ -178,3 +198,6 @@ def test_check_sfp_low_power_mode(testbed_devices, conn_graph_facts): intf_facts = ans_host.interface_facts(up_ports=mg_facts["minigraph_ports"])["ansible_facts"] assert len(intf_facts["ansible_interface_link_down_ports"]) == 0, \ "Some interfaces are down: %s" % str(intf_facts["ansible_interface_link_down_ports"]) + + if ans_host.facts["asic_type"] in ["mellanox"]: + loganalyzer.analyze(marker)