Skip to content

[Feature] Add new configuration to specify the snapshot directory#944

Merged
yzang2019 merged 15 commits intomainfrom
yzang/SEI-4777
Jun 30, 2023
Merged

[Feature] Add new configuration to specify the snapshot directory#944
yzang2019 merged 15 commits intomainfrom
yzang/SEI-4777

Conversation

@yzang2019
Copy link
Contributor

Describe your changes and provide context

Changes:

  1. Add a new config snapshot-directory in app.toml to override default directory
  2. If not specified, default to the previous behavior
  3. Regroup integration test

Testing performed to validate your change

Added integration test:

  1. Validate creating a new snapshot in a customize directory
  2. Validate spin up an rpc node from the snapshot

yzang2019 added a commit to sei-protocol/sei-cosmos that referenced this pull request Jun 30, 2023
## Describe your changes and provide context
This change add a new config `snapshot-directory` in app.toml to
override default directory for storing state sync snapshot.

## Testing performed to validate your change
Tests are covered in sei-protocol/sei-chain#944

---------

Co-authored-by: Yiming Zang <yzang@twitter.com>
@yzang2019 yzang2019 merged commit 9bdc2b8 into main Jun 30, 2023
@yzang2019 yzang2019 deleted the yzang/SEI-4777 branch June 30, 2023 21:05
masih pushed a commit that referenced this pull request Sep 29, 2025
## Describe your changes and provide context
This change add a new config `snapshot-directory` in app.toml to
override default directory for storing state sync snapshot.

## Testing performed to validate your change
Tests are covered in #944

---------

Co-authored-by: Yiming Zang <yzang@twitter.com>
masih pushed a commit that referenced this pull request Sep 30, 2025
## Describe your changes and provide context
This change add a new config `snapshot-directory` in app.toml to
override default directory for storing state sync snapshot.

## Testing performed to validate your change
Tests are covered in #944

---------

Co-authored-by: Yiming Zang <yzang@twitter.com>
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.

3 participants