[action] [PR:17859] filterd backplane ports from STATE_DB while checking 'sudo sfputil show error-status'#19881
Merged
mssonicbld merged 1 commit intosonic-net:202505from Jul 29, 2025
Conversation
8 tasks
Collaborator
Author
|
Original PR: #17859 |
Collaborator
Author
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of PR
Summary: test_check_sfputil_error_status errors out on cisco-8000 smartswitch platforms when the DPU backplane ports are configured in STATE DB.
Fixes # (issue)
RC: sudo sfputil show error-status CLI errors out on cisco-8000 smartswitch platform when STATE_DB populate with backplane ports
cisco@MtFuji-dut:~$ sudo sfputil show error-status
Error: invalid port 'Ethernet224'
Valid values for port: ['Ethernet0', 'Ethernet8', 'Ethernet16', 'Ethernet24', 'Ethernet32', 'Ethernet40', 'Ethernet48', 'Ethernet56', 'Ethernet64', 'Ethernet72', 'Ethernet80', 'Ethernet88', 'Ethern
et96', 'Ethernet104', 'Ethernet112', 'Ethernet120', 'Ethernet128', 'Ethernet136', 'Ethernet144', 'Ethernet152', 'Ethernet160', 'Ethernet168', 'Ethernet176', 'Ethernet184', 'Ethernet192', 'Ethernet2
00', 'Ethernet208', 'Ethernet216']
Type of change
Filtered backplane ports from STATE_DB
cisco@MtFuji-dut:~$ sonic-db-cli STATE_DB KEYS "TRANSCEIVER_STATUS|*"
TRANSCEIVER_STATUS|Ethernet32
TRANSCEIVER_STATUS|Ethernet176
TRANSCEIVER_STATUS|Ethernet40
TRANSCEIVER_STATUS|Ethernet8
TRANSCEIVER_STATUS|Ethernet24
TRANSCEIVER_STATUS|Ethernet128
TRANSCEIVER_STATUS|Ethernet152
TRANSCEIVER_STATUS|Ethernet64
TRANSCEIVER_STATUS|Ethernet144
TRANSCEIVER_STATUS|Ethernet0
TRANSCEIVER_STATUS|Ethernet56
TRANSCEIVER_STATUS|Ethernet208
TRANSCEIVER_STATUS|Ethernet96
TRANSCEIVER_STATUS|Ethernet192
TRANSCEIVER_STATUS|Ethernet216
TRANSCEIVER_STATUS|Ethernet160
TRANSCEIVER_STATUS|Ethernet104
TRANSCEIVER_STATUS|Ethernet112
TRANSCEIVER_STATUS|Ethernet80
TRANSCEIVER_STATUS|Ethernet72
TRANSCEIVER_STATUS|Ethernet48
TRANSCEIVER_STATUS|Ethernet120
TRANSCEIVER_STATUS|Ethernet16
TRANSCEIVER_STATUS|Ethernet168
TRANSCEIVER_STATUS|Ethernet200
TRANSCEIVER_STATUS|Ethernet136
TRANSCEIVER_STATUS|Ethernet184
TRANSCEIVER_STATUS|Ethernet88
Back port request
Approach
Filtered backplane ports from STATE_DB
What is the motivation for this PR?
How did you do it?
Filtered backplane ports from STATE_DB
How did you verify/test it?
Ran the test_check_sfputil_error_status with fix and verified below CLI.
cisco@MtFuji-dut:~$ sudo sfputil show error-status
Port Error Status
Ethernet0 OK
Ethernet8 OK
Ethernet16 OK
Ethernet24 OK
Ethernet32 OK
Ethernet40 OK
Ethernet48 OK
Ethernet56 OK
Ethernet64 OK
Ethernet72 OK
Ethernet80 OK
Ethernet88 OK
Ethernet96 OK
Ethernet104 OK
Ethernet112 OK
Ethernet120 OK
Ethernet128 OK
Ethernet136 OK
Ethernet144 OK
Ethernet152 OK
Ethernet160 OK
Ethernet168 OK
Ethernet176 OK
Ethernet184 OK
Ethernet192 OK
Ethernet200 OK
Ethernet208 OK
Ethernet216 OK
`========================================================================================= warnings summary ==========================================================================================
../../usr/local/lib/python3.8/dist-packages/paramiko/transport.py:236
/usr/local/lib/python3.8/dist-packages/paramiko/transport.py:236: CryptographyDeprecationWarning: Blowfish has been deprecated
"class": algorithms.Blowfish,
-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
---------------------------------- generated xml file: /run_logs/APR03/platform_tests/sfp/test_sfputil.py::test_check_sfputil_error_status_2025-04-05-06-44-07.xml ----------------------------------
INFO:root:Can not get Allure report URL. Please check logs
-------------------------------------------------------------------------------------- live log sessionfinish ---------------------------------------------------------------------------------------
06:47:11 init.pytest_terminal_summary L0067 INFO | Can not get Allure report URL. Please check logs
============================================================================= 2 passed, 1 warning in 182.23s (0:03:02) ==============================================================================
sonic@sonic-ucs-m6-16:/data/tests$
`
Any platform specific information?
Platform: x86_64-8102_28fh_dpu_o-r0
HwSKU: Cisco-8102-28FH-DPU-O-T1
Supported testbed topology if it's a new test case?
Documentation