Skip to content

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

Merged
judyjoseph merged 1 commit into
Azure:202405from
arista-setu:arista-setu/msft-202405-backport-PR3819
Mar 18, 2026
Merged

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

Conversation

@arista-setu

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

Copy link
Copy Markdown

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 202405 branch.

What I did

PR3819 already address the fixes issue. This PR has already merged but it was not backported to 202405 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 for 202503 branch : #286

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
Copy Markdown
Author

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

@kenneth-arista

Copy link
Copy Markdown

@judyjoseph judyjoseph merged commit 2c9562f into Azure:202405 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