Skip to content

Commit 5ea7505

Browse files
committed
Increase slashing protection test SQL timeout to 500ms (#4574)
## Issue Addressed NA ## Proposed Changes We've been seeing a lot of [CI failures](https://github.com/sigp/lighthouse/actions/runs/5781296217/job/15666209142) with errors like this: ``` ---- extra_interchange_tests::export_same_key_twice stdout ---- thread 'extra_interchange_tests::export_same_key_twice' panicked at 'called `Result::unwrap()` on an `Err` value: SQLError("Unable to open database: Error(None)")', validator_client/slashing_protection/src/extra_interchange_tests.rs:48:67 ``` I'm assuming they're timeouts. I noticed that tests have a 0.1s timeout. Perhaps this just doesn't cut it when our new runners are overloaded. ## Additional Info NA
1 parent 5214321 commit 5ea7505

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

validator_client/slashing_protection/src/slashing_database.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ pub const POOL_SIZE: u32 = 1;
2323
#[cfg(not(test))]
2424
pub const CONNECTION_TIMEOUT: Duration = Duration::from_secs(5);
2525
#[cfg(test)]
26-
pub const CONNECTION_TIMEOUT: Duration = Duration::from_millis(100);
26+
pub const CONNECTION_TIMEOUT: Duration = Duration::from_millis(500);
2727

2828
/// Supported version of the interchange format.
2929
pub const SUPPORTED_INTERCHANGE_FORMAT_VERSION: u64 = 5;

0 commit comments

Comments
 (0)