sonic-net / sonic-mgmt

Configuration management examples for SONiC
Other
194 stars 714 forks source link

Add script `platform_tests/sfp/test_sfputil.py` into T0 PR checker. #13428

Closed yutongzhang-microsoft closed 3 months ago

yutongzhang-microsoft commented 3 months ago

Description of PR

In PR #13220, we add a batch of control plane test scripts into onboarding T0 PR checker, but some of them failed. In this PR, we fix the failed test script platform_tests/sfp/test_sfputil.py and let it run successfully in T0 PR checker.

Summary: Fixes # (issue)

Type of change

Back port request

Approach

What is the motivation for this PR?

In PR #13220, we add a batch of control plane test scripts into onboarding T0 PR checker, but some of them failed. In this PR, we fix the failed test script platform_tests/sfp/test_sfputil.py and let it run successfully in T0 PR checker.

How did you do it?

How did you verify/test it?

platform_tests/sfp/test_sfputil.py::test_check_sfputil_presence[vlab-01-None] PASSED [ 16%]
platform_tests/sfp/test_sfputil.py::test_check_sfputil_error_status[vlab-01-None-sudo sfputil show error-status] PASSED [ 33%]
platform_tests/sfp/test_sfputil.py::test_check_sfputil_error_status[vlab-01-None-sudo sfputil show error-status --fetch-from-hardware] PASSED [ 50%]
platform_tests/sfp/test_sfputil.py::test_check_sfputil_eeprom[vlab-01-None] PASSED [ 66%]
platform_tests/sfp/test_sfputil.py::test_check_sfputil_reset[vlab-01-None] PASSED [ 83%]
platform_tests/sfp/test_sfputil.py::test_check_sfputil_low_power_mode[vlab-01-None] SKIPPED [100%]INFO:root:Can not get Allure report URL. Please check logs

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

wsycqyz commented 3 months ago

Have triggered to rerun failed jobs.