Skip to content

[action] [PR:18170] [testARPCompleted] Cleanup ptf ip after test failure#18175

Merged
mssonicbld merged 1 commit into
sonic-net:202411from
mssonicbld:cherry/202411/18170
Apr 29, 2025
Merged

[action] [PR:18170] [testARPCompleted] Cleanup ptf ip after test failure#18175
mssonicbld merged 1 commit into
sonic-net:202411from
mssonicbld:cherry/202411/18170

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. ensure the testARPCompleted cleanup the ptf/dut ip if the test fails.
  2. use reload to restore the DUT.

Signed-off-by: Longxiang Lyu lolv@microsoft.com

How did you do it?

As the motivation.

How did you verify/test it?

Run on dualtor/t0 testbed and pass.

Any platform specific information?

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

Documentation

* [testARPCompleted] Cleanup ptf ip after test failure

Signed-off-by: Longxiang Lyu <lolv@microsoft.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator Author

Original PR: #18170

@mssonicbld

Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit de454d5 into sonic-net:202411 Apr 29, 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