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

[dualtor] Enhance IPinIP tunnel traffic distribution util. #3045

Merged
merged 2 commits into from
Mar 1, 2021

Conversation

bingwang-ms
Copy link
Collaborator

@bingwang-ms bingwang-ms commented Feb 26, 2021

Signed-off-by: bingwang [email protected]

Description of PR

Summary:
Fixes # (issue)
This PR is to enhance IPinIP tunnel traffic distribution util.

The verification is spearated into two steps:
Step 1. Verify traffic is not forwarded to server from standby ToR directly
Strp 2. Verify traffic is distributed evenly among all T1s.

Type of change

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

Approach

What is the motivation for this PR?

This PR is to enhance IPinIP tunnel traffic distribution util.

How did you do it?

Check the ttl of packet captured in server(ptf).
For packets arrived from standby ToR -> IPinIP tunnel -> active ToR -> server, the ttl should be decreaded by 2;
For packets arrived from standby ToR -> server (Negative scenario), the ttl should be decreaded by 1.
This feature is leveraged to verify packets are not forwarded from standby ToR to server directly.

How did you verify/test it?

Verified on dualtor testbed.

Any platform specific information?

No.

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

dualtor testbed specific.

Documentation

No.

The ptf script for checking IpinIP tunnel traffic balance is enhanced.
The verification is spearated into two steps:
Step 1. Verify traffic is not forwarded to server form standby ToR
directly
Strp 2. Verify traffic is distributed evenly among all T1s.

Signed-off-by: bingwang <[email protected]>
Signed-off-by: bingwang <[email protected]>
@bingwang-ms bingwang-ms requested a review from a team as a code owner February 26, 2021 09:39
@bingwang-ms
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms bingwang-ms merged commit 1b4d0d8 into sonic-net:master Mar 1, 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.

3 participants