Too many unknown peers in test mainnet nodes about lodestar HOT 4 OPEN

curl http://localhost:9596/eth/v1/lodestar/peers shows too many NA peers, which means agentVersion is not set for them https://github.com/ChainSafe/lodestar/blob/v1.15.0/packages/beacon-node/src/network/peers/peersData.ts#L38

from lodestar.

looks like it caused by this one https://github.com/ChainSafe/lodestar/pull/6028/files#diff-e50e7caad2bee8b503fd63320dbc6c32e9f90f05aa76b211df989d02030a9aa6R613

it's supposed to get agent version after 1s but right now we try to get agent version immediately right after a peer:connect event cc @nazarhussain

from lodestar.

@libp2p/identify now provides a way to manually trigger identify calls. We should use that rather than waiting X seconds and hoping that an identify happened.

should look something like:

libp2p.identify.identify(peerId).then((i) => {
  const agentVersionBytes = i.AgentVersion
  ...
}).catch(...);

We also need to configure the identify module to not automatically trigger identify too.

createLibp2p({
  identify: identify({... // something here

from lodestar.

@wemeetagain Looking forward to work on this issue after we release the fix you made libp2p/js-libp2p#2387

from lodestar.