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] Support Policer Counter #1484

Merged
merged 8 commits into from
Feb 5, 2025
Merged

Conversation

shiraez
Copy link
Contributor

@shiraez shiraez commented Dec 12, 2024

Added the implantation for policer counter -
Support in POLICER group and sai_serialize functions
Unit Tests: Included unit tests to add and remove policer counter.

@shiraez shiraez marked this pull request as draft December 12, 2024 12:28
@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).

@shiraez shiraez marked this pull request as ready for review December 18, 2024 16:36
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kcudnik
Copy link
Collaborator

kcudnik commented Dec 19, 2024

seems like build is failing because not all consts are defined in swss-common

@shiraez
Copy link
Contributor Author

shiraez commented Dec 19, 2024

@kcudnik
Yes, it has not been merged yet.
sonic-net/sonic-swss-common#959

@kcudnik
Copy link
Collaborator

kcudnik commented Dec 22, 2024

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

@shiraez
Copy link
Contributor Author

shiraez commented Feb 5, 2025

Hi @kcudnik,
My changes in SWSS (PR #3491) depend on this.
Would it be possible to merge the changes?

@kcudnik kcudnik merged commit e11a959 into sonic-net:master Feb 5, 2025
12 checks passed
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202411: #1533

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants