filestore loglevels cleanup (#2708)

ydb-platform · Dec 16, 2024 · d200ddd · d200ddd
1 parent b85f014
commit d200ddd
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 5 deletions.
diff --git a/cloud/filestore/libs/storage/tablet/shard_request_actor.h b/cloud/filestore/libs/storage/tablet/shard_request_actor.h
@@ -103,7 +103,7 @@ void TShardRequestActor<TRequest, TResponse>::SendRequests(
         request->Record = Request;
         request->Record.SetFileSystemId(shardId);
 
-        LOG_INFO(
+        LOG_DEBUG(
             ctx,
             TFileStoreComponents::TABLET_WORKER,
             "%s Sending %s to shard %s",
@@ -140,7 +140,7 @@ void TShardRequestActor<TRequest, TResponse>::HandleResponse(
         return;
     }
 
-    LOG_INFO(
+    LOG_DEBUG(
         ctx,
         TFileStoreComponents::TABLET_WORKER,
         "%s %s succeeded for shard %s",

diff --git a/cloud/filestore/libs/storage/tablet/tablet_actor_cleanupsessions.cpp b/cloud/filestore/libs/storage/tablet/tablet_actor_cleanupsessions.cpp
@@ -426,14 +426,23 @@ void TIndexTabletActor::HandleSyncSessionsCompleted(
         LogTag.c_str(),
         FormatError(msg->GetError()).c_str());
 
+    ui32 sessionsSynced = 0;
     for (const auto& shardSessionsInfo: msg->ShardSessionsInfos) {
-        LOG_INFO(ctx, TFileStoreComponents::TABLET,
+        LOG_DEBUG(ctx, TFileStoreComponents::TABLET,
             "%s Synced %lu sessions for shard %s",
             LogTag.c_str(),
             shardSessionsInfo.SessionCount,
             shardSessionsInfo.ShardId.c_str());
+
+        sessionsSynced += shardSessionsInfo.SessionCount;
     }
 
+    LOG_INFO(ctx, TFileStoreComponents::TABLET,
+        "%s Synced %u sessions for %lu shards",
+        LogTag.c_str(),
+        sessionsSynced,
+        msg->ShardSessionsInfos.size());
+
     WorkerActors.erase(ev->Sender);
     ScheduleSyncSessions(ctx);
 }

diff --git a/cloud/filestore/libs/storage/tablet/tablet_actor_counters.cpp b/cloud/filestore/libs/storage/tablet/tablet_actor_counters.cpp
@@ -92,7 +92,7 @@ void TGetShardStatsActor::SendRequests(const TActorContext& ctx)
         request->Record = Request;
         request->Record.SetFileSystemId(shardId);
 
-        LOG_INFO(
+        LOG_DEBUG(
             ctx,
             TFileStoreComponents::TABLET_WORKER,
             "%s Sending GetStorageStatsRequest to shard %s",

diff --git a/cloud/filestore/libs/storage/tablet/tablet_actor_createhandle.cpp b/cloud/filestore/libs/storage/tablet/tablet_actor_createhandle.cpp
@@ -428,7 +428,7 @@ void TIndexTabletActor::CompleteTx_CreateHandle(
     }
 
     if (args.OpLogEntry.HasCreateNodeRequest() && !HasError(args.Error)) {
-        LOG_INFO(ctx, TFileStoreComponents::TABLET,
+        LOG_DEBUG(ctx, TFileStoreComponents::TABLET,
             "%s Creating node in shard upon CreateHandle: %s, %s",
             LogTag.c_str(),
             args.ShardId.c_str(),

diff --git a/cloud/filestore/libs/storage/tablet/tablet_actor_renamenode.cpp b/cloud/filestore/libs/storage/tablet/tablet_actor_renamenode.cpp
@@ -425,6 +425,7 @@ void TIndexTabletActor::CompleteTx_RenameNode(
     if (!HasError(args.Error)) {
         auto& op = args.OpLogEntry;
         if (op.HasUnlinkNodeRequest()) {
+            // rename + unlink is pretty rare so let's keep INFO level here
             LOG_INFO(ctx, TFileStoreComponents::TABLET,
                 "%s Unlinking node in shard upon RenameNode: %s, %s",
                 LogTag.c_str(),