Skip to content
This repository has been archived by the owner on May 22, 2023. It is now read-only.

Check operator balance before status update #570

Closed
wants to merge 4 commits into from

Conversation

lukasz-zimnoch
Copy link
Member

@lukasz-zimnoch lukasz-zimnoch commented Oct 5, 2020

Closes: #551

This pull request introduces a new MinimumKeyGenerationBalance config property which defines a minimum WEI balance below which the operator should not participate in new keeps. The main motivation is to avoid spending funds during the key generation protocol and hold them for signatures requests to mitigate the risk of bonds seizing by not having enough ETH to submit requested signatures to the chain.

Technically, the aforementioned behavior is achieved by checking the current operator's balance against the MinimumKeyGenerationBalance value and aborting the operator status update in the sortition pool in case the current balance is below the minimum. Not updating the operator status will force the sortition pool to omit the operator during group selection.

`MinimumKeyGenerationBalance` config option
allows to specify the minimum operator ETH
balance to participate in new keeps.
Check the operator balance against the
`MinimumKeyGenerationBalance` config property
before updating the status in the sortition pool.
This should prevent the operator being selected
to new signing groups in case when the operator
doesn't have enough funds to take part in the
key generation protocol.
@lukasz-zimnoch lukasz-zimnoch requested a review from pdyraga October 5, 2020 14:29
@lukasz-zimnoch
Copy link
Member Author

After a discussion with @pdyraga and @nkuba the approach presented here is not enough and it will require a lot of changes in the contract side. For now we limit ourselves to the monitoring feature. Closed in favour of #572.

@lukasz-zimnoch lukasz-zimnoch deleted the minimum-keygen-balance branch October 6, 2020 11:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Operator account balance monitoring
1 participant