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

gui, util: Enhance verify checkpoints fail handling; use RegistryBookmarks for DB passivation #2758

Conversation

jamescowens
Copy link
Member

This changes the verify checkpoints fail handling to use the GUI reset blockchain facility if in the GUI, or direct the user to use the -resetblockchaindata option if using the daemon, rather than instructing the user to manually delete the blockchain data files.

This changes the verify checkpoints fail handling to use the
GUI reset blockchain facility if in the GUI, or direct the user
to use the -resetblockchaindata option if using the daemon,
rather than instructing the user to manually delete the blockchain
data files.
@jamescowens jamescowens requested a review from div72 April 5, 2024 20:35
@jamescowens jamescowens self-assigned this Apr 5, 2024
@jamescowens jamescowens added this to the Natasha milestone Apr 5, 2024
This will cause the wallet to ShowChainCorruptedMessage() and
either delete the blockchain files (in the GUI case) and end, or
post a log entry with instructions and end.

A missing pprev on any blockindex entry is fatal and the wallet should
not be allowed to continue operation without rebuilding the chain
data.
@jamescowens jamescowens changed the title gui, util: Enhance verify checkpoints fail handling gui, util: Enhance verify checkpoints fail handling, use RegistryBookmarks for DB passivation Apr 8, 2024
@jamescowens jamescowens changed the title gui, util: Enhance verify checkpoints fail handling, use RegistryBookmarks for DB passivation gui, util: Enhance verify checkpoints fail handling; use RegistryBookmarks for DB passivation Apr 8, 2024
Copy link
Member

@div72 div72 left a comment

Choose a reason for hiding this comment

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

utACK

@jamescowens jamescowens merged commit 1c31aa3 into gridcoin-community:development Apr 8, 2024
21 checks passed
jamescowens added a commit to jamescowens/Gridcoin-Research that referenced this pull request Apr 10, 2024
Added
 - build: add option for sanitizers gridcoin-community#2553 (@div72)
 - build: CMake: Initial Windows support (MSYS2) gridcoin-community#2733 (@CyberTailor)

Changed
 - build: enforce SSE2 on x86 targets gridcoin-community#2746 (@div72)
 - consensus: Update checkpoint data for mainnet and testnet gridcoin-community#2756 (@jamescowens)
 - gui, util: Enhance verify checkpoints fail handling; use RegistryBookmarks for DB passivation gridcoin-community#2758 (@jamescowens)

Removed

Fixed
 - build, depends: fix compilation with XCode 15 gridcoin-community#2747 (@div72)
 - Fix man page installation path for cmake builds gridcoin-community#2749 (@theMarix)
 - consensus, mrc, sidestake: add mrc fees to staker to rewards to be allocated via sidestaking gridcoin-community#2753 (@jamescowens)
 - Fix Systemd unit install location gridcoin-community#2754 (@theMarix)
 - scraper: Corrections to scraper_net after removal of cntPartsRcvd decrement and increment gridcoin-community#2755 (@jamescowens)
 - rpc: fix setban segfault gridcoin-community#2757 (@div72)
@jamescowens jamescowens deleted the enhance_checkpoint_fail_actions branch July 21, 2024 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants