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.

Sign up for GitHub

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

Jump to bottom

[3.2 -> main] Fix reporting of dirty db #710

Merged
merged 7 commits into from
Feb 13, 2023
Merged

[3.2 -> main] Fix reporting of dirty db #710

merged 7 commits into from
Feb 13, 2023

Conversation

heifner
Copy link
Member

@heifner heifner commented Feb 10, 2023

Fix issue with dirty db being reported as genesis not found instead of dirty db.

Merges release/3.2 into main including #700 & #708

Resolves #695
Resolves #565

@heifner heifner requested review from linh2931 and greg7mdp February 10, 2023 15:56
@heifner heifner added the OCI Work exclusive to OCI team label Feb 10, 2023
@heifner heifner merged commit 8fa4126 into main Feb 13, 2023
@heifner heifner deleted the GH-695-dirty-db-main branch February 13, 2023 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vladtr vladtr vladtr approved these changes

@greg7mdp greg7mdp greg7mdp approved these changes

@linh2931 linh2931 Awaiting requested review from linh2931

Assignees
No one assigned
Labels
OCI Work exclusive to OCI team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

dirty db not reported correctly nodeos should exit with exit code 2 if database is unclean, but doesn't
3 participants
@heifner @greg7mdp @vladtr