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

Add configurable max-savepoints and savepoint-interval #4191

Merged
merged 5 commits into from
Jan 27, 2025

Conversation

emilcondrea
Copy link
Contributor

Testnet4 has deep reorgs, causing ord to end-up in unrecoverable state. It would be good to have configurable reorg environment so it can be more aggressive on testnet4 but also not risk too much on mainnet. @raphjaph, this is related to #4169

@emilcondrea
Copy link
Contributor Author

Looking a bit through existing reorgs on testnet4 some are of 15 blocks depth.

bitcoin-cli -chain=testnet4  getchaintips | jq -r '.[]|"\(.branchlen) \(.height)"'  | sort -n

...
8 65609
8 65609
8 65826
8 65927
8 66003
9 64845
9 65050
9 65334
9 65451
9 65451
9 65515
9 65515
9 65610
10 64846
10 65051
10 65051
10 65335
10 65452
10 65452
10 65452
10 65611
10 65611
10 65929
11 65336
11 65336
12 65613
12 65613
15 65616
15 65616

Copy link
Collaborator

@raphjaph raphjaph left a comment

Choose a reason for hiding this comment

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

LGTM

@raphjaph raphjaph enabled auto-merge (squash) January 27, 2025 14:28
@raphjaph raphjaph merged commit bfcd096 into ordinals:master Jan 27, 2025
5 checks passed
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.

2 participants