-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Make init command more customizable #878
Comments
I think that by default init should not create a genesis.json. Creating one is not a standard production use case and should require an explicit option. |
I think init should create an output file that can be sent to a maintainer to help construct a genesis.json. So init.json should look like
In this way everyone can run |
@MikeSofaer yeah I was thinking something similar to the
|
I like the idea of a command designed specifically for production workflow - but I think we should keep the simple Perhaps we could rename the current |
I like the idea of having it all in one command - init for testing where it does write the files is good as a flag. But it doesn't really matter can be different for each Application |
in the staking module the default token for staking is
we should make the genesis file consistent with design. |
Currently the init command (within
server/init.go
) is quite restrictive - you cannot add new flags, pretty much the only thing you can do is pass in a function which is used to populate the the genesis config file. For gaia for instance, I would like to have the init command take certain options to allow for various numbers validators being created in the genesis config. Additionally it would be useful to be able to specify the addresses that may have tokens in them and how much tokens... For these reasons and more I think we need to totally reconfigure the init command and create something much more customizable which theapp.go
file can import and extend/modify with functions.The ability to create a much more streamlined init process should be able to greatly ease some of the annoyances with testnet setup as well.
@adrianbrink @cwgoes @ebuchman
The text was updated successfully, but these errors were encountered: