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

[orchagent]: Handle additional SAI error conditions gracefully #2755

Merged

Conversation

prabhataravind
Copy link
Contributor

  • Handle SAI_STATUS_TABLE_FULL errors for routes
  • Handle SAI_STATUS_ITEM_ALREADY_EXISTS, SAI_STATUS_ITEM_NOT_FOUND and SAI_STATUS_TABLE_FULL errors for nexthops, nexthop groups and neighbors

What I did

Why I did it

How I verified it

Details if related

 * Handle SAI_STATUS_TABLE_FULL errors for routes
 * Handle SAI_STATUS_ITEM_ALREADY_EXISTS, SAI_STATUS_ITEM_NOT_FOUND and
   SAI_STATUS_TABLE_FULL errors for nexthops, nexthop groups and neighbors

Signed-off-by: Prabhat Aravind <paravind@microsoft.com>
@prabhataravind prabhataravind marked this pull request as ready for review May 3, 2023 02:22
@prabhataravind prabhataravind merged commit 6cbf7a0 into sonic-net:master May 3, 2023
yxieca pushed a commit that referenced this pull request May 11, 2023
* Handle SAI_STATUS_TABLE_FULL errors for routes
 * Handle SAI_STATUS_ITEM_ALREADY_EXISTS, SAI_STATUS_ITEM_NOT_FOUND and
   SAI_STATUS_TABLE_FULL errors for nexthops, nexthop groups and neighbors

Signed-off-by: Prabhat Aravind <paravind@microsoft.com>
@puffc
Copy link
Contributor

puffc commented Aug 10, 2023

@yxieca Would you please push this commit to 202211 branch? We're experiencing the duplicate routes problem in 202211. Thank you!

@yxieca
Copy link
Contributor

yxieca commented Aug 10, 2023

@prabhataravind can you help create a PR for 202211 branch?

yxieca pushed a commit that referenced this pull request Aug 29, 2023
* Handle SAI_STATUS_TABLE_FULL errors for routes
 * Handle SAI_STATUS_ITEM_ALREADY_EXISTS, SAI_STATUS_ITEM_NOT_FOUND and
   SAI_STATUS_TABLE_FULL errors for nexthops, nexthop groups and neighbors

Signed-off-by: Prabhat Aravind <paravind@microsoft.com>
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.

4 participants