Skip to content

Run thread-heavy integration tests in serial#1216

Open
ludfjig wants to merge 1 commit intohyperlight-dev:mainfrom
ludfjig:serial_tests_thread
Open

Run thread-heavy integration tests in serial#1216
ludfjig wants to merge 1 commit intohyperlight-dev:mainfrom
ludfjig:serial_tests_thread

Conversation

@ludfjig
Copy link
Contributor

@ludfjig ludfjig commented Feb 7, 2026

Tries to fix some recent test hangs that shows up in CI due to thread congestion and interrupt_infinite_loop_stress_test sleeping for 1000ms not being enough for a sandbox on another thread to enter a guest call. (see here for example)

If this doesn't fix it, we can add the attribute to some more thread-heavy tests in sandbox_host_tests, or introduce a barrier in a host call, but hopefully this is sufficient

@ludfjig ludfjig changed the title Run thread-heavy tests serially Run thread-heavy integration tests serially Feb 7, 2026
@ludfjig ludfjig changed the title Run thread-heavy integration tests serially Run thread-heavy integration tests in serial Feb 7, 2026
@ludfjig ludfjig added the kind/bugfix For PRs that fix bugs label Feb 7, 2026
@ludfjig ludfjig requested a review from Copilot February 7, 2026 00:17
Signed-off-by: Ludvig Liljenberg <4257730+ludfjig@users.noreply.github.com>
Copy link
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 aims to reduce CI hangs by preventing the most thread-heavy integration tests from running concurrently and by adjusting timing in an interrupt-related stress test.

Changes:

  • Marked several thread-heavy interrupt stress tests to run serially via serial_test (thread_heavy group).
  • Increased the wait duration before sending an interrupt in interrupt_custom_signal_no_and_retry_delay and asserted kill() succeeds.
  • Simplified the Justfile integration-test invocation by removing the special-case execution of interrupt_random_kill_stress_test.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/hyperlight_host/tests/integration_test.rs Adds serial_test and serializes selected thread-heavy interrupt stress tests; tweaks interrupt timing/assertion in one test.
Justfile Runs integration tests in a single invocation instead of skipping/rerunning one stress test separately.

@ludfjig ludfjig marked this pull request as ready for review February 7, 2026 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bugfix For PRs that fix bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant