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

Add a governance proposal to change client parameters #1171

Open
3 tasks
joneskm opened this issue Mar 25, 2022 · 2 comments
Open
3 tasks

Add a governance proposal to change client parameters #1171

joneskm opened this issue Mar 25, 2022 · 2 comments
Labels
07-tendermint type: feature New features, sub-features or integrations

Comments

@joneskm
Copy link

joneskm commented Mar 25, 2022

Summary

Add a governance proposal which allows client parameters to be changed

Problem Definition

There is no mechanism for changing client parameters, such as the trusting period, once a client has been created. In some cases it would benefit users of a client and the chain/relayer operators if the parameters could be changed e.g. if the trusting period was set to too low a value when a client was created both users and operators would benefit from increasing the trusting period.

It makes sense that these parameters cannot be changed unilaterally, however given that a majority of validators can change client parameter values anyway, having a built-in governance proposal to change them would not change the security fundamentals. Without a built-in governance proposal the maintainers must resort to makeshift and divergent changes to their codebase to force the client parameter values to change.

The disadvantage of reducing the friction for maintainers is that it might make it easier for them to inadvertently break the client for users relying on the original client parameters. Without the built-in governance proposal the maintainers would need to modify parts of the code themselves, arguably, this might lead them to consider more carefully the impact of changing parameter values.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged/assigned
@crodriguezvega crodriguezvega added the needs discussion Issues that need discussion before they can be worked on label Mar 25, 2022
@crodriguezvega
Copy link
Contributor

Adding here comment from #1758.

@colin-axner
Copy link
Contributor

We can do this following without adding additional interface functions following merge of #5806. It can be achieved by adding a msg server to the 07-tendermint light client module which allows governance to modify any of the fields in the client state

@colin-axner colin-axner added 07-tendermint and removed needs discussion Issues that need discussion before they can be worked on labels Feb 13, 2024
@crodriguezvega crodriguezvega added the type: feature New features, sub-features or integrations label Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
07-tendermint type: feature New features, sub-features or integrations
Projects
Status: Backlog 🕐
Development

No branches or pull requests

3 participants