cerc-io/lighthouse
16
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Update engine state in broadcast (#3071)

Browse Source 
## Issue Addressed

N/A

## Proposed Changes

Set the engine state to `EngineState::Offline` if the engine api call fails during broadcast. This caused issues while pausing sync when the execution engine is offline because `EngineState` always returned `Synced`.
This commit is contained in:
Pawan Dhananjay 2022-03-28 23:32:11 +00:00
parent 20e32f5812
commit a42cb69f6e
1 changed files with 15 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
beacon_node/execution_layer/src/engines.rs
View File

@ -399,18 +399,22 @@ impl<T: EngineApi> Engines<T> {
let futures = self.engines.iter().map(|engine| async move { let futures = self.engines.iter().map(|engine| async move {
let is_offline = *engine.state.read().await == EngineState::Offline; let is_offline = *engine.state.read().await == EngineState::Offline;
if !is_offline { if !is_offline {
func(engine).await.map_err(|error| { match func(engine).await {
debug!( Ok(res) => Ok(res),
self.log, Err(error) => {
"Execution engine call failed"; debug!(
"error" => ?error, self.log,
"id" => &engine.id "Execution engine call failed";
); "error" => ?error,
EngineError::Api { "id" => &engine.id
id: engine.id.clone(), );
error, *engine.state.write().await = EngineState::Offline;
Err(EngineError::Api {
id: engine.id.clone(),
error,
})
} }
}) }
} else { } else {
Err(EngineError::Offline { Err(EngineError::Offline {
id: engine.id.clone(), id: engine.id.clone(),