-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
feat(rpc): introduce and use setmnthreadactive
(regtest-only)
#6286
feat(rpc): introduce and use setmnthreadactive
(regtest-only)
#6286
Conversation
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.
LGTM, ACK 1e17b74
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.
utACK 1e17b74
{ | ||
|
||
if (Params().NetworkIDString() != CBaseChainParams::REGTEST) { | ||
throw std::runtime_error("setmnthreadactive is for regression testing (-regtest mode) only."); |
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.
should not it be JSONRPCError?
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.
For whatever reason all such messages are thrown this way 🤷♂️
https://github.com/search?q=repo%3Abitcoin%2Fbitcoin+%22%28-regtest+mode%29%22&type=code
Issue being fixed or feature implemented
This adds a new rpc command to enable/disable automatic masternode connections creation. We need this for #6276. 1e17b74 is extracted from ede1833 to avoid multiple jobs calling
setmnthreadactive
on the same node in parallel.What was done?
Add
setmnthreadactive
rpc and use itHow Has This Been Tested?
run tests
Breaking Changes
n/a
Checklist: