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

Clear process to update MinimumUntrustedScore #456

Open
kianenigma opened this issue Aug 2, 2022 · 2 comments · May be fixed by paritytech/substrate#14328
Open

Clear process to update MinimumUntrustedScore #456

kianenigma opened this issue Aug 2, 2022 · 2 comments · May be fixed by paritytech/substrate#14328
Assignees

Comments

@kianenigma
Copy link
Contributor

We deployed this last year, and it is a great security addition, but we need a better way to update it.

Currently I mostly do it manually, every quarter or so, through a proposal to the governance.

We need a new system where this value is updated automatically, either via bots offchain, or preferably via a simple onchain mechanism.

The onchain mechanism should still be overwritten by any governance controlled origin.

This is the blocker for paritytech/substrate#11703 and alike as well.

@kianenigma
Copy link
Contributor Author

an urgent sub-task of this is to table a proposal to kusama, reverting the existing values that are fairly low in response to a recent slashing incident.

Screenshot 2023-04-01 at 21 19 21

@gpestana
Copy link
Contributor

gpestana commented Apr 4, 2023

A simple and low overhead solution to update the MinimumUntrustedScore on-chain https://hackmd.io/IwJiYgMwRKer0e7khbLhUQ

@juangirini juangirini transferred this issue from paritytech/substrate Aug 24, 2023
helin6 pushed a commit to boolnetwork/polkadot-sdk that referenced this issue Feb 5, 2024
* Update ethereum style

* Fix conflict

* Remove useless type

* Update `on_finalize` to remove block_hash

* Fix compile issues
jonathanudd pushed a commit to jonathanudd/polkadot-sdk that referenced this issue Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ⌛️ Sometime-soon
Development

Successfully merging a pull request may close this issue.

4 participants