Skip to content

[action] [PR:9075] [chassis] accomodate test_memory_exhaustion.py if random dut is chassis sup #9329

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

Merged
merged 1 commit into from
Aug 8, 2023

Conversation

mssonicbld
Copy link
Collaborator

Description of PR

There are 2 fixes in this PR to enhance for chassis:

  1. Right now in processes_utils.py when check_critical_services, we rely on what is pre-defined in inventory file, otherwise it is 300 sec, but for SUP, we should give 900sec as most SUPs have 10+ dockers that need to come back up, it should differ from linecards or any single-dut. magic 900 sec comes from experientment, where I had run twice the test, 1st time need 800+sec, 2nd time needs ~702sec
  2. During memory exhaustion test, if the random selected dut is chassis SUP, we also need to check linecards bootup and critical services are up.

Summary:
Fixes # (issue)
#9087

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Collaborator Author

Original PR: #9075

@mssonicbld mssonicbld merged commit a08b96e into sonic-net:202305 Aug 8, 2023
@mssonicbld mssonicbld deleted the cherry/202305/9075 branch February 4, 2024 09:39
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