Skip to content

[202503] : Backport PR3819 to add multi-asic support for show interface counters fec-histogram CLI command#286

Merged
judyjoseph merged 1 commit intoAzure:202503from
arista-setu:arista-setu/msft-202503-backport-PR3819
Mar 18, 2026
Merged

[202503] : Backport PR3819 to add multi-asic support for show interface counters fec-histogram CLI command#286
judyjoseph merged 1 commit intoAzure:202503from
arista-setu:arista-setu/msft-202503-backport-PR3819

Conversation

@arista-setu
Copy link

@arista-setu arista-setu commented Feb 18, 2026

Currently, show fec histogram CLI command provides option for namespace but is not using it. This is why the CLI command aborts and does not return any meaningful results on 202503.

What I did

PR3819 already fixes the underlying issue. This PR has already merged but it was not backported to 202503 branch

In this pull request, I have backported changes from PR3819 to fix the issue on multi-asic systems.

I've also opened another PR to backport the same changes to 202405 branch : #287

How to verify it

Testing command returns expected output.

Previous command output (if the output of a command-line utility has changed)

(switch)~# show interface counters fec-histogram -n asic0 Ethernet0
Port Ethernet0 not found in COUNTERS_PORT_NAME_MAP
Aborted!

New command output (if the output of a command-line utility has changed)

With the changes from this PR, the CLI command return following expected output

(switch):~$ show interfaces counters fec-histogram -n asic0 Ethernet0
Symbol Errors Per Codeword      Codewords
----------------------------  -----------
BIN0                                    0
BIN1                                    0
BIN2                                    0
BIN3                                    0
BIN4                                    0
BIN5                                    0
BIN6                                    0
BIN7                                    0
BIN8                                    0
BIN9                                    0
BIN10                                   0
BIN11                                   0
BIN12                                   0
BIN13                                   0
BIN14                                   0
BIN15                                   0

@arista-setu
Copy link
Author

@microsoft-github-policy-service agree company="Arista"

@kenneth-arista
Copy link

Copy link
Contributor

@judyjoseph judyjoseph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@judyjoseph judyjoseph merged commit 2677602 into Azure:202503 Mar 18, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants