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

Minimum Commission for proposal 826 #2768

Closed
4 tasks
Tracked by #2790
brianmori opened this issue Oct 21, 2023 · 3 comments · Fixed by #2852 or #2855
Closed
4 tasks
Tracked by #2790

Minimum Commission for proposal 826 #2768

brianmori opened this issue Oct 21, 2023 · 3 comments · Fixed by #2852 or #2855
Assignees
Labels
type: feature-request New feature or request improvement

Comments

@brianmori
Copy link

Summary

The proposal 826 has passed. The validator minimum commission should be enforced onchain.

Problem Definition

The MsgEditValidator and MsgCreateValidator allow any commission value. It should have a min value of 5%.
An existing issue was raised time ago: #1697

Proposal

An existing PR was developed in the past. Resolves #1698

Task list

Must have

Nice to have


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
  • Is a spike necessary to map out how the issue should be approached?
@brianmori brianmori added type: feature-request New feature or request improvement admin: epic An EPIC -- meta issue used to track a body of work status: waiting-triage This issue/PR has not yet been triaged by the team. labels Oct 21, 2023
@github-project-automation github-project-automation bot moved this to 🩹 F1: Triage in Cosmos Hub Oct 21, 2023
@mmulji-ic mmulji-ic removed the status: waiting-triage This issue/PR has not yet been triaged by the team. label Oct 31, 2023
@mmulji-ic
Copy link
Contributor

Thanks @brianmori for the reminder and linking in the original PR. We try to get this merged for the next release.

@mpoke mpoke moved this from 🩹 F1: Triage to 📥 F2: Todo in Cosmos Hub Nov 9, 2023
@mpoke
Copy link
Contributor

mpoke commented Nov 9, 2023

Due to the timeline of the next release (i.e., a proposal of v14.0.0 is expected to be submitted on the Hub by Nov 17), this feature will go into Gaia v15. Since we want to avoid releases around the winter holidays, v15 is scheduled for January 2024. Also, v15 will most likely update SDK to 0.47, which enables setting the minimum commission through a gov param. Therefore, in v15 we'll just set the default of this param to 5%.

Note though that just setting the param to 5% will not change the commission of existing validators. The param affects only the MsgEditValidator and MsgCreateValidator messages. To enforce the existing validators to have a minimum commission, migration code needs to be written.

@mpoke mpoke removed the admin: epic An EPIC -- meta issue used to track a body of work label Nov 9, 2023
@toschdev
Copy link
Contributor

toschdev commented Nov 29, 2023

It's not so complicated actually

https://github.com/CosmosContracts/juno/pull/81/files

@sainoe sainoe moved this from 📥 F2: Todo to 👀 F3: InReview in Cosmos Hub Dec 19, 2023
@sainoe sainoe linked a pull request Dec 19, 2023 that will close this issue
8 tasks
@mpoke mpoke linked a pull request Jan 4, 2024 that will close this issue
8 tasks
@github-project-automation github-project-automation bot moved this from 👀 F3: InReview to 👍 F4: Assessment in Cosmos Hub Jan 17, 2024
@mpoke mpoke reopened this Jan 17, 2024
@github-project-automation github-project-automation bot moved this from 👍 F4: Assessment to 🩹 F1: Triage in Cosmos Hub Jan 17, 2024
@mpoke mpoke moved this from 🩹 F1: Triage to 🏗 F3: InProgress in Cosmos Hub Jan 17, 2024
@sainoe sainoe moved this from 🏗 F3: InProgress to 👀 F3: InReview in Cosmos Hub Jan 18, 2024
@github-project-automation github-project-automation bot moved this from 👀 F3: InReview to 👍 F4: Assessment in Cosmos Hub Jan 19, 2024
@mpoke mpoke moved this from 👍 F4: Assessment to ✅ Done in Cosmos Hub Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature-request New feature or request improvement
Projects
Status: ✅ Done
5 participants