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

T2-Snappi-route-conv: Add Ungraceful restart testcase #16636

Conversation

deepak-singhal0408
Copy link
Contributor

Description of PR

Summary:
T2-Route-Convergence: Add Ungraceful restart(Sup/Uplink LC/Downlink LC) testcase

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?

New test case to verify route convergence in ungraceful reset scenarios

How did you do it?

Reset DUT by creating kernel panic, and measure traffic loss

How did you verify/test it?

Ran the test on T2 DUTs

Any platform specific information?

NA

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

T2

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

@deepak-singhal0408 deepak-singhal0408 self-assigned this Jan 22, 2025
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@deepak-singhal0408
Copy link
Contributor Author

deepak-singhal0408 commented Jan 22, 2025

Created a new PR by taking changes from #16359 (this PR will be abandoned) and fixed a few issues and adjusted code as per new infra..

@rlhui rlhui removed request for abdosi and YatishSVC January 22, 2025 19:24
@rlhui rlhui merged commit 4e26f31 into sonic-net:master Jan 22, 2025
12 of 13 checks passed
@mssonicbld
Copy link
Collaborator

@deepak-singhal0408 PR conflicts with 202405 branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

7 participants