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

Ledger cleanup fix (bp #9131) #9175

Merged
merged 1 commit into from
Mar 31, 2020
Merged

Ledger cleanup fix (bp #9131) #9175

merged 1 commit into from
Mar 31, 2020

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Mar 31, 2020

This is an automated backport of pull request #9131 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
  • @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 simulator.

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

* Fix purging happening every slot when cleanup service is not started at slot 0
* Purge by shred count instead of slots since slots can have variable
number of shreds

(cherry picked from commit 73e99cc)
@mergify mergify bot added the automerge Merge this Pull Request automatically once CI passes label Mar 31, 2020
@solana-grimes solana-grimes merged commit 8778eca into v1.0 Mar 31, 2020
@codecov
Copy link

codecov bot commented Mar 31, 2020

Codecov Report

Merging #9175 into v1.0 will increase coverage by <.1%.
The diff coverage is 95.3%.

@@           Coverage Diff           @@
##            v1.0   #9175     +/-   ##
=======================================
+ Coverage   80.3%   80.3%   +<.1%     
=======================================
  Files        266     266             
  Lines      57636   57726     +90     
=======================================
+ Hits       46304   46393     +89     
- Misses     11332   11333      +1

@mvines mvines deleted the mergify/bp/v1.0/pr-9131 branch April 4, 2020 02:53
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.

2 participants