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
Use validator protection datadir #7355
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7355 +/- ##
=======================================
Coverage 61.81% 61.81%
=======================================
Files 422 422
Lines 29866 29866
=======================================
Hits 18461 18461
Misses 8453 8453
Partials 2952 2952 |
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.
lets reuse our old flag, adding a new flag would just be confusing
@nisdas reused the same flag |
…sm into add_protection_db_flag
…sm into add_protection_db_flag
PTAL @prestonvanloon and @nisdas |
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.
PR looks good, had some minor cosmetic comments
…ection_db_flag # Conflicts: # validator/flags/flags.go # validator/main.go # validator/node/BUILD.bazel # validator/usage.go
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.
Please remove the checked in site data
PTAL @prestonvanloon |
This reverts commit ab76bda.
What type of PR is this?
Bug fix
why?
two reasons:
missing db file is a slashing protection problem. warn users and allow them to copy the db file if its empty
some users reported they don't have write permissions to wallet dir and need to configure a different location for it
What does this PR do? Why is it needed?
Which issues(s) does this PR fix?
Fixes #7331