Skip to content
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

Missing the Polkadot People Chain chain-spec #5112

Closed
josepot opened this issue Jul 23, 2024 · 7 comments
Closed

Missing the Polkadot People Chain chain-spec #5112

josepot opened this issue Jul 23, 2024 · 7 comments
Labels
I10-unconfirmed Issue might be valid, but it's not yet known.

Comments

@josepot
Copy link

josepot commented Jul 23, 2024

As of today the Polkadot People Chain chain-spec is nowhere to be found on this repo. Would it be possible to add it into the same directory where all the other system-chain-specs are? 🙏

Also, FWIW if RFC-8 was implemented, then we wouldn't need this chain-spec 😉

@wirednkod
Copy link
Contributor

wirednkod commented Jul 23, 2024

It exists here: https://github.com/paseo-network/runtimes/tree/main/chain-specs
But is it something that should exist into the polkadot-sdk repo?

@josepot
Copy link
Author

josepot commented Jul 23, 2024

It exists here: https://github.com/paseo-network/runtimes/tree/main/chain-specs But is it something that should exist into the polkadot-sdk repo? @bkchr or @ggwpez maybe you can give some insight?

I'm afraid that you got mixed up, those are the chain-specs of Paseo. This issue is about the chain-spec of the Polkadot People Chain.

@wirednkod
Copy link
Contributor

🤦🏽 Ufff... too many parallel things going on. Thank you for straighting this up @josepot

@seadanda
Copy link
Contributor

Hi @josepot. The Polkadot People chainspec was included as an asset in the Fellows runtime 1.2.6 release here: https://github.com/polkadot-fellows/runtimes/releases/tag/v1.2.6

The hardcoded system parachain chain specs will soon be removed from polkadot-parachain and so chainspecs for new system parachains will not been added there. They will be found alongside their first released wasm in the fellowship repo from now on.

@josepot
Copy link
Author

josepot commented Jul 23, 2024

The Polkadot People chainspec was included as an asset in the Fellows runtime 1.2.6 release here: https://github.com/polkadot-fellows/runtimes/releases/tag/v1.2.6

Thanks! Although, those bootnodes are all from Parity... where can we get a more complete list of bootnodes?

They will be found alongside their first released wasm in the fellowship repo from now on

This would make a lot of sense if the RFC-8 had been implemented, as we could obtain the list of parachain bootnodes from the relay-chain. However, until that's not a reality, then we will need some sort of "authoritative" place for retrieving the bootnodes of the parachains.

@seadanda
Copy link
Contributor

Good point, I've added it to the sdk repo in #5124 so we have something in place until a better solution is found.

RE the bootnodes, there are five community invulnerables here but none marked as boot nodes at launch. I think usually the community add theirs afterwards in PRs like this one. This is another point for keeping the chain specs somewhere which can be edited and changes tracked.

For the Polkadot Coretime Chain I'll also gather some of these community boot nodes before launch.

@josepot
Copy link
Author

josepot commented Jul 24, 2024

RE the bootnodes, there are five community invulnerables here but none marked as boot nodes at launch. I think usually the community add theirs afterwards in PRs like this one. This is another point for keeping the chain specs somewhere which can be edited and changes tracked.

Exactly, yes. Thanks! This is exactly what I meant.

For the Polkadot Coretime Chain I'll also gather some of these community boot nodes before launch.

🙌

@josepot josepot closed this as completed Jul 24, 2024
github-merge-queue bot pushed a commit that referenced this issue Jul 25, 2024
Published as part of the fellowship
[v1.2.6](https://github.com/polkadot-fellows/runtimes/releases/tag/v1.2.6)
release and originally intentionally left out of the repo as the
hardcoded system chains will soon be removed from the
`polkadot-parachain`.

After a conversation in
#5112 it was pointed
out by @josepot that there should be a single authoritative source for
these chainspecs. Since this is already the place for these it will
serve until something more fitting can be worked out.
TarekkMA pushed a commit to moonbeam-foundation/polkadot-sdk that referenced this issue Aug 2, 2024
Published as part of the fellowship
[v1.2.6](https://github.com/polkadot-fellows/runtimes/releases/tag/v1.2.6)
release and originally intentionally left out of the repo as the
hardcoded system chains will soon be removed from the
`polkadot-parachain`.

After a conversation in
paritytech#5112 it was pointed
out by @josepot that there should be a single authoritative source for
these chainspecs. Since this is already the place for these it will
serve until something more fitting can be worked out.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I10-unconfirmed Issue might be valid, but it's not yet known.
Projects
None yet
Development

No branches or pull requests

3 participants