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

Problem: no testnet compatibility tests #204

Closed
tomtau opened this issue Oct 22, 2020 · 4 comments
Closed

Problem: no testnet compatibility tests #204

tomtau opened this issue Oct 22, 2020 · 4 comments
Assignees

Comments

@tomtau
Copy link
Contributor

tomtau commented Oct 22, 2020

as there's already a public testnet running (on top of unreleased SDK version), it'll be good to find out early if something broke with a simple dependency upgrade (and one needs to trigger some network upgrade proposal and/or provide some conversion tools), so this includes:

  1. parsing genesis.json
  2. processing block data / transactions -- (either from network or reading data persisted by the old binary)
  3. external API changes
@yihuang
Copy link
Collaborator

yihuang commented Nov 2, 2020

I guess we can make an integration test case to join the running testnet with recent binary. and issue a warning if it don't work.

@yihuang
Copy link
Collaborator

yihuang commented Nov 4, 2020

https://github.com/cosmos/cosmos-sdk/pull/7526/files#diff-046f100a1eb97e25aa6ecc5b5a618c222775184faf805d7a7cbb9e9e6a92aba1R47

current master binary can't decode the testnet genesis file because of above change.

It's part of bigger stream of ongoing refactoring: cosmos/cosmos-sdk#7357

@yihuang
Copy link
Collaborator

yihuang commented Nov 12, 2020

  • investigate if it's possible to do snapshot in tendermint before upgrade, then we don't need to worry about migrating old abci messages.
  • then we can focus on migration of abci state storage.
  • investigate a potential issue that node paniced too soon, the block is not being relayed successfully.

@tomtau
Copy link
Contributor Author

tomtau commented Feb 25, 2021

this was more relevant to pre-release versions which broke in undocumented ways -- it should be clearer now, but perhaps we can open separate issues for mainnet/testnet once the final things are up and running

@tomtau tomtau closed this as completed Feb 25, 2021
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

5 participants