Skip to content

Commit

Permalink
fix(inoculate): bigger test run timeout (#511)
Browse files Browse the repository at this point in the history
Currently, the inoculate test command sets a timeout of 60 seconds, and
abruptly kills QEMU if it takes longer than that. This is no longer
nearly long enough to run our tests. This commit increases the timeout
to 1200 seconds (20 minutes). This is a long time, but our tests take a
while to run, and recall that this timeout applies to booting the kernel
*and running all the tests*, rather than being enforced on a per-test
basis.

We should probably make the timeout much smarter than just "kill the
QEMU process after 20 minutes", and instead do it only if we don't see
any interesting test output, or something. But, this is better than just
always failing test runs because we're stupid about timeouts, I guess.
  • Loading branch information
hawkw committed Jan 11, 2025
1 parent e7af292 commit 3380090
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion inoculate/src/qemu.rs
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,9 @@ pub enum Cmd {
///
/// If a test run doesn't complete before this timeout has elapsed, it's
/// considered to have failed.
#[clap(long, value_parser = parse_secs, default_value = "60")]
///
/// By default, this is 20 minutes (1200 seconds).
#[clap(long, value_parser = parse_secs, default_value = "1200")]
timeout_secs: Duration,

/// Disables capturing test serial output.
Expand Down

0 comments on commit 3380090

Please sign in to comment.