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

[mlag]Fix mlag fdb not remove in kernel when remote fdb entry removed #1924

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

Conversation

pettershao-ragilenetworks
Copy link
Contributor

What I did
Fix mlag fdb not remove in kernel when remote fdb entry removed issue
Why I did it
Kernel fdb table entry residue
How I verified it

before modify, Kernel fdb table entry residue:

root@TOR2:/home/admin# bridge fdb show | grep 1a:00  -->>kernel Has 1a:00 entry
00:00:01:01:1a:00 dev PortChannel16 vlan 103 master Bridge static
00:00:01:00:1a:00 dev PortChannel16 vlan 101 master Bridge static
root@TOR2:/home/admin# show mac   -->>but asic db has removed already
  No.    Vlan  MacAddress         Port           Type
-----  ------  -----------------  -------------  ------
    1     101  00:00:01:00:10:12  PortChannel16  Static
    2     102  00:00:01:00:10:22  PortChannel16  Static
Total number of entries 2
root@TOR2:/home/admin# redis-cli -n 6   -->>but state db has removed already
127.0.0.1:6379[6]> keys "*FDB*"
1) "MCLAG_REMOTE_FDB_TABLE|Vlan102:00:00:01:00:10:22"
2) "MCLAG_REMOTE_FDB_TABLE|Vlan101:00:00:01:00:10:12"

after modify, every thing seems fine:

root@TOR2:/home/admin# bridge fdb show | grep 1a:00  -->>kernel Has not 1a:00 entry
root@TOR2:/home/admin# show mac   -->>asic db has removed
  No.    Vlan  MacAddress         Port           Type
-----  ------  -----------------  -------------  ------
    1     101  00:00:01:00:10:12  PortChannel16  Static
    2     102  00:00:01:00:10:22  PortChannel16  Static
Total number of entries 2
root@TOR2:/home/admin# redis-cli -n 6   -->>state db has removed 
127.0.0.1:6379[6]> keys "*FDB*"
1) "MCLAG_REMOTE_FDB_TABLE|Vlan102:00:00:01:00:10:22"
2) "MCLAG_REMOTE_FDB_TABLE|Vlan101:00:00:01:00:10:12"

Details if related

@pettershao-ragilenetworks
Copy link
Contributor Author

@Praveen-Brcm @qiluo-msft help forward this, thanks!

@lguohan
Copy link
Contributor

lguohan commented Oct 13, 2021

where is the vstest?

Copy link
Contributor

@lguohan lguohan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where is the vstest?

Signed-off-by: pettershao-ragilenetworks <[email protected]>
Signed-off-by: pettershao-ragilenetworks <[email protected]>
@pettershao-ragilenetworks
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 1924 in repo Azure/sonic-swss

EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
*Increase test timeout to 240mts from 180mts
Signed-off-by: Sudharsan Dhamal Gopalarathnam <[email protected]>
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.

2 participants