From 60dc5873ef26f175152aa6ad81d39de76d2a4950 Mon Sep 17 00:00:00 2001
From: Vlad Ivanov <vlad@ivanov.email>
Date: Thu, 27 Jun 2024 11:25:02 +0200
Subject: [PATCH] Repack cruft in overlay repo (#1343)

--keep-unreachable keeps unreachable objects unpacked. over time, this
leads to creation of many loose objects, increasing i/o pressure.
instead, pack loose objects into cruft packs.

commit-id:edb8ab6d
---
 josh-core/src/housekeeping.rs | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/josh-core/src/housekeeping.rs b/josh-core/src/housekeeping.rs
index 86109ddf..88cc5157 100644
--- a/josh-core/src/housekeeping.rs
+++ b/josh-core/src/housekeeping.rs
@@ -322,6 +322,8 @@ pub fn get_known_filters() -> JoshResult<std::collections::BTreeMap<String, BTre
 }
 
 pub fn run(repo_path: &std::path::Path, do_gc: bool) -> JoshResult<()> {
+    const CRUFT_PACK_SIZE: usize = 1024 * 1024 * 64;
+
     let transaction_mirror = cache::Transaction::open(&repo_path.join("mirror"), None)?;
     let transaction_overlay = cache::Transaction::open(&repo_path.join("overlay"), None)?;
 
@@ -382,9 +384,11 @@ pub fn run(repo_path: &std::path::Path, do_gc: bool) -> JoshResult<()> {
                     "git",
                     "repack",
                     "-dn",
-                    "--keep-unreachable",
+                    "--cruft",
+                    &format!("--max-cruft-size={}", CRUFT_PACK_SIZE),
                     "--no-write-bitmap-index",
-                    "--threads=4"
+                    "--window-memory=128m",
+                    "--threads=4",
                 ]
             )
         );