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

SAI NAT aging notification #987

Merged
merged 14 commits into from
Oct 6, 2022
Merged

SAI NAT aging notification #987

merged 14 commits into from
Oct 6, 2022

Conversation

arvbb
Copy link
Contributor

@arvbb arvbb commented Dec 20, 2021

The sairedis changes related to the SAI here: opencomputeproject/SAI#1365

@ghost
Copy link

ghost commented Dec 20, 2021

CLA assistant check
All CLA requirements met.

@arlakshm
Copy link
Contributor

@AkhileshSamineni , can you review this PR

@arlakshm
Copy link
Contributor

/Azp run Azure.sonic-sairedis

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kperumalbfn
Copy link

@arlakshm @AkhileshSamineni could you please review the PR.

@arlakshm
Copy link
Contributor

Can you please add Unit tests

@kcudnik
Copy link
Collaborator

kcudnik commented May 25, 2022

please also satisfy code coverage

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 18, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

@kcudnik
Copy link
Collaborator

kcudnik commented Aug 22, 2022

please fix build errors

kcudnik
kcudnik previously approved these changes Aug 31, 2022
@arvbb
Copy link
Contributor Author

arvbb commented Sep 1, 2022

Can you please add Unit tests

Hi @arlakshm, added,.. thanks

meta/Meta.cpp Show resolved Hide resolved
@kcudnik kcudnik merged commit fd9bc84 into sonic-net:master Oct 6, 2022
pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
The sairedis changes related to the SAI here: opencomputeproject/SAI#1365
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.

4 participants