simplify get all log filepaths and add cmt

engine/server/engine/centralized_logs/client_implementations/persistent_volume/log_file_manager/log_file_manager.go

@@ -264,17 +264,18 @@ func (manager *LogFileManager) createSymlinkLogFile(targetLogFilePath, symlinkLo
 	return nil
 }
 
+// if [enclaveUuid] is empty, gets log file paths from all enclaves
 func (manager *LogFileManager) getAllLogFilePaths(enclaveUuid string) ([]string, error) {
 	var paths []string
 	walkFunc := func(path string, info fs.FileInfo, err error) error {
-		if enclaveUuid != emptyEnclaveUuid {
-			if strings.Contains(path, enclaveUuid) && !info.IsDir() {
-				paths = append(paths, path)
-			}
-		} else {
-			if !info.IsDir() {
-				paths = append(paths, path)
-			}
+		if err != nil {
+			return err
+		}
+		if info.IsDir() {
+			return nil
+		}
+		if enclaveUuid == emptyEnclaveUuid || strings.Contains(path, enclaveUuid) {
+			paths = append(paths, path)
 		}
 		return nil
 	}

engine/server/engine/centralized_logs/client_implementations/persistent_volume/persistent_volume_helpers/persistent_volume_helpers.go

@@ -2,7 +2,10 @@ package persistent_volume_helpers
 
 import "time"
 
+const (
+	hoursInWeek = 7 * 24
+)
+
 func ConvertWeeksToDuration(weeks int) time.Duration {
-	hoursInWeek := 7 * 24
 	return time.Duration(weeks*hoursInWeek) * time.Hour
 }