Fix repository cache regression

If implemented this make sure we don't unload an index before caching it
during a chart build phase.

Signed-off-by: Soule BA <[email protected]>

Author: souleb

controllers/helmchart_controller_test.go

@@ -159,6 +159,9 @@ func TestHelmChartReconciler_Reconcile(t *testing.T) {
 		return obj.Status.LastHandledReconcileAt == "now"
 	}, timeout).Should(BeTrue())
 
+	// Check if the cache contains the index.
+	g.Expect(testCache.ItemCount()).To(Equal(1))
+
 	g.Expect(testEnv.Delete(ctx, obj)).To(Succeed())
 
 	// Wait for HelmChart to be deleted

controllers/suite_test.go

@@ -103,6 +103,7 @@ var (
 
 var (
 	testRegistryServer *registryClientTestServer
+	testCache          *cache.Cache
 )
 
 func init() {
@@ -246,15 +247,15 @@ func TestMain(m *testing.M) {
 		panic(fmt.Sprintf("Failed to start HelmRepositoryOCIReconciler: %v", err))
 	}
 
-	c := cache.New(5, 1*time.Second)
+	testCache = cache.New(5, 1*time.Second)
 	cacheRecorder := cache.MustMakeMetrics()
 	if err := (&HelmChartReconciler{
 		Client:        testEnv,
 		EventRecorder: record.NewFakeRecorder(32),
 		Metrics:       testMetricsH,
 		Getters:       testGetters,
 		Storage:       testStorage,
-		Cache:         c,
+		Cache:         testCache,
 		TTL:           1 * time.Second,
 		CacheRecorder: cacheRecorder,
 	}).SetupWithManager(testEnv); err != nil {

internal/helm/chart/builder_remote.go

@@ -186,7 +186,6 @@ func (b *remoteChartBuilder) downloadFromRepository(remote *repository.ChartRepo
 		err = fmt.Errorf("could not load repository index for remote chart reference: %w", err)
 		return nil, &BuildError{Reason: ErrChartPull, Err: err}
 	}
-	defer remote.Unload()
 
 	// Get the current version for the RemoteReference
 	cv, err := remote.Get(remoteRef.Name, remoteRef.Version)