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

upgrade simulation for the liquidity module that is rebased to Cosmos SDK 0.44.x #4

Draft
wants to merge 34 commits into
base: master
Choose a base branch
from

Conversation

jaybxyz
Copy link
Contributor

@jaybxyz jaybxyz commented Aug 24, 2021

Summary

DO NOT MERGE THIS PR UNLESS WE ALL AGREE TO

This PR is NOT to be merged, but to outline the steps to proceed the upgrade simulation test and provide some contexts on what are some of the things that are changed from the previous upgrade test.

Steps

  • Export state from Cosmos Hub mainnet cosmoshub-4 from certain block height
  • Sort the exported state in JSON format
  • Prepare new validator consensus key files to substitute them with the existing validators' keys
  • Prepare new account mnemonics
  • Change voting power of 2 validators and staking delegation to over 67% in order to start the chain locally
  • Modify governance parameters to improve test efficiency
  • Initialize the chain and copy the exported genesis file and the prepared validator consensus key files to the chain directories respectively
  • Configure settings in both app.toml and config.toml files
  • Start the chain
  • Send an upgrade proposal to the network, vote for the proposal, and monitor the proposal until the upgrade height
  • Monitor the chains stop at the upgrade block height
  • Build new gaiad with Gravity DEX that is rebased to Cosmos SDK v0.43.0 and restart the chain
  • Test liquidity module to make sure everything is working as intended
  • Compress the exported genesis file and push it to this github repo
  • Write documentation for public good

@jaybxyz jaybxyz self-assigned this Aug 24, 2021
@dongsam dongsam changed the title upgrade simulation for the liquidity module that is rebased to Cosmos SDK 0.43.0 upgrade simulation for the liquidity module that is rebased to Cosmos SDK 0.44.0 Sep 8, 2021
Copy link
Member

@dongsam dongsam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

v5.0.5/README.md Outdated Show resolved Hide resolved
v5.0.5/README.md Outdated Show resolved Hide resolved
v5.0.5/README.md Outdated Show resolved Hide resolved
@dongsam dongsam changed the title upgrade simulation for the liquidity module that is rebased to Cosmos SDK 0.44.0 upgrade simulation for the liquidity module that is rebased to Cosmos SDK 0.44.x Oct 15, 2021
@dongsam
Copy link
Member

dongsam commented Oct 15, 2021

for tendermint/liquidity#447, to confirm that there is no problem, the above simulation is required with the corresponding versions before releasing liquidity v1.4.1, gaia vega version @kogisin

@jaybxyz
Copy link
Contributor Author

jaybxyz commented Oct 15, 2021

@dongsam Okay, I'll create another branch to work on it

@dongsam
Copy link
Member

dongsam commented Oct 21, 2021

@kogisin Could you refer tendermint/farming#184 this modification and apply it on this PR also?

@jaybxyz
Copy link
Contributor Author

jaybxyz commented Oct 21, 2021

@dongsam Okay. That is necessary since it is written specifically for MacOS.

@dongsam
Copy link
Member

dongsam commented Oct 26, 2021

Looks good to me now but we could add some details for sed OS difference, and let us leave it as PR and merge when Gaia 6 is released after RC phase.

@dongsam
Copy link
Member

dongsam commented Nov 4, 2021

Due to the possibility that vega includes farming and budget, branches and PRs were created separately to prepare and simulate for this. #5

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.

2 participants