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

refactor: prune everything #11177

Merged
merged 9 commits into from
Feb 23, 2022
Merged

refactor: prune everything #11177

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
c904b15
updates
alexanderbez Feb 12, 2022
eb8079f
updates
alexanderbez Feb 12, 2022
b898619
cl++
alexanderbez Feb 12, 2022
f3ea18c
Merge branch 'master' into bez/10352-fix-prune-everything
alexanderbez Feb 14, 2022
6d0fadd
fix tests
alexanderbez Feb 22, 2022
4241ac4
fix conflicts
alexanderbez Feb 22, 2022
b614469
fix conflicts
alexanderbez Feb 22, 2022
22a10af
updsates
alexanderbez Feb 22, 2022
0f74645
Merge branch 'master' into bez/10352-fix-prune-everything
alexanderbez Feb 23, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
updsates
  • Loading branch information
@alexanderbez
alexanderbez committed Feb 22, 2022
commit 22a10af138b6a1a4ad4283e933b7365ef4c9e721
5 changes: 5 additions & 0 deletions server/config/config.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -318,6 +318,11 @@ func (c Config) ValidateBasic() error {
if c.BaseConfig.MinGasPrices == "" {
return sdkerrors.ErrAppConfig.Wrap("set min gas price in app.toml or flag or env variable")
}
if c.Pruning == storetypes.PruningOptionEverything && c.StateSync.SnapshotInterval > 0 {
return sdkerrors.ErrAppConfig.Wrapf(
"cannot enable state sync snapshots with '%s' pruning setting", storetypes.PruningOptionEverything,
)
}

return nil
}