Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Delete files older than the lowest_cleanup_slot in LedgerCleanupService::cleanup_ledger (backport #26651)#27304

Merged
mergify[bot] merged 1 commit intov1.11from
mergify/bp/v1.11/pr-26651
Aug 22, 2022
Merged

Delete files older than the lowest_cleanup_slot in LedgerCleanupService::cleanup_ledger (backport #26651)#27304
mergify[bot] merged 1 commit intov1.11from
mergify/bp/v1.11/pr-26651

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify Bot commented Aug 22, 2022

This is an automatic backport of pull request #26651 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

…ce::cleanup_ledger (#26651)

#### Problem
LedgerCleanupService requires compactions to propagate & digest range-delete tombstones
to eventually reclaim disk space.

#### Summary of Changes
This PR makes LedgerCleanupService::cleanup_ledger delete any file whose slot-range is
older than the lowest_cleanup_slot.  This allows us to reclaim disk space more often with
fewer IOps.  Experimental results on mainnet validators show that the PR can effectively
reduce 33% to 40% ledger disk size.

(cherry picked from commit 99ef218)
@mergify mergify Bot added the automerge Merge this Pull Request automatically once CI passes label Aug 22, 2022
@mergify
Copy link
Copy Markdown
Contributor Author

mergify Bot commented Aug 22, 2022

automerge label removed due to a CI failure

@mergify mergify Bot added automerge Merge this Pull Request automatically once CI passes and removed automerge Merge this Pull Request automatically once CI passes labels Aug 22, 2022
@mergify mergify Bot merged commit b1d5d6d into v1.11 Aug 22, 2022
@mergify mergify Bot deleted the mergify/bp/v1.11/pr-26651 branch August 22, 2022 20:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automerge Merge this Pull Request automatically once CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant