mirror of
https://github.com/sigp/lighthouse.git
synced 2026-05-30 12:47:05 +00:00
Gloas dont enforce peer column custody on block import (#9341)
Peers that advertise that they have imported a block may not have the columns for that slot available post-Gloas. Ensure that we dont penalize them. Co-Authored-By: Eitan Seri-Levi <eserilev@ucsc.edu>
This commit is contained in:
@@ -305,7 +305,12 @@ impl<T: BeaconChainTypes> ActiveCustodyRequest<T> {
|
|||||||
// must have its columns in custody. In that case, set `true = enforce max_requests`
|
// must have its columns in custody. In that case, set `true = enforce max_requests`
|
||||||
// and downscore if data_columns_by_root does not return the expected custody
|
// and downscore if data_columns_by_root does not return the expected custody
|
||||||
// columns. For the rest of peers, don't downscore if columns are missing.
|
// columns. For the rest of peers, don't downscore if columns are missing.
|
||||||
lookup_peers.contains(&peer_id),
|
//
|
||||||
|
// Post-Gloas, blocks and payload envelopes are decoupled. A peer may
|
||||||
|
// have the block but not yet imported the envelope and data columns.
|
||||||
|
// Don't enforce max_responses in this case.
|
||||||
|
lookup_peers.contains(&peer_id)
|
||||||
|
&& !cx.fork_context.current_fork_name().gloas_enabled(),
|
||||||
)
|
)
|
||||||
.map_err(Error::SendFailed)?;
|
.map_err(Error::SendFailed)?;
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user