feat: support specific BP port information display for show interfaces#3926
Merged
cyw233 merged 1 commit intosonic-net:masterfrom Jun 23, 2025
Merged
Conversation
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
ecf7b49 to
432c03e
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
432c03e to
417538f
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
417538f to
44b5cb8
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
69a5cce to
bdf42f5
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
cyw233
commented
Jun 18, 2025
| assert result == intf_description_asic0_all | ||
|
|
||
| def test_invalid_asic_name(self): | ||
| def test_invalid_asic_name_all(self): |
Contributor
Author
There was a problem hiding this comment.
Duplicated method name with the one below
Collaborator
|
Cherry-pick PR to 202505: #3930 |
Contributor
Author
|
Cherry-pick PR to msft-202405: Azure/sonic-utilities.msft#191 |
nmoray
pushed a commit
to nmoray/sonic-utilities
that referenced
this pull request
Jun 25, 2025
YairRaviv
pushed a commit
to YairRaviv/sonic-utilities
that referenced
this pull request
Jan 12, 2026
cyw233
added a commit
to cyw233/sonic-utilities
that referenced
this pull request
Mar 2, 2026
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
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.
What I did
This PR resolves #3925.
In this PR, I enhanced the
show interfacescommand to support specific BP port information display for multi-ASIC devices, for example, T2 Chassis. It means we can use commands likeshow interfaces status Ethernet-BP128 -d allto get the status of a backplane interface, e.g. Ethernet-BP128 itself, after this PR, just like what we used to do for a specific frontend interface (plus an extra-d allof course).The commands I updated for getting specific BP interface information:
show interfaces descriptionshow interfaces statusshow interfaces tpidshow interfaces countersshow interfaces autoneg statusshow interfaces link-training statusshow interfaces fec statusPlease note that nothing has changed when running these commands on a single-ASIC device, and nothing has changed when running these commands on a multi-ASIC device without specifying an interface. The only behavior change is when we run the above commands for a specific interface on a multi-ASIC device.
How I did it
How to verify it
We can try the following commands after this PR to see the information of the backplane interface Ethernet-BP128 itself:
show interfaces description Ethernet-BP128 -d allshow interfaces status Ethernet-BP128 -d allshow interfaces tpid Ethernet-BP128 -d allshow interfaces counters -i Ethernet-BP128 -d allshow interfaces autoneg status Ethernet-BP128 -d allshow interfaces link-training status Ethernet-BP128 -d allshow interfaces fec status Ethernet-BP128 -d allPrevious command output (if the output of a command-line utility has changed)
Before this PR, when we run
show interfaces status Ethernet-BP128 -d all, the output will be empty:New command output (if the output of a command-line utility has changed)
After this PR, when we run
show interfaces status Ethernet-BP128 -d all, the output will be something like the following: