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

Do we need to regenerate genesis binary after modifying runtime? #395

Closed
hackfisher opened this issue Mar 25, 2020 · 3 comments
Closed

Do we need to regenerate genesis binary after modifying runtime? #395

hackfisher opened this issue Mar 25, 2020 · 3 comments

Comments

@hackfisher hackfisher added the U-Question [Uncategorized] Further information/discussion is requested label Mar 25, 2020
@hackfisher
Copy link
Contributor Author

hackfisher commented Mar 25, 2020

This only applies to cases of resetting or purging network, for long running networks, since the genesis binary will not be used, they will not be confused by this question.

@aurexav
Copy link
Member

aurexav commented Mar 25, 2020

Xavier:

Hi.
Is necessary to re build-spec after modifying runtime?
Just for starting a new network, not runtime-upgrade.
I only rebuild it when the chain_spec.rs changed.

Jam:

if there are new config()s, yes, otherwise I don't believe so.

Xavier:

We modify the primitives/runtime/digest.rs and deposit_log a new custom DigestItem in our frame/module. Got Number of digest items must match that calculated.
These modification did't touch the chain_spec.rs or config().
But we've to rebuild it.
If we add a flag `--execution=Native`, this panic gone.
frame_system: Some(SystemConfig {
    code: WASM_BINARY.to_vec(),
    changes_trie_config: Default::default(),
}),
Rebuild it to upgrade the WASM_BINARY?

Jam:

correct - the chain spec contains the entire wasm blob - either the native or json formats - but usually just rebuilding the node or runtime is sufficient for that 

@aurexav
Copy link
Member

aurexav commented Mar 27, 2020

Yes, for deploying a brand new network.

@aurexav aurexav closed this as completed Mar 27, 2020
@aurexav aurexav removed the U-Question [Uncategorized] Further information/discussion is requested label Mar 27, 2020
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

No branches or pull requests

2 participants