From 09bb15c36aac0e2fcfa6323bf5f90bc30a45c9c7 Mon Sep 17 00:00:00 2001 From: Steven Malis <137308034+smalis-msft@users.noreply.github.com> Date: Tue, 7 Jan 2025 13:32:15 -0800 Subject: [PATCH] Increase the vmm test timeouts (#623) (#625) The ubuntu + tpm test is bumping right up against our timeouts. Increase them. If we can bring the test time back down we may revert this in the future. --- .config/nextest.toml | 2 +- petri/src/vm/start.rs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.config/nextest.toml b/.config/nextest.toml index 77a35b2036..5fb10368ef 100644 --- a/.config/nextest.toml +++ b/.config/nextest.toml @@ -36,7 +36,7 @@ filter = 'package(~vmm_tests)' # Allow VMM tests longer timeouts. # Important Note: Petri's watchdogs need to know this value too. # If you change it here then change it there! -slow-timeout = { period = "2m", terminate-after = 2 } +slow-timeout = { period = "3m", terminate-after = 2 } # Add retries for known flaky tests. # TODO: Fix these tests and remove the retries. diff --git a/petri/src/vm/start.rs b/petri/src/vm/start.rs index f6fa46c667..aadffc762b 100644 --- a/petri/src/vm/start.rs +++ b/petri/src/vm/start.rs @@ -248,7 +248,7 @@ impl PetriVmConfig { ) -> anyhow::Result>> { // Our CI environment will kill tests after some time. We want to save // some information about the VM if it's still running at that point. - const TIMEOUT_DURATION_MINUTES: u64 = 4; + const TIMEOUT_DURATION_MINUTES: u64 = 6; const TIMER_DURATION: Duration = Duration::from_secs(TIMEOUT_DURATION_MINUTES * 60 - 10); let inspect_log_path = output_dir.join("timeout_inspect.log");