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

[enable counters] enable RIF flex counter by default #4655

Merged
merged 1 commit into from
Jun 4, 2020

Conversation

mykolaf
Copy link
Collaborator

@mykolaf mykolaf commented May 27, 2020

Signed-off-by: Mykola Faryma mykolaf@mellanox.com

- Why I did it
We need RIF counters to be enabled by default. Flex Counter does probe for supported counters. If a platform does not support RIF counters, SAI will return NOT_SUPPORTED and Flex Counter will stop polling the counter.
- How I did it

- How to verify it
After fresh install rif counter gropup is enabled by default:

$ counterpoll show
Type                  Interval (in ms)    Status
--------------------  ------------------  --------
QUEUE_STAT            default (10000)     enable
PORT_STAT             default (1000)      enable
RIF_STAT              default (1000)      enable
QUEUE_WATERMARK_STAT  default (10000)     enable
PG_WATERMARK_STAT     default (10000)     enable

- Description for the changelog

enable RIF flex counter group by default

- A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Mykola Faryma <mykolaf@mellanox.com>
@lguohan
Copy link
Collaborator

lguohan commented May 31, 2020

If a platform does not support RIF counters, SAI will return NOT_SUPPORTED and Flex Counter will stop polling the counter.

in this case, will be be better if the status shows "not supported"?

$ counterpoll show
Type                  Interval (in ms)    Status
--------------------  ------------------  --------
QUEUE_STAT            default (10000)     enable
PORT_STAT             default (1000)      enable
RIF_STAT              default (1000)      not supported
QUEUE_WATERMARK_STAT  default (10000)     enable
PG_WATERMARK_STAT     default (10000)     enable

@mykolaf
Copy link
Collaborator Author

mykolaf commented Jun 2, 2020

If a platform does not support RIF counters, SAI will return NOT_SUPPORTED and Flex Counter will stop polling the counter.

in this case, will be be better if the status shows "not supported"?

The Flex Counter does not signal which counter is supported/not supported back to DB. It only produces warning log. I don't think we can implement something like this without a "system capabilities" mechanism in place.

@lguohan lguohan merged commit 49a9374 into sonic-net:master Jun 4, 2020
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.

2 participants