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

[voq] SAI support for changing encap index in voq neighbor attribute programming (set function) #7879

Closed
vganesan-nokia opened this issue Jun 15, 2021 · 2 comments
Labels
Triaged this issue has been triaged

Comments

@vganesan-nokia
Copy link
Contributor

Description

For VOQ neighbors we program encap index for remote neighbor records. If the encap index is reallocated to a different encap index than previously allocated by the owner of the neighbor, at the remote side, the neighbor should be reprogrammed with changed voq encap index. Currently SAI (brcm SAI, for example) does not support change (set) of voq encap index for existing neighbors in neighbor entry set attribute function . As a work-around we handle the encap index attribute change by deleting the existing old neighbor record and re-adding it back with new encap index. To avoid this two steps change handling, we need SAI support for voq encap index attribute setting in neighbor entry attribute programming (set operation) (similar to MAC change programming)

Steps to reproduce the issue:

Please refer issue #7451 for reproducing a problem whose fix (sonic-net/sonic-swss#1729) will have this double action change handling.

Also refer to BRCM SAI neighbor entry set attribute function for sample.

Describe the results you received:

Refer the fix for this issue in PR sonic-net/sonic-swss#1729

Describe the results you expected:

The SAI neighbor entry set attribute API should handle voq encap index setting similar to MAC address setting

Additional information you deem important (e.g. issue happens only occasionally):

This issue is raised as response to a review comment to PR sonic-net/sonic-swss#1729

@zhangyanzhao
Copy link
Collaborator

For tracking purpose. @abdosi will take care

@zhangyanzhao zhangyanzhao added the Triaged this issue has been triaged label Jun 23, 2021
@vganesan-nokia
Copy link
Contributor Author

Fixed in SAI release 5.0.0.6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Triaged this issue has been triaged
Projects
None yet
Development

No branches or pull requests

2 participants