filterd backplane ports from STATE_DB while checking 'sudo sfputil show error-status'#17859
Merged
bingwang-ms merged 1 commit intosonic-net:masterfrom Jun 4, 2025
Conversation
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Contributor
Author
|
@rameshraghupathy and @prabhataravind |
rameshraghupathy
approved these changes
Apr 7, 2025
Contributor
|
lgtm, @prgeor please review/signoff. |
prabhataravind
approved these changes
May 8, 2025
mssonicbld
pushed a commit
to mssonicbld/sonic-mgmt
that referenced
this pull request
Jul 29, 2025
Collaborator
|
Cherry-pick PR to 202505: #19881 |
8 tasks
mssonicbld
pushed a commit
that referenced
this pull request
Jul 29, 2025
mssonicbld
added
Included in 202505 branch
and removed
Created PR to 202505 branch
labels
mssonicbld
pushed a commit
to mssonicbld/sonic-mgmt
that referenced
this pull request
Jul 30, 2025
Collaborator
|
Cherry-pick PR to 202411: #19937 |
8 tasks
mssonicbld
pushed a commit
that referenced
this pull request
Jul 30, 2025
mssonicbld
added
Included in 202411 branch
and removed
Created PR to 202411 branch
labels
wangxin
pushed a commit
to wangxin/sonic-mgmt
that referenced
this pull request
Oct 27, 2025
Code sync sonic-net/sonic-mgmt:202411 => 202412 ``` * c3fd4c0 (HEAD -> code-sync-202412, origin/code-sync-202412) r12f 250802:1704 - Merge remote-tracking branch 'base/202411' into code-sync-202412 |\ | * a7a638a (base/202411) nnelluri-cisco 250722:1131 - cherry pick for the PR#17101 (sonic-net#19748) | * fd4ae21 siva-prasad-cisco 250731:0002 - Ignore log analyzer error message related to source port GID mapping in kernel (sonic-net#19879) | * f986cfa Xu Chen 250730:2120 - Fix saithrift URL generation for Mellanox on internal-202411 branch sonic-net#19899 | * a07fb04 nnelluri-cisco 250604:1347 - filterd backplane ports while checking 'sudo sfputil show error-status' (sonic-net#17859) | * d707fcf nnelluri-cisco 250729:1208 - Added DPC role support for SmartSwitch. (sonic-net#18157) | * 80ea05a dypet 250414:1059 - Fix BFD status check and ipv6 PTF intermittent issue. (sonic-net#17819) | * fc623da Xu Chen 250725:1036 - support dynamic saithrift URL generation (sonic-net#19528) | * 0e5fc8a Mai Bui 250724:1807 - [202411] Ignore auditd error in loganalyzer (sonic-net#19767) | * d03770d Hua Liu 250218:1409 - [TACACS] Increase tacacs server reachability check timeout (sonic-net#16971) | * e8111ac nnelluri-cisco 250717:1033 - Generate new npu-dpu ports Ethernet224 To Ethernet280 for cisco smartswicth "Cisco-8102-28FH-DPU-O" (sonic-net#19257) | * e0060fa Zain Budhwani 250716:1109 - Remove yang_validate option from 202411 (sonic-net#19674) | * 89c4281 AharonMalkin 250715:2042 - Add Nvidia to supported platforms on intf_fec test (sonic-net#19398) | * f35fc20 mssonicbld 250716:0129 - Fix buffer queue cnt test to account for different BUFFER_QUEUE configs (sonic-net#19310) (sonic-net#19466) | * 3e0cabd Chuan Wu 250704:0327 - Update vlan ping test to override the affection of secondary vlan ip (sonic-net#18867) | * ff9d112 Longxiang Lyu 250714:1508 - [dualtor-io] Fix `test_active_link_admin_down_config_reload_link_up_downstream_standby` on Cisco/MLNX (sonic-net#18829) (sonic-net#19511) | * e870d66 Yawen 250714:1246 - [manual cherry-pick] add topo dualtor-aa-64-breakout to qos_sai_base.py (sonic-net#19543) | * 8369780 prabhataravind 250711:0929 - Enable skipped BFD and VxLAN tests on smartswitch T1 (sonic-net#19498) ```
opcoder0
pushed a commit
to opcoder0/sonic-mgmt
that referenced
this pull request
Dec 8, 2025
…s' (sonic-net#17859) Signed-off-by: opcoder0 <[email protected]>
AharonMalkin
pushed a commit
to AharonMalkin/sonic-mgmt
that referenced
this pull request
Dec 16, 2025
…s' (sonic-net#17859) Signed-off-by: Aharon Malkin <[email protected]>
gshemesh2
pushed a commit
to gshemesh2/sonic-mgmt
that referenced
this pull request
Dec 21, 2025
…s' (sonic-net#17859) Signed-off-by: Guy Shemesh <[email protected]>
venu-nexthop
pushed a commit
to venu-nexthop/sonic-mgmt
that referenced
this pull request
Jan 13, 2026
gshemesh2
pushed a commit
to gshemesh2/sonic-mgmt
that referenced
this pull request
Jan 26, 2026
…s' (sonic-net#17859) Signed-off-by: Guy Shemesh <[email protected]>
ytzur1
pushed a commit
to ytzur1/sonic-mgmt
that referenced
this pull request
Feb 2, 2026
…s' (sonic-net#17859) Signed-off-by: Yael Tzur <[email protected]>
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
8 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