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

[AclOrch] Custom ACL Based Metering #3484

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

Conversation

shaygol
Copy link

@shaygol shaygol commented Jan 26, 2025

What I did
Added a mechanism integrating policers with ACL rules to provide granular traffic management.

Why I did it
To allow customers to enforce advanced traffic management policies.

How I verified it

  • Python & C++ unit tests to validate the configuration flow.

Details if related

HLD link: Custom ACL Based Metering

@shaygol shaygol requested a review from prsunny as a code owner January 26, 2025 22:35
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

linux-foundation-easycla bot commented Jan 26, 2025

CLA Missing ID CLA Not Signed

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny
Copy link
Collaborator

prsunny commented Jan 28, 2025

Please provide the HLD link in PR description

@prsunny
Copy link
Collaborator

prsunny commented Jan 30, 2025

@shaygol , could you please rebase?

@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).

shaygol and others added 3 commits February 6, 2025 18:13
- New 'AclRule' subclass: 'AclRulePolicer'
- ACL rule table schema update
- Unit Tests
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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