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

feat(v8)!: validators can have different genesis and self-delegation amounts (backport #1028) #1039

Merged
merged 2 commits into from
Mar 30, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Mar 30, 2024

Closes #1027

Summary

Some issues only arise when there's an unequal distribution of stake among validators. This PR allows interchaintest to create chains whose validators have different stake. It does this by changing the ModifyGenesisAmounts function in the chain config so that it takes in an index; the returned values will be used as the genesis and self-delegation amounts for the validator at that specific index.


This is an automatic backport of pull request #1028 done by Mergify.

…amounts (#1028)

* feat validators can have different genesis and self-delegation amounts

this is done by making ModifyGenesisAmounts take in an index parameter;
the returned values will be used for the validator at that index.

* Add a test for stake distribution in cosmos chains

(cherry picked from commit 2908114)
@mergify mergify bot requested a review from a team as a code owner March 30, 2024 22:09
@mergify mergify bot requested a review from misko9 March 30, 2024 22:09
Copy link

vercel bot commented Mar 30, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
interchaintest-docs ⬜️ Ignored (Inspect) Visit Preview Mar 30, 2024 10:12pm

@Reecepbcups Reecepbcups changed the title feat(v7)!: validators can have different genesis and self-delegation amounts (backport #1028) feat(v8)!: validators can have different genesis and self-delegation amounts (backport #1028) Mar 30, 2024
@Reecepbcups Reecepbcups enabled auto-merge (squash) March 30, 2024 22:12
@Reecepbcups Reecepbcups merged commit 5f41f4b into main Mar 30, 2024
19 checks passed
@Reecepbcups Reecepbcups deleted the mergify/bp/main/pr-1028 branch March 30, 2024 22:26
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.

feat: Allow unequal distribution of stake among validators
2 participants