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

Make the counter polling interval accurate by setting bulk counter poll chunk size per group or per counter #1864

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

stephenxs
Copy link
Collaborator

@stephenxs stephenxs commented Dec 10, 2024

Make counter-polling interval more accurate by

  1. setting bulk chunk size for a counter or per counter
  2. collect timestamp in sairedis instead of in the Lua plugin

Implementation PRs

Module PR title state
sonic-swss-common Add field for bulk chunk size in flex counter GitHub issue/pull request detail
sonic-sairedis Optimize counter polling interval by making it more accurate GitHub issue/pull request detail
sonic-swss Optimize counter polling interval by making it more accurate GitHub issue/pull request detail
sonic-buildimage Support yang model for bulk size per counter GitHub issue/pull request detail

Signed-off-by: Stephen Sun [email protected]

@stephenxs stephenxs changed the title Bulk counter poll chunk size Make the counter polling interval accurate by setting Bulk counter poll chunk size Dec 10, 2024
@stephenxs stephenxs changed the title Make the counter polling interval accurate by setting Bulk counter poll chunk size Make the counter polling interval accurate by setting bulk counter poll chunk size per group or per counter Dec 10, 2024
Signed-off-by: Stephen Sun <[email protected]>
@stephenxs stephenxs marked this pull request as ready for review December 11, 2024 07:58
Signed-off-by: Stephen Sun <[email protected]>
@stephenxs stephenxs requested a review from kcudnik December 14, 2024 00:44
Signed-off-by: Stephen Sun <[email protected]>
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

No pipelines are associated with this pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🏗 In Progress
Development

Successfully merging this pull request may close these issues.

4 participants