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

[vlanmgr]Fix modify VLAN mac Bridge mac also be modified. #2581

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

maggie2202
Copy link

@maggie2202 maggie2202 commented Dec 20, 2022

…ied, the mac address of bridge is also modified.

Signed-off-by: liting liting@asterfusion.com

How I verified it
Step 1: Add the MAC address "00:00:00:00:11" for Vlan 10 in CONFIG_DB as follows,
admin@sonic:~$ redis-cli -n 4 hgetall "VLAN|Vlan10"

  1. "vlanid"
  2. "10"
  3. "mac"
  4. "00:00:00:00:00:11"

Step 2: Execute "ip link" command,Bridge's MAC address also being modified to "00:00:00:00:11" as follows,
60: Bridge: <BROADCAST,MULTICAST,ALLMULTI,UP,LOWER_UP> mtu 9216 qdisc noqueue state UP mode DEFAULT group default qlen 1000
link/ether 00:00:00:00:00:11 brd ff:ff:ff:ff:ff:ff
and Information about Vlan10:
74: Vlan10@Bridge: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 9216 qdisc noqueue state UP mode DEFAULT group default qlen 1000
link/ether 00:00:00:00:00:11 brd ff:ff:ff:ff:ff:ff

…ied, the mac address of bridge is also modified.

Signed-off-by: liting <liting@asterfusion.com>
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Dec 20, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

@maggie2202 maggie2202 closed this Dec 22, 2022
@maggie2202 maggie2202 reopened this Dec 22, 2022
@maggie2202 maggie2202 changed the title [vlanmgr]Solve the problem that When the mac address of vlan is modif… [vlanmgr]Fix modify VLAN mac Bridge mac also be modified. Dec 23, 2022
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.

None yet

1 participant