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
[Merged by Bors] - check if the slashing protection database is locked before creating keys #1949
[Merged by Bors] - check if the slashing protection database is locked before creating keys #1949
Conversation
@michaelsproul you're the expert in these parts :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
I just tested it with both create
and import
and it worked well 👍
I think it would be nice to have a simple test for test_transaction
, but I won't block on it
Added a test 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect
bors r+
…eys (#1949) ## Issue Addressed Closes #1790 ## Proposed Changes Make a new method that creates an empty transaction with `TransactionBehavior::Exclusive` to check whether the slashing protection is locked. Call this method before attempting to create or import new validator keystores. ## Additional Info N/A Co-authored-by: realbigsean <seananderson33@gmail.com>
Pull request successfully merged into unstable. Build succeeded: |
Issue Addressed
Closes #1790
Proposed Changes
Make a new method that creates an empty transaction with
TransactionBehavior::Exclusive
to check whether the slashing protection is locked. Call this method before attempting to create or import new validator keystores.Additional Info
N/A