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

[DASH] Add support for ENI counters #3496

Merged
merged 6 commits into from
Sep 18, 2024
Merged

Conversation

vivekrnv
Copy link
Contributor

@vivekrnv vivekrnv commented Aug 23, 2024

What I did

Updated counterpoll cli to support configuration for ENI counters

root@dpu:/home/admin# counterpoll eni enable

root@dpu:/home/admin# counterpoll eni interval 1000 

root@dpu:/home/admin# counterpoll show
Type                        Interval (in ms)    Status
--------------------------  ------------------  --------
QUEUE_STAT                  default (10000)     enable
PORT_STAT                   default (1000)      enable
PORT_BUFFER_DROP            default (60000)     enable
RIF_STAT                    default (1000)      enable
QUEUE_WATERMARK_STAT        default (60000)     enable
PG_WATERMARK_STAT           default (60000)     enable
PG_DROP_STAT                default (10000)     enable
BUFFER_POOL_WATERMARK_STAT  default (60000)     enable
ACL                         10000               enable
ENI_STAT                    1000                enable 

How I did it

How to verify it

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

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

Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
@prsunny
Copy link
Contributor

prsunny commented Sep 10, 2024

Can you please provide the command output in the description?

@vivekrnv
Copy link
Contributor Author

Can you please provide the command output in the description?

Updated

@qiluo-msft qiluo-msft merged commit 867fc54 into sonic-net:master Sep 18, 2024
7 checks passed
@vivekrnv vivekrnv deleted the eni_cntrs branch November 9, 2024 00:45
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.

4 participants