Skip to content

[action] [PR:22252] [Smartswitch] Stabilize the DPU kernel panic and memory exhaustion tests#22691

Open
mssonicbld wants to merge 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/22252
Open

[action] [PR:22252] [Smartswitch] Stabilize the DPU kernel panic and memory exhaustion tests#22691
mssonicbld wants to merge 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/22252

Conversation

@mssonicbld
Copy link
Collaborator

Description of PR

Summary:
We should check the DPUs are offline before checking they are online after the kernel panic or memory exhaustion.
Otherwise the check for DPU online could pass even before the DPUs are rebooted and the later critical services check will fail.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
  • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Approach

What is the motivation for this PR?

Stabilize the DPU kernel panic and memory exhaustion tests

How did you do it?

Add a check for DPUs offline before the post_test_dpus_check in 2 test cases of test test_reload_dpu.py:
test_dpu_status_post_dpu_kernel_panic
test_dpu_check_post_dpu_mem_exhaustion

How did you verify/test it?

Run the test on SN4280 testbed.

Any platform specific information?

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

Documentation

…#22252)

We should check the DPUs are offline before checking they are online
after the kernel panic or memory exhaustion.
Otherwise the check for DPU online could pass even before the DPUs are
rebooted and the later crirical services check will fail.

Signed-off-by: mssonicbld <sonicbld@microsoft.com>
@mssonicbld
Copy link
Collaborator Author

Original PR: #22252

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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