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

[FC] Policer counter #3491

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

Conversation

shiraez
Copy link

@shiraez shiraez commented Feb 2, 2025

What I did
Added the implantation for policer counter
Added mechanisms to collect and expose counters for various policing actions, such as packets dropped or marked.
Orchagent Updates: Enhanced Orchagent to manage the new policer counters and ensure their correct synchronization with SAI.
Unit Tests: Included unit tests to validate the correctness of the policer counter.

Why I did it
Simplifies debugging of policy enforcement issues by allowing real-time counter analysis.

How I verified it

Details if related

@shiraez shiraez requested a review from prsunny as a code owner February 2, 2025 21:06
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Pull request contains merge conflicts.

@shiraez shiraez changed the title Policer counter [FC] Policer counter Feb 2, 2025
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

linux-foundation-easycla bot commented Feb 17, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

Copy link

Pull request contains merge conflicts.

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny
Copy link
Collaborator

prsunny commented Feb 18, 2025

Can you please update the description with HLD link?

@prsunny
Copy link
Collaborator

prsunny commented Feb 18, 2025

Also, please update for any dependent PR that should be merged first, before this PR

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.

3 participants