From 0368db95716e9047e0828b5299dd97ddb7ffeee1 Mon Sep 17 00:00:00 2001
From: Michael Habinsky <michael.habinsky@seagate.com>
Date: Thu, 14 Dec 2023 12:07:09 -0700
Subject: [PATCH] Keep file cache on shutdown.

---
 component/file_cache/file_cache.go | 4 +++-
 setup/baseConfig.yaml              | 2 +-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/component/file_cache/file_cache.go b/component/file_cache/file_cache.go
index 1119917ff..e16bd9a04 100644
--- a/component/file_cache/file_cache.go
+++ b/component/file_cache/file_cache.go
@@ -181,7 +181,9 @@ func (c *FileCache) Stop() error {
 	log.Trace("Stopping component : %s", c.Name())
 
 	_ = c.policy.ShutdownPolicy()
-	_ = c.TempCacheCleanup()
+	if !c.allowNonEmpty {
+		_ = c.TempCacheCleanup()
+	}
 
 	fileCacheStatsCollector.Destroy()
 
diff --git a/setup/baseConfig.yaml b/setup/baseConfig.yaml
index 7108e5122..0dcae0091 100644
--- a/setup/baseConfig.yaml
+++ b/setup/baseConfig.yaml
@@ -106,7 +106,7 @@ file_cache:
   low-threshold: <% disk space consumed which triggers eviction to stop when previously triggered by the high-threshold. Default - 60>
   create-empty-file: true|false <create an empty file on container when create call is received from kernel>
   allow-non-empty-temp: true|false <allow non empty temp directory at startup. Set true to persist local cache across reboots.>
-  cleanup-on-start: true|false <cleanup the temp directory on startup, if its not empty>
+  cleanup-on-start: true|false <cleanup the temp directory on startup, if it's not empty>
   policy-trace: true|false <generate eviction policy logs showing which files will expire soon>
   offload-io: true|false <by default libfuse will service reads/writes to files for better perf. Set to true to make file-cache component service read/write calls.>
   sync-to-flush: true|false <sync call to a file will force upload of the contents to storage account. Default - true>