Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Parachains registered at IDs other than 0 do not collate #25

Closed
JoshOrndorff opened this issue Nov 29, 2019 · 2 comments
Closed

Parachains registered at IDs other than 0 do not collate #25

JoshOrndorff opened this issue Nov 29, 2019 · 2 comments

Comments

@JoshOrndorff
Copy link
Contributor

JoshOrndorff commented Nov 29, 2019

The example parachain node has a hardcoded ID of 100. When I follow the instructions to register the parachian, I use id 100 as instructed, and the registration extrinsic completes successfully. However, the collator does not begin collating. One hint from the relay chain node logs is Local parachain duty is Some(LocalDuty { validation: Parachain(Id(0)) }). I expected that value to be 100, not 0.

Second Attempt
Kill all parachain and relay chain nodes, and purge all chains. Start the relay chain again. Start the Collator again, this time try to register at id 0 despite the hard-coded id 100 in the collator code.
Same behavior as before. Now, the collator actually starts producing blocks despite the mismatch between hard-coded id and registered id.

Third Attempt
Modify the collator node to hard-code id 0, and recompile. Kill all nodes and purge all chains. Re-start the relay chain. Re-start the collator. Register at id 0. As in the second attempt, the collator now begins producing blocks as expected.

@JoshOrndorff JoshOrndorff changed the title Cannot register parachain at ID 100 Parachains registered at IDs other than 0 do not colalte Nov 29, 2019
@JoshOrndorff JoshOrndorff changed the title Parachains registered at IDs other than 0 do not colalte Parachains registered at IDs other than 0 do not collate Nov 29, 2019
@JoshOrndorff
Copy link
Contributor Author

Also just noticed that the non-hacked collator itself reports ID(0) upon startup.

Cumulus Test Parachain Collator
2019-11-29 12:19:27   version 0.1.0-x86_64-linux-gnu
2019-11-29 12:19:27   by Parity Technologies <[email protected]>, 2019
2019-11-29 12:19:27 Chain specification: Local Testnet
2019-11-29 12:19:27 Node name: agreeable-tomatoes-0019
2019-11-29 12:19:27 Roles: FULL
2019-11-29 12:19:27 Parachain id: Id(0)

@bkchr
Copy link
Member

bkchr commented Jan 15, 2020

Works for me. Probably was related to not being up-to-date.

@bkchr bkchr closed this as completed Jan 15, 2020
Maharacha pushed a commit to Maharacha/cumulus that referenced this issue May 10, 2023
* bump rust-toolchain

* [GA] add fmt job

* cargo fmt to prepare for cumulus upgrade

* upgrade to new toolchain.toml to get fmt in ci
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants