Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Revert "[voq][chassis]Add show fabric counters port/queue com… #2652

Merged

Conversation

jfeng-arista
Copy link
Contributor

@jfeng-arista jfeng-arista commented Feb 2, 2023

…mands (#2522)" (#2611)"

This reverts commit b6bbc3e.
Reverts https://github.com/sonic-net/sonic-swss/pull/](https://github.com/sonic-net/sonic-swss/pull/2611)

Before merging this revert, the Broadcom SAI needs to be updated to support fabric counters.

@jfeng-arista jfeng-arista requested a review from prsunny as a code owner February 2, 2023 23:12
@prsunny prsunny requested a review from arlakshm February 2, 2023 23:17
@kenneth-arista
Copy link
Contributor

We need this revert in 202205 for chassis. The Broadcom SAI used in 202205 already has the corresponding changes to enable fabric counters.

@rlhui
Copy link
Contributor

rlhui commented Feb 3, 2023

waiting for Brcm SAI 8.1.x or SAI 8.x to be in master branch

@arlakshm
Copy link
Contributor

arlakshm commented Feb 4, 2023

Dont merge till fix is available in SAI 8.x

@kenneth-arista
Copy link
Contributor

I see the fix has landed in Brcm's SAI 8.4 release.

@arlakshm arlakshm merged commit 28ffc3a into sonic-net:master Jun 16, 2023
@kenneth-arista
Copy link
Contributor

@arlakshm the master branch is still pointing at 7.1.x and thus the fix is not available yet.

theasianpianist pushed a commit to theasianpianist/sonic-swss that referenced this pull request Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants