-
Notifications
You must be signed in to change notification settings - Fork 36
add base code for testnet #34
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
Conversation
very hot. |
@@ -200,7 +200,7 @@ func initRootCmd(rootCmd *cobra.Command, encodingConfig params.EncodingConfig) { | |||
genutilcli.ValidateGenesisCmd(app.ModuleBasics), | |||
AddGenesisAccountCmd(app.DefaultNodeHome), | |||
tmcli.NewCompletionCmd(rootCmd, true), | |||
// NewTestnetCmd(app.ModuleBasics, banktypes.GenesisBalancesIterator{}), | |||
testnetCmd(app.ModuleBasics, banktypes.GenesisBalancesIterator{}), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We removed this since we are going to have to test with docker / compose
also if we did use it,
eved testnet init
Error: unable to resolve type URL /cosmos.staking.v1beta1.MsgCreateValidator
built in testnet is too hard to add custom logic for, might as well just run natively & not use the SDK testnet at all
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like it in
|
||
# Start a 4-node testnet locally | ||
localnet-start: localnet-clean |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docker-compose down
sudo rm -rf build
rm mnemonics.txt
rm: cannot remove 'mnemonics.txt': No such file or directory
make: *** [Makefile:160: localnet-clean] Error 1
Should run the add-keys.sh script here automatically?
when I run manually
Could not open file build/node3/eved/key_seed.json: No such file or directory
also mainnet = eve-1, so should do eve-t1 or l1 for test/local version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should not have sudo in a makefile
## Starting localnet | ||
|
||
``` | ||
make install | ||
make localnet-build | ||
make localnet-start |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(vinh/testnet-cmd) -> $ make localnet-start
docker-compose down
sudo rm -rf build
rm mnemonics.txt
rm: cannot remove 'mnemonics.txt': No such file or directory
make: *** [Makefile:160: localnet-clean] Error 1
@Reecepbcups if we can use the sdk's native testnet, I'd like to. can we keep makefile changes for the docker based testnetting to another pr? Also, I think that we should keep the docker based testnet. |
add cmd, script for testnet