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

Introduce chain spec generator #78

Closed
wants to merge 5 commits into from
Closed

Conversation

bkchr
Copy link
Contributor

@bkchr bkchr commented Oct 25, 2023

Until paritytech/polkadot-sdk#25 is finished, we need a way to generate a chain spec for the runtimes. This pull request introduces a chain spec generator that can fulfill this task. It currently only supports the relay chain, but can be extended in the future to also support the system chains.

When paritytech/polkadot-sdk#25 is delivered, we can remove the chain spec generator as it will not be required anymore.

@bkchr bkchr changed the title Introduce a chain spec generator Introduce chain spec generator Oct 25, 2023
Hi @bkchr, I know that this is temporary tool but in order to create a
workaround for zombienet is useful to allow to pass customized plain
spec file and get the `raw` one.

Thx!
@NachoPal
Copy link
Contributor

NachoPal commented Nov 3, 2023

Extension for System Parachains in #81

Based on: #78

Without this fix we cannot run benchmarks for kusama/polkadot.

Tested here:
#58 (comment)
@NachoPal
Copy link
Contributor

This PR and #81 are going to be merged?

@bkchr
Copy link
Contributor Author

bkchr commented Nov 29, 2023

This PR and #81 are going to be merged?

Not really needed anymore with the new chain spec builder runtime api.

@bkchr
Copy link
Contributor Author

bkchr commented Dec 8, 2023

Yes!

@bkchr bkchr closed this Dec 8, 2023
bkchr added a commit that referenced this pull request Jan 17, 2024
…ss (#127)

Closes #50

- Adds the `chain-spec-generator` from
#78 and
#81, updated to work
with `master`.
- Adds detailed instructions to the `README.md` for generating weights
for fellowship runtimes

I am currently generating weights and will add them in a seperate PR
when they are ready.

TODO
- [x] Add step for checking weight outputs using
https://github.com/ggwpez/substrate-weight-compare in readme
- [ ] Fix encointer chain spec defaults, or open new issue for it

### Why not use the `polkadot-sdk` generic chain-spec-builder?

The genesis state configs generated by this CLI are unusable for
benchmarking, due to poor defaults. We need better defaults for our
pallet genesis state before we can use this CLI. See also
paritytech/polkadot-sdk#2713.

---------

Co-authored-by: Bastian Köcher <info@kchr.de>
Co-authored-by: Javier Viola <javier@parity.io>
Co-authored-by: NachoPal <ignacio.palacios.santos@gmail.com>
Co-authored-by: Svyatoslav Nikolsky <svyatonik@gmail.com>
Co-authored-by: Branislav Kontur <bkontur@gmail.com>
Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Co-authored-by: Bastian Köcher <git@kchr.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants