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

[Bridge/Vlan MAC] Add test to cover bridge mac and flooding #3916

Closed
prsunny opened this issue Jul 30, 2021 · 2 comments · Fixed by #4637
Closed

[Bridge/Vlan MAC] Add test to cover bridge mac and flooding #3916

prsunny opened this issue Jul 30, 2021 · 2 comments · Fixed by #4637
Assignees
Labels
Dual ToR Test Dev ♊ Action item for dual ToR test development P0 Test gap

Comments

@prsunny
Copy link
Contributor

prsunny commented Jul 30, 2021

Description

With the support to set Vlan mac, need a test to ensure there is no flooding of traffic for the packets destined to the configured Vlan IP address. This is to ensure Bridge in sonic is not incorrectly flooding the self-destined packets.

Steps to reproduce the issue:

  1. Send ICMP packets destined to Vlan IP

Describe the results you received:

Describe the results you expected:
No flooding of ICMP packets

Additional information you deem important:

**Output of `show version`:**

```
(paste your output here)
```

**Attach debug file `sudo generate_dump`:**

```
(paste your output here)
```
@lguohan
Copy link
Contributor

lguohan commented Aug 2, 2021

add a control plane test only first, still sonic-mgmt test with control plane verification only.

@yxieca yxieca added the P0 label Sep 9, 2021
@wangxin
Copy link
Collaborator

wangxin commented Sep 16, 2021

@lguohan Sending packets to Vlan IP is dataplane. What do you mean by add control plane test? Can you elaborate?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dual ToR Test Dev ♊ Action item for dual ToR test development P0 Test gap
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants