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

Address observed proposers behaviour (#4192)

Browse Source 
## Issue Addressed

NA

## Proposed Changes

Apply two changes to code introduced in #4179:

1. Remove the `ERRO` log for when we error on `proposer_has_been_observed()`. We were seeing a lot of this in our logs for finalized blocks and it's a bit noisy.
1. Use `false` rather than `true` for `proposal_already_known` when there is an error. If a block raises an error in `proposer_has_been_observed()` then the block must be invalid, so we should process (and reject) it now rather than queuing it.

For reference, here is one of the offending `ERRO` logs:

```
ERRO Failed to check observed proposers block_root: 0x5845…878e, source: rpc, error: FinalizedBlock { slot: Slot(5410983), finalized_slot: Slot(5411232) }
```

## Additional Info

NA
This commit is contained in:
Paul Hauner 2023-04-14 06:37:16 +00:00
parent 2b3084f578
commit dd124b2d68
2 changed files with 11 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
beacon_node/beacon_chain/src/lib.rs
View File

@ -32,7 +32,7 @@ pub mod migrate;
mod naive_aggregation_pool; mod naive_aggregation_pool;
mod observed_aggregates; mod observed_aggregates;
mod observed_attesters; mod observed_attesters;
mod observed_block_producers; pub mod observed_block_producers;
pub mod observed_operations; pub mod observed_operations;
pub mod otb_verification_service; pub mod otb_verification_service;
mod persisted_beacon_chain; mod persisted_beacon_chain;

23
beacon_node/network/src/beacon_processor/worker/sync_methods.rs
View File

@ -9,8 +9,8 @@ use crate::sync::manager::{BlockProcessType, SyncMessage};
use crate::sync::{BatchProcessResult, ChainId}; use crate::sync::{BatchProcessResult, ChainId};
use beacon_chain::CountUnrealized; use beacon_chain::CountUnrealized;
use beacon_chain::{ use beacon_chain::{
BeaconChainError, BeaconChainTypes, BlockError, ChainSegmentResult, HistoricalBlockError, observed_block_producers::Error as ObserveError, BeaconChainError, BeaconChainTypes,
NotifyExecutionLayer, BlockError, ChainSegmentResult, HistoricalBlockError, NotifyExecutionLayer,
}; };
use lighthouse_network::PeerAction; use lighthouse_network::PeerAction;
use slog::{debug, error, info, warn}; use slog::{debug, error, info, warn};
@ -85,21 +85,18 @@ impl<T: BeaconChainTypes> Worker<T> {
}; };
// Check if a block from this proposer is already known. If so, defer processing until later // Check if a block from this proposer is already known. If so, defer processing until later
// to avoid wasting time processing duplicates. // to avoid wasting time processing duplicates.
let proposal_already_known = self let proposal_already_known = match self
.chain .chain
.observed_block_producers .observed_block_producers
.read() .read()
.proposer_has_been_observed(block.message()) .proposer_has_been_observed(block.message())
.map_err(|e| { {
error!( Ok(is_observed) => is_observed,
self.log, // Both of these blocks will be rejected, so reject them now rather
"Failed to check observed proposers"; // than re-queuing them.
"error" => ?e, Err(ObserveError::FinalizedBlock { .. })
"source" => "rpc", | Err(ObserveError::ValidatorIndexTooHigh { .. }) => false,
"block_root" => %block_root };
);
})
.unwrap_or(true);
if proposal_already_known { if proposal_already_known {
debug!( debug!(
self.log, self.log,