Skip to content

Mark hung Cloud Hypervisor tests as failed + a few fixes#4384

Merged
LiliDeng merged 2 commits intomicrosoft:mainfrom
anirudhrb:fail_hung_ch_tests
Mar 27, 2026
Merged

Mark hung Cloud Hypervisor tests as failed + a few fixes#4384
LiliDeng merged 2 commits intomicrosoft:mainfrom
anirudhrb:fail_hung_ch_tests

Conversation

@anirudhrb
Copy link
Copy Markdown
Collaborator

Description

See commit messages for details.

Related Issue

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Documentation update

Checklist

  • Description is filled in above
  • No credentials, secrets, or internal details are included
  • Peer review requested (if not, add required peer reviewers after raising PR)
  • Tests executed and results posted below

Test Validation

Key Test Cases:

Impacted LISA Features:

Tested Azure Marketplace Images:

Test Results

Image VM Size Result
PASSED / FAILED / SKIPPED

@anirudhrb anirudhrb requested a review from vyadavmsft as a code owner March 23, 2026 14:09
Copilot AI review requested due to automatic review settings March 23, 2026 14:09
@anirudhrb anirudhrb requested a review from pupacha as a code owner March 23, 2026 14:09
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Cloud Hypervisor test runner to better detect/flag hung subtests and improve failure reporting/diagnostics when tests time out.

Changes:

  • Add explicit hung-test detection and propagate it into subtest results/failure summaries.
  • Update timeout/failure handlers to include concise summaries of failures vs hung tests.
  • Reduce console noise from watchdog diagnostics while still appending details to the artifact log.

Comment thread lisa/microsoft/testsuites/cloud_hypervisor/ch_tests_tool.py
Comment thread lisa/microsoft/testsuites/cloud_hypervisor/ch_tests_tool.py Outdated
Comment thread lisa/microsoft/testsuites/cloud_hypervisor/ch_tests_tool.py
Comment thread lisa/microsoft/testsuites/cloud_hypervisor/ch_tests_tool.py
Move the hung test detection logic earlier into _extract_test_results()
and mark those tests as failed. Change the rest of the code accordingly
so that we still get the hung tests in the diagnostic summary.

Signed-off-by: Anirudh Rayabharam <anrayabh@microsoft.com>
The pstree, ps and core dump diagnostics are not useful in the LISA
logs. They're already being collected to a separate file. These produce
hundreds of lines of output and make it difficult to find the actual
dev_cli.sh output that is more critical for figuring out what went wrong
in a run.

Signed-off-by: Anirudh Rayabharam <anrayabh@microsoft.com>
@anirudhrb anirudhrb force-pushed the fail_hung_ch_tests branch from 9b9bb94 to dd9abe0 Compare March 25, 2026 13:00
Comment thread lisa/microsoft/testsuites/cloud_hypervisor/ch_tests_tool.py
@LiliDeng LiliDeng merged commit 3b0f674 into microsoft:main Mar 27, 2026
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants