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 support for state snapshots #327

Closed
wants to merge 9 commits into from
Closed

Add support for state snapshots #327

wants to merge 9 commits into from

Conversation

erikgrinaker
Copy link

Description

Adds support for state snapshots, based on cosmos/cosmos-sdk#5803.

Would appreciate suggestions for improvements here, primarily how to avoid panics for errors, and if it might be possible to do this setup automagically in the Cosmos SDK (main problem: supplying the home directory path to the base app without changing AppCreator).


For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

For admin use:

  • Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
  • Reviewers assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

erikgrinaker added a commit to tendermint/tendermint that referenced this pull request Apr 29, 2020
Fixes #828. Adds state sync, as outlined in [ADR-053](https://github.com/tendermint/tendermint/blob/master/docs/architecture/adr-053-state-sync-prototype.md). See related PRs in Cosmos SDK (cosmos/cosmos-sdk#5803) and Gaia (cosmos/gaia#327).

This is split out of the previous PR #4645, and branched off of the ABCI interface in #4704. 

* Adds a new P2P reactor which exchanges snapshots with peers, and bootstraps an empty local node from remote snapshots when requested.

* Adds a new configuration section `[statesync]` that enables state sync and configures the light client. Also enables `statesync:info` logging by default.

* Integrates state sync into node startup. Does not support the v2 blockchain reactor, since it needs some reorganization to defer startup.
tac0turtle pushed a commit to tendermint/tendermint that referenced this pull request Apr 29, 2020
Fixes #828. Adds state sync, as outlined in [ADR-053](https://github.com/tendermint/tendermint/blob/master/docs/architecture/adr-053-state-sync-prototype.md). See related PRs in Cosmos SDK (cosmos/cosmos-sdk#5803) and Gaia (cosmos/gaia#327).

This is split out of the previous PR #4645, and branched off of the ABCI interface in #4704.

* Adds a new P2P reactor which exchanges snapshots with peers, and bootstraps an empty local node from remote snapshots when requested.

* Adds a new configuration section `[statesync]` that enables state sync and configures the light client. Also enables `statesync:info` logging by default.

* Integrates state sync into node startup. Does not support the v2 blockchain reactor, since it needs some reorganization to defer startup.
@stale
Copy link

stale bot commented May 24, 2020

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label May 24, 2020
@fedekunze fedekunze added the pinned Pinned issues that won't be closed by stalebot label May 24, 2020
@stale stale bot removed the stale label May 24, 2020
@tac0turtle
Copy link
Member

is it okay to close this PR?

@erikgrinaker
Copy link
Author

Sure. We’re going to have to update Gaia at some point, but it’s outdated and all kinds of broken so I’ll submit a new one when appropriate.

@tac0turtle tac0turtle deleted the erik/snapshot branch February 2, 2021 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pinned Pinned issues that won't be closed by stalebot
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants