From 4f49cc46daa7cde87ba71e14dda3f54cead3f939 Mon Sep 17 00:00:00 2001
From: YuviPanda <yuvipanda@gmail.com>
Date: Sat, 8 Mar 2025 12:40:52 -0800
Subject: [PATCH] Disable image cleaner

As part of https://github.com/jupyterhub/binderhub/pull/1941,
I want to not port imageCleaner to the new chart. Since the
image store is not really used to push anymore (since
https://github.com/jupyterhub/repo2docker/pull/1421) and we will
be able to disregard it completely in the future, we can simply
just prune the entire image store than do any complex logic here.
---
 mybinder/templates/buildkit-pruner.yaml | 1 +
 mybinder/values.yaml                    | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/mybinder/templates/buildkit-pruner.yaml b/mybinder/templates/buildkit-pruner.yaml
index de016a6f8..d7272f8a0 100644
--- a/mybinder/templates/buildkit-pruner.yaml
+++ b/mybinder/templates/buildkit-pruner.yaml
@@ -26,6 +26,7 @@ spec:
             - /bin/sh
             - -c
             - |
+              docker image prune ---force -all && \
               docker builder prune --force --all --keep-storage={{ .Values.buildkitPruner.buildkitCacheSize }} && \
               docker system df
             volumeMounts:
diff --git a/mybinder/values.yaml b/mybinder/values.yaml
index dd691558c..514428991 100644
--- a/mybinder/values.yaml
+++ b/mybinder/values.yaml
@@ -362,7 +362,7 @@ binderhub:
         memory: 4Gi
 
   imageCleaner:
-    enabled: true
+    enabled: false
     # Use 40GB as upper limit, size is given in bytes
     imageGCThresholdHigh: 40e9
     imageGCThresholdLow: 10e9