-
Notifications
You must be signed in to change notification settings - Fork 941
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Need help debugging potential libp2p issue #3205
Comments
Without having the logs of the node you are connecting to, it is hard to say why the connection times out. Do you know what |
@thomaseizinger could please provide exact command to get logs you need? I will try get it from remote node.
Based on https://github.com/paritytech/substrate/blob/polkadot-v0.9.23/client/network/Cargo.toml it must be 0.44.0 |
The jump was from I created a local bootnode for polkadot version 0.9.23 (with libp2p 0.44.0) there AND a local client node with polkadot version 0.9.26 (with libp2p 0.46.1) and couldn't connect. I used debug level, so feel free to reach out when you need trace level. |
Above attached node seems to be from bootnode-2, following two errors are interesring in the log
seems like bootnode-1 has different PeerID then expected I am trying to get logs from other bootnodes too. |
Both of these PeerIds are the same though 😄 |
full_parachain.zip log_bootnode-1.txt In bootnode log I don't see anything related to 12D3KooWAd5NrFfuCWYr2hqhpJqKWoQb7cbNJB6xh3u9ikmHvxNb @thomaseizinger could you please shed some light? |
So the first thing that is suspicious in
Can you show the code of the bootnode? My suspicion is that not everything that would be interesting is logged there. I also tried to connect to the bootnode via https://github.com/mxinden/libp2p-lookup and it also fails with a timeout:
This makes me think that something is definitely wrong with the configuration of the bootnode. |
Thanks @thomaseizinger for spending time one this!
In substrate libp2p is used at https://github.com/paritytech/substrate/tree/polkadot-v0.9.23/client/network
Please let me know if you any experiment in your mind, I will collect data/logs etc for our analysis. |
One thing also makes me curious is following
Here for relay chain I can see both ip4 and ip6 addresses but for para-chain only ip6 , why ? |
Update:
if I remove ws/ from MultiAddress then it does not sync. |
I am not at all familiar with that code. Are these two different
I assume the double slash in there is a typo? That is not a valid address to begin with. If a node only listens on the websocket interface, then removing that bit from the address will have it fail to connect. |
Responded here: paritytech/substrate#12704 (comment) |
Looking at the libp2p codebase this can only happen in one case: when a node tries to dial itself (via external address) and fails because rust-libp2p/swarm/src/connection/pool.rs Line 695 in 7665e74
LocalPeerId , but here we are. (cc @thomaseizinger)
|
Here where? If you're referring to https://github.com/libp2p/rust-libp2p/files/10193919/full_parachain.zip log, then it contains this line:
where the parachain binds to |
I'd suggest to close this issue and move the discussion to paritytech/substrate#12704. @thomaseizinger I'll let you and the team decide if you want to do anything with #2441 (review). |
I would be in favor of a more elaborate error name. Please open an issue or provide a patch in case you feel strongly about this @melekes. |
I am also okay with introducing another error variant for this. |
to diff between wrong peer ID and local peer ID. Refs libp2p#3205
Is this still an issue? |
Oops, seems like we needed more information for this issue, please comment with more details or this issue will be closed in 7 days. |
This issue was closed because it is missing author input. |
Summary
Hello, libp2p community I work on substrate based project and when I am trying to upgrade to substrate version from 0.9.23 to 0.9.26 or 0.9.29 the substrate node does not connect to parachain peer to sync data. Is there any issue with libp2p version for backward compatibility? as production deployed version uses 0.9.23 substrate and libp2p version must be old according to that.
Here is relevant issue on substrate repo Client side connection issues from polkadot v0.9.23 to v0.9.26 · Issue #12704 · paritytech/substrate · GitHub
However I need more help understanding debug log generated when executing the node.
Here is complete log filtered for “Parachain” gist:352ae47d5c79aef149033aea71cbd9a7 · GitHub
And below is last few relevant lines to quick glance. Kindly please help what is going wrong.
Expected behaviour
Expected behaviour is that client connects fine with old version node which uses substrate 0.9.23.
Actual behaviour
As shown on logs above it gives timeout error.
Detailed Log
Here is complete log filtered for “Parachain” gist:352ae47d5c79aef149033aea71cbd9a7 · GitHub
Version
0.46.1
Would you like to work on fixing this bug?
No
The text was updated successfully, but these errors were encountered: