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

Inform and stop node if tier two initialization failed. #2756

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

furszy
Copy link

@furszy furszy commented Mar 7, 2022

Fixing two problems:

  1. If tier two initialization failed, shutdown the node at startup after informing the problem to the user. (right now, it could only fail due an invalid masternode configuration file).

  2. Do not error out at startup if the masternodes metadata manager was not loaded properly. The node will re-create its state all over while it's running.

@furszy furszy self-assigned this Mar 7, 2022
@Liquid369
Copy link
Member

tACK f273603
We can either rebase this into the new PR, or merge this as there is no conflicts between the two
Tested this while working on PR #2804

Copy link

@panleone panleone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK f273603
Tested on regtest with the full v6 update and this part is working fine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants