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_ipinip] Strip ip prefix #3140

Merged
merged 1 commit into from
Mar 15, 2021
Merged

[test_ipinip] Strip ip prefix #3140

merged 1 commit into from
Mar 15, 2021

Conversation

lolyu
Copy link
Contributor

@lolyu lolyu commented Mar 15, 2021

Description of PR

Strip the ip prefix in the ip address passed to simple_ip_packet

Signed-off-by: Longxiang Lyu [email protected]

Summary:
Fixes # (issue)

Type of change

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

Approach

What is the motivation for this PR?

Strip the ip prefix in the ip address passed to simple_ip_packet

How did you do it?

Strip the prefix.

How did you verify/test it?

dualtor/test_ipinip.py::test_decap_active_tor PASSED                                                                                                                                           [ 50%]
dualtor/test_ipinip.py::test_decap_standby_tor FAILED                                                                                                                                          [100%]

Any platform specific information?

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

Documentation

Signed-off-by: Longxiang Lyu <[email protected]>
@lolyu lolyu requested a review from a team as a code owner March 15, 2021 02:56
@lolyu lolyu merged commit 6335756 into sonic-net:master Mar 15, 2021
@lolyu lolyu deleted the fix_ipinip branch March 15, 2021 06:01
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.

2 participants