Skip to content

[dualtor-io] Improve dualtor-io utility#17094

Merged
yxieca merged 1 commit intosonic-net:masterfrom
lolyu:improve_dualtor_io
Feb 21, 2025
Merged

[dualtor-io] Improve dualtor-io utility#17094
yxieca merged 1 commit intosonic-net:masterfrom
lolyu:improve_dualtor_io

Conversation

@lolyu
Copy link
Copy Markdown
Collaborator

@lolyu lolyu commented Feb 21, 2025

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 lolv@microsoft.com

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 <lolv@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@lolyu lolyu requested review from bingwang-ms and yxieca February 21, 2025 09:13
@yxieca yxieca merged commit 5cf2a1b into sonic-net:master Feb 21, 2025
17 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Feb 28, 2025
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 <lolv@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202411: #17240

mssonicbld pushed a commit that referenced this pull request Mar 4, 2025
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 <lolv@microsoft.com>
nnelluri-cisco pushed a commit to nnelluri-cisco/sonic-mgmt that referenced this pull request Mar 15, 2025
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 <lolv@microsoft.com>
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.

3 participants