Skip to content

Commit

Permalink
increase spin up timeout to 2 minutes to fix CI
Browse files Browse the repository at this point in the history
We're not sure yet why we've started consistently exceeding the old 20 second
timeout on tests which run in a 2-3 seconds locally, but that's what's
happening.  If it turns out even 2 minutes is insufficient, we'll need to dig
deeper.

Signed-off-by: Joel Dice <joel.dice@fermyon.com>
  • Loading branch information
dicej committed Jan 11, 2024
1 parent de4b658 commit 7eed4a5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/testing-framework/src/spin.rs
Original file line number Diff line number Diff line change
Expand Up @@ -55,13 +55,13 @@ impl Spin {
);
}

if start.elapsed() > std::time::Duration::from_secs(20) {
if start.elapsed() > std::time::Duration::from_secs(2 * 60) {
break;
}
std::thread::sleep(std::time::Duration::from_millis(50));
}
anyhow::bail!(
"`spin up` did not start server or error after 20 seconds. stderr:\n\t{}",
"`spin up` did not start server or error after two minutes. stderr:\n\t{}",
spin.stderr.output_as_str().unwrap_or("<non-utf8>")
)
}
Expand Down

0 comments on commit 7eed4a5

Please sign in to comment.