diff --git a/node/core/candidate-validation/src/lib.rs b/node/core/candidate-validation/src/lib.rs index c9e78db77c47..0b71c77df6bf 100644 --- a/node/core/candidate-validation/src/lib.rs +++ b/node/core/candidate-validation/src/lib.rs @@ -455,9 +455,9 @@ async fn validate_candidate_exhaustive( Ok(ValidationResult::Invalid(InvalidCandidate::Timeout)), Err(ValidationError::InvalidCandidate(WasmInvalidCandidate::WorkerReportedError(e))) => Ok(ValidationResult::Invalid(InvalidCandidate::ExecutionError(e))), - Err(ValidationError::InvalidCandidate(WasmInvalidCandidate::AmbigiousWorkerDeath)) => + Err(ValidationError::InvalidCandidate(WasmInvalidCandidate::AmbiguousWorkerDeath)) => Ok(ValidationResult::Invalid(InvalidCandidate::ExecutionError( - "ambigious worker death".to_string(), + "ambiguous worker death".to_string(), ))), Ok(res) => diff --git a/node/core/candidate-validation/src/tests.rs b/node/core/candidate-validation/src/tests.rs index 30fa96ccc398..a1e4a9d15036 100644 --- a/node/core/candidate-validation/src/tests.rs +++ b/node/core/candidate-validation/src/tests.rs @@ -422,7 +422,7 @@ fn candidate_validation_bad_return_is_invalid() { let v = executor::block_on(validate_candidate_exhaustive( MockValidatorBackend::with_hardcoded_result(Err(ValidationError::InvalidCandidate( - WasmInvalidCandidate::AmbigiousWorkerDeath, + WasmInvalidCandidate::AmbiguousWorkerDeath, ))), validation_data, validation_code, diff --git a/node/core/pvf/src/error.rs b/node/core/pvf/src/error.rs index 8834d3c51738..977f24b1482d 100644 --- a/node/core/pvf/src/error.rs +++ b/node/core/pvf/src/error.rs @@ -71,7 +71,7 @@ pub enum InvalidCandidate { /// thrown at it and hopefully the operator notices it by decreased reward performance of the /// validator. On the other hand, if the worker died because of (b) we would have better chances /// to stop the attack. - AmbigiousWorkerDeath, + AmbiguousWorkerDeath, /// PVF execution (compilation is not included) took more time than was allotted. HardTimeout, } diff --git a/node/core/pvf/src/execute/queue.rs b/node/core/pvf/src/execute/queue.rs index 9376d7d76f78..c8d75fb03987 100644 --- a/node/core/pvf/src/execute/queue.rs +++ b/node/core/pvf/src/execute/queue.rs @@ -240,7 +240,7 @@ fn handle_job_finish( Outcome::HardTimeout => (None, Err(ValidationError::InvalidCandidate(InvalidCandidate::HardTimeout))), Outcome::IoErr => - (None, Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbigiousWorkerDeath))), + (None, Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbiguousWorkerDeath))), }; queue.metrics.execute_finished(); diff --git a/node/core/pvf/src/host.rs b/node/core/pvf/src/host.rs index 62cf77ba0912..dccb52781a01 100644 --- a/node/core/pvf/src/host.rs +++ b/node/core/pvf/src/host.rs @@ -646,7 +646,7 @@ async fn handle_cleanup_pulse( artifact_ttl: Duration, ) -> Result<(), Fatal> { let to_remove = artifacts.prune(artifact_ttl); - tracing::info!( + tracing::debug!( target: LOG_TARGET, "PVF pruning: {} artifacts reached their end of life", to_remove.len(), @@ -1039,27 +1039,27 @@ mod tests { ); result_tx_pvf_1_1 - .send(Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbigiousWorkerDeath))) + .send(Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbiguousWorkerDeath))) .unwrap(); assert_matches!( result_rx_pvf_1_1.now_or_never().unwrap().unwrap(), - Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbigiousWorkerDeath)) + Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbiguousWorkerDeath)) ); result_tx_pvf_1_2 - .send(Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbigiousWorkerDeath))) + .send(Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbiguousWorkerDeath))) .unwrap(); assert_matches!( result_rx_pvf_1_2.now_or_never().unwrap().unwrap(), - Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbigiousWorkerDeath)) + Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbiguousWorkerDeath)) ); result_tx_pvf_2 - .send(Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbigiousWorkerDeath))) + .send(Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbiguousWorkerDeath))) .unwrap(); assert_matches!( result_rx_pvf_2.now_or_never().unwrap().unwrap(), - Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbigiousWorkerDeath)) + Err(ValidationError::InvalidCandidate(InvalidCandidate::AmbiguousWorkerDeath)) ); }