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

Add test to verify IPinIP packet targeting at vnet route is decapsulated for T1 #15325

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

Conversation

xwjiang-ms
Copy link
Contributor

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

In Azure network, Netscan uses IP-decap to check for route blackholes, but servers do not support decapsulation, making it impossible to perform server-to-server checks. To address this, we developed the subnet decap feature to detect if the link between the server and T0 is functioning properly. Additionally, new test cases are required to ensure the functionality of subnet decap is working as expected.
Sever to T0 test was added in #14720, we need to add server to T1 test

How did you do it?

Add test to verify IPinIP packet targeting at vnet route is decapsulated for T1
Ref test gap issue: #13122
Ref test plan: #12990

How did you verify/test it?

Any platform specific information?

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

Documentation

@xwjiang-ms xwjiang-ms marked this pull request as draft November 1, 2024 07:19
@xwjiang-ms
Copy link
Contributor Author

Currently subnet decap feature could only run on Broadcom td3 hwskus, but Broadcom td3 hwskus doesn't support vnet route with provided APIs
Created issue #15259 to track

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.

1 participant