diff options
| author | Klaus Aehlig <klaus.aehlig@huawei.com> | 2024-03-21 11:16:40 +0100 |
|---|---|---|
| committer | Klaus Aehlig <klaus.aehlig@huawei.com> | 2024-03-21 11:16:40 +0100 |
| commit | ee3fed474db766d085546028875afd6b6c656cc7 (patch) | |
| tree | 94bbfec1d0098685b41973da6bc2a9f61ea90dfe | |
| parent | 20d9ae259f57e7f80919b1615de1b777f9b473ad (diff) | |
| download | justbuild-ee3fed474db766d085546028875afd6b6c656cc7.tar.gz | |
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.
| -rw-r--r-- | src/buildtool/serve_api/serve_service/source_tree.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
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; } |