Skip to content

[action] [PR:17094] [dualtor-io] Improve dualtor-io utility#17240

Merged
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/17094
Mar 4, 2025
Merged

[action] [PR:17094] [dualtor-io] Improve dualtor-io utility#17240
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/17094

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
  • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

Two improvements:

  1. force replace the dst arp responder config.
  2. add show arp and neighbor check in I/O utility to help debug.

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

How did you do it?

As the motivation.

How did you verify/test it?

dualtor_io/test_tor_failure.py::test_active_tor_reboot_downstream[active-active] PASSED [100%]

============================================================================================================================== warnings summary ==============================================================================================================================

Any platform specific information?

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

Documentation

Two improvements:
1. force replace the dst arp responder config.
2. add show arp and neighbor check in I/O utility to help debug.

Signed-off-by: Longxiang Lyu <[email protected]>
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #17094

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit dfec007 into sonic-net:202411 Mar 4, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants