paritytech / polkadot-sdk

The Parity Polkadot Blockchain SDK
https://polkadot.network/
1.78k stars 640 forks source link

Westend People chain - not discovering peers #3856

Open miloskriz opened 5 months ago

miloskriz commented 5 months ago

Is there an existing issue?

Experiencing problems? Have you tried our Stack Exchange first?

Description of bug

Dear team, hello,

New nodes trying to connect to the People system chain in Westend are not discovering any peers. We are led to believe this is as a consequence of an error in the current chainspecs available in this repository.

Please find the logs of a node trying to connect to this chain here, the command used for this node is shown further below. An interesting point to highlight from the logs is that the genesis block look different than the one shown in the polkajs explorer:

[Parachain] 🔨 Initializing Genesis block/state (state: 0x2bcb…3f58, header-hash: 0x5bc9…7d9a)

Finally, please note that the following bootnodes of the specification file are not responding to ping or telnet requests on the relevant ports (30333 & 443):

regarding the rest of the bootnodes, we cannot comment on whether their peerIds are still correct for the relevant chain.

We would be grateful if prior to updating the whole chainspecs you could kindly advise the details of at least one current bootnode to try connect?

Many thanks!

Best regards

Milos

Steps to reproduce

using the latest polkadot-parachain binary (v1.9.0), execute it as:

polkadot-parachain \
  --name TEST_WND_PEOPLE \
  --chain people-westend \
  --database paritydb \
  --state-pruning 256 \
  --log sync=warn,afg=warn,babe=warn \
  --no-mdns \
  --no-hardware-benchmarks \
  --rpc-external \
  --prometheus-external \
  --rpc-methods safe \
  --rpc-cors all 
miloskriz commented 5 months ago

Dear team,

Found it!!!.. the correct chainspecs are still not merged to main.

This issue can now be closed!

Cheers!

Milos

miloskriz commented 5 months ago

Dear team, dear @NachoPal, hello,

Just reopening this issue to remind that the correct chainspecs for people-westend still need to be merged into the master branch.

The current chainspecs of this chain are still stored in this commit that belongs to this branch and is part of pull request #2931.

The Infrastructure Builders' Programme has also championed the alternative pull request #3957 inserting all bootnodes provided to the chain by the programme..

Please consider merging them at your earliest convenience?

Many thanks!

Milos