Skip to content

feat: add node info metrics - #43

Merged
MegaRedHand merged 2 commits into
mainfrom
node-info-metrics
Jan 21, 2026
Merged

MegaRedHand merged 2 commits into
mainfrom
node-info-metrics

Conversation

@MegaRedHand

Copy link
Copy Markdown
Collaborator

No description provided.

Base automatically changed from devnet2 to main January 21, 2026 20:41
@MegaRedHand
MegaRedHand marked this pull request as ready for review January 21, 2026 20:41
@MegaRedHand
MegaRedHand merged commit 35c42f2 into main Jan 21, 2026
2 checks passed
@MegaRedHand
MegaRedHand deleted the node-info-metrics branch January 21, 2026 20:46
tcoratger pushed a commit to tcoratger/ethlambda that referenced this pull request Oct 30, 2026
…ctor (lambdaclass#43)

The chain actor re-ran, on its single thread, the column checks that
gossip validation had already run in p2p: a KZG batch, a BLS verification
and a `process_slots` per sidecar. After an empty slot, that
`process_slots` merkleizes the full mainnet state once per sidecar
(~1.5 s each, 8 custody columns). On the eth-4 follower on 2026-09-23
this held block 15280808 in the actor's mailbox for 7.6 s, and then for
another 4.7 s of "columns_wait" that was really the actor checking
columns that had already arrived.

The checks now live only in `beacon::gossip::column`. A sidecar gossip
did not accept (`Queue`, `Ignore(Overloaded)`), every sidecar fetched by
root or by range, and parked sidecars replayed after their parent imports
all go through the new `chain_checks`, run by `p2p::beacon::column_checks`
on blocking threads. The chain actor stores what reaches it without
checking it; debug builds re-run `chain_checks` there, so a p2p path that
forwards an unchecked sidecar fails a test.

`chain_checks` reads the expected proposer from the parent state's
proposer lookahead, and only advances a state clone for a slot outside
it. The column rejection and KZG metrics move with the checks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant