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

Create a hardfork rolling upgrade for incentivized testnet #1594

Closed
evan-forbes opened this issue Apr 3, 2023 · 1 comment
Closed

Create a hardfork rolling upgrade for incentivized testnet #1594

evan-forbes opened this issue Apr 3, 2023 · 1 comment
Assignees

Comments

@evan-forbes
Copy link
Member

Part of the incnetivized testnet requires participating in some form of hardfork. We should create a special version of the app for this exact purpose, where we increment the app version at a hardcoded height, and use the app version to determine some state change. This could be a simple parameter change, or involve something larger.

@evan-forbes evan-forbes added the itn label Apr 3, 2023
@evan-forbes evan-forbes added this to the Mainnet milestone Apr 17, 2023
@evan-forbes evan-forbes self-assigned this Apr 17, 2023
evan-forbes added a commit that referenced this issue Apr 27, 2023
## Overview

This PR adds a very minimal version module, which will update the app
version in the header using end block based on a config. The configs are
just hardcoded maps atm, but its possible for a user to pass their own
via app options. This PR also isn't meant to be a final solution for
#1014 or #1568, but more of an exploratory demo of one way they could
done.

the majority of #1594 

## Checklist

- [x] New and updated code has appropriate documentation
- [x] New and updated code has new and/or updated testing
- [x] Required CI checks are passing
- [x] Visual proof for any user facing features like CLI or
documentation updates
- [x] Linked issues closed with keywords

---------

Co-authored-by: Rootul P <[email protected]>
Co-authored-by: rene <[email protected]>
evan-forbes added a commit that referenced this issue May 1, 2023
## Overview

This PR changes the `SignificantPowerDifferenceThreshold` qgb constant
depending on the app version. Note that this PR is targeting only the
v0.13.x branch.

closes #1594

## Checklist

- [x] New and updated code has appropriate documentation
- [x] New and updated code has new and/or updated testing
- [x] Required CI checks are passing
- [x] Visual proof for any user facing features like CLI or
documentation updates
- [x] Linked issues closed with keywords

---------

Co-authored-by: CHAMI Rachid <[email protected]>
Co-authored-by: Rootul P <[email protected]>
@evan-forbes
Copy link
Member Author

completed in #1672

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

1 participant