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

[fdb]cli: fdb entries are cleared according to vlan or port or vlan&&port #657

Merged
merged 1 commit into from
May 24, 2021

Conversation

yangshp1987
Copy link
Contributor

Fdb entries can be deleted with user specifying vlan or port or vlan&&port.
fdborch is in charge of deleting fdb entries in another issues#1064,and it had pull request.

- What I did

- How I did it

- How to verify it

- Previous command output (if the output of a command-line utility has changed)

- New command output (if the output of a command-line utility has changed)

-->

@prsunny
Copy link
Contributor

prsunny commented Sep 19, 2019

Can you provide the output of the command before/after the fix in description?

@yangshp1987
Copy link
Contributor Author

yangshp1987 commented Sep 23, 2019

Can you provide the output of the command before/after the fix in description?

OK,I will provide it later.
@prsunny

the output of the command **before** the fix:

root@sonic:/home/admin# bcmcmd "l2 show"
l2 show
mac=00:8c:fa:c7:ae:b8 vlan=1 GPORT=0x0 modid=0 port=0/cpu0 Static CPU
mac=00:00:00:00:00:01 vlan=1001 GPORT=0x7 modid=0 port=7/ce6 ReplacePriority
drivshell>
root@sonic:/home/admin# fdbclear -p Ethernet24 -v Vlan1001
command not supported yet.
root@sonic:/home/admin# bcmcmd "l2 show"
l2 show
mac=00:8c:fa:c7:ae:b8 vlan=1 GPORT=0x0 modid=0 port=0/cpu0 Static CPU
mac=00:00:00:00:00:01 vlan=1001 GPORT=0x7 modid=0 port=7/ce6 ReplacePriority

the output of the command **after** the fix:

root@sonic:/home/admin# cp /usr/bin/fdbclear.new /usr/bin/fdbclear
root@sonic:/home/admin# fdbclear -p Ethernet24 -v Vlan1001
Port Ethernet24 + VlanVlan1001 FDB entries are cleared.
root@sonic:/home/admin# bcmcmd "l2 show"
l2 show
mac=00:8c:fa:c7:ae:b8 vlan=1 GPORT=0x0 modid=0 port=0/cpu0 Static CPU
drivshell>

@yangshp1987
Copy link
Contributor Author

@prsunny Could you please review,thanks.

@prsunny prsunny merged commit 7744c8d into sonic-net:master May 24, 2021
gitsabari pushed a commit to gitsabari/sonic-utilities that referenced this pull request Jun 15, 2021
…port (sonic-net#657)

Fdb entries can be deleted with user specifying vlan or port or vlan&&port.
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-utilities that referenced this pull request Aug 10, 2021
…port (sonic-net#657)

Fdb entries can be deleted with user specifying vlan or port or vlan&&port.
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.

2 participants