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
HLD for aggregate VOQ counters. #1587
base: master
Are you sure you want to change the base?
Conversation
@arlakshm @zhangyanzhao for awareness |
#### CHASSIS_APP_DB Changes | ||
|
||
The following new VOQ counters should be available for each VOQ entry in the DB: | ||
* `COUNTERS|fsi_id|asic_id|intf@fsi_id|asic_id:VOQ_index` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add an example here for an entry?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vivekverma-arista please share key as well as images are not getting loaded here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vmittal-msft you should be able to see the images when viewing the files. For example, see https://github.com/sonic-net/SONiC/blob/f4d03c5efb0c40fc3159c52d49918b1724b111b1/doc/voq/aggregate_voq_counters.md
community review recording https://zoom.us/rec/share/l3l_S2iJIdKTdsnAqEGTI-kEYsr5HL5C4EJfSC4Ct9FO5_2Y66QkFE3zhdz8cmgh.Qkk5g7TeFGFEz328 |
@vivekverma-arista can you please help to add the code PRs to this HLD by referring to #806 ? Thanks. |
@vivekverma-arista can you please help to update the code PR list by referring to #806 ? Thanks. |
This document provides general information about the feature aggregate VOQ counters in a distributed VOQ architecture.
This feature has been tracked in #1543