Pankaj Garg

Results 8 comments of Pankaj Garg

It might be more useful to have overrides for pubkey -> stake mapping (instead of IP address -> stake). Most known connections will be from peer nodes that have active...

> @mvines @pgarg66 As browsing through the code I would like to check with you one detail. > > The goal of this PR is to override the stake amounts...

> > It makes sense to use `all_peers()`. It'll allow a validator to allocate more QUIC streams to a node that's not a staked validator (e.g. an RPC node, or...

Looks good to me. Let's also give @alessandrod a chance to take a look before merging it.

Also, is it needed on v1.18? We are limiting backports to a minimum, but given this is a test-only modification, maybe its ok.

Do we need to backport this to v1.18?

LGTM. @alessandrod, are you planning to review as well?