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

[test_fdb] Only check dummy_mac when checking fdb clear #4093

Merged
merged 1 commit into from
Aug 24, 2021

Conversation

bingwang-ms
Copy link
Collaborator

@bingwang-ms bingwang-ms commented Aug 20, 2021

Signed-off-by: bingwang [email protected]

Description of PR

Summary:
Fixes #3934

This PR is to focus on only dummy mac when checking if FDB table is cleared completely.

When SONiC is configured to L2 mode, VMs will keep sending packets to L2 interfaces, which will populate FDB on DUT and result in test failure. This PR is to address the issue.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911

Approach

What is the motivation for this PR?

This PR is to fix #3934

How did you do it?

Focus on dummy mac when checking if FDB table is cleared completely.

How did you verify/test it?

Verified on SN4600

  1. Config device to L2 switch
  2. Run test_fdb, and confirm test can pass now.

Any platform specific information?

No.

Supported testbed topology if it's a new test case?

No.

Documentation

@bingwang-ms bingwang-ms requested a review from a team as a code owner August 20, 2021 08:59
@bingwang-ms bingwang-ms changed the title [test_fdb] Filter out dummy mac for checking fdb clear [test_fdb] Filter out unexpected FDB entry when checking fdb clear Aug 20, 2021
@bingwang-ms bingwang-ms changed the title [test_fdb] Filter out unexpected FDB entry when checking fdb clear [test_fdb] Focus on dummy_mac when checking fdb clear Aug 20, 2021
@bingwang-ms bingwang-ms changed the title [test_fdb] Focus on dummy_mac when checking fdb clear [test_fdb] Only check dummy_mac when checking fdb clear Aug 20, 2021
@bingwang-ms bingwang-ms merged commit ceaa97f into sonic-net:master Aug 24, 2021
vmittal-msft pushed a commit to vmittal-msft/sonic-mgmt that referenced this pull request Sep 28, 2021
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.

FDB tests need to be modified in order to pass on setup with L2 configuration
2 participants