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

[action] [PR:14654] [Nokia-7215] Improve test_reboot stability at teardown stage #15086

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

mssonicbld
Copy link
Collaborator

Description of PR

Summary:
Nokia-7215 has low performance. After device reboot, it's critical processes may not be fully up at the time SSH is reachable. For this platform, I add wait_critical_processes at teardown stage to improve the test stability.

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

Improve the test_reboot stability on Nokia-7215 platform.

How did you do it?

Add wait_critical_processes at teardown stage.

How did you verify/test it?

Verified on Nokia-7215 M0 testbed.

Any platform specific information?

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

Documentation

…et#14654)

Nokia-7215 has low performance. After device reboot, it's critical processes may not be fully up at the time SSH is reachable. For this platform, I add wait_critical_processes at teardown stage to improve the test stability.

What is the motivation for this PR?
Improve the test_reboot stability on Nokia-7215 platform.

How did you do it?
Add wait_critical_processes at teardown stage.

How did you verify/test it?
Verified on Nokia-7215 M0 testbed.
@mssonicbld
Copy link
Collaborator Author

Original PR: #14654

@mssonicbld mssonicbld merged commit 49dcdc6 into sonic-net:202405 Oct 22, 2024
13 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