From ee3fed474db766d085546028875afd6b6c656cc7 Mon Sep 17 00:00:00 2001
From: Klaus Aehlig <klaus.aehlig@huawei.com>
Date: Thu, 21 Mar 2024 11:16:40 +0100
Subject: Reduce log level of common operations

In general, in the log produced by `just serve`, we want at most one
entry at INFO level or higher per successfull request. Therefore,
downgrade the log level of all operations that are carried out in
a loop over all repositories that just serve takes care of.
---
 src/buildtool/serve_api/serve_service/source_tree.cpp | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

(limited to 'src/buildtool/serve_api/serve_service/source_tree.cpp')

diff --git a/src/buildtool/serve_api/serve_service/source_tree.cpp b/src/buildtool/serve_api/serve_service/source_tree.cpp
index 90de9f0e..2674b3d7 100644
--- a/src/buildtool/serve_api/serve_service/source_tree.cpp
+++ b/src/buildtool/serve_api/serve_service/source_tree.cpp
@@ -115,7 +115,7 @@ auto SourceTreeService::GetSubtreeFromCommit(
                             commit,
                             repo_path.string(),
                             msg);
-                        logger->Emit(LogLevel::Info, err);
+                        logger->Emit(LogLevel::Debug, err);
                     }
                 });
             return repo->GetSubtreeFromCommit(commit, subdir, wrapped_logger);
@@ -143,7 +143,7 @@ auto SourceTreeService::GetSubtreeFromTree(
                             tree_id,
                             repo_path.string(),
                             msg);
-                        logger->Emit(LogLevel::Info, err);
+                        logger->Emit(LogLevel::Debug, err);
                     }
                 });
             if (auto subtree_id =
@@ -172,7 +172,7 @@ auto SourceTreeService::GetBlobFromRepo(std::filesystem::path const& repo_path,
                             blob_id,
                             repo_path.string(),
                             msg);
-                        logger->Emit(LogLevel::Info, err);
+                        logger->Emit(LogLevel::Debug, err);
                     }
                 });
             auto res = repo->TryReadBlob(blob_id, wrapped_logger);
@@ -183,7 +183,7 @@ auto SourceTreeService::GetBlobFromRepo(std::filesystem::path const& repo_path,
                 auto str = fmt::format("Blob {} not found in repository {}",
                                        blob_id,
                                        repo_path.string());
-                logger->Emit(LogLevel::Info, str);
+                logger->Emit(LogLevel::Debug, str);
                 return false;  // non-fatal failure
             }
             return res.second.value();
@@ -191,7 +191,7 @@ auto SourceTreeService::GetBlobFromRepo(std::filesystem::path const& repo_path,
     }
     // failed to open repository
     logger->Emit(
-        LogLevel::Info,
+        LogLevel::Debug,
         fmt::format("Failed to open repository {}", repo_path.string()));
     return true;  // fatal failure
 }
@@ -645,7 +645,7 @@ auto SourceTreeService::IsTreeInRepo(std::string const& tree_id,
                             tree_id,
                             repo_path.string(),
                             msg);
-                        logger->Emit(LogLevel::Info, err);
+                        logger->Emit(LogLevel::Debug, err);
                     }
                 });
             return repo->CheckTreeExists(tree_id, wrapped_logger);
@@ -653,7 +653,7 @@ auto SourceTreeService::IsTreeInRepo(std::string const& tree_id,
     }
     // failed to open repository
     logger->Emit(
-        LogLevel::Info,
+        LogLevel::Debug,
         fmt::format("Failed to open repository {}", repo_path.string()));
     return std::nullopt;
 }
-- 
cgit v1.2.3