From 8a364a57a3d4586ae3c57b96e113e4ba5fe0a142 Mon Sep 17 00:00:00 2001
From: Mikhail Montsev <svartmetal@nebius.com>
Date: Fri, 25 Oct 2024 18:58:16 +0100
Subject: [PATCH] x[Filestore] add client ids of active sessions to error
 message when destroying filesystem (#2351)

---
 .../libs/storage/service/service_actor_destroyfs.cpp     | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/cloud/filestore/libs/storage/service/service_actor_destroyfs.cpp b/cloud/filestore/libs/storage/service/service_actor_destroyfs.cpp
index 52c55dcc536..8aa62055504 100644
--- a/cloud/filestore/libs/storage/service/service_actor_destroyfs.cpp
+++ b/cloud/filestore/libs/storage/service/service_actor_destroyfs.cpp
@@ -112,9 +112,12 @@ void TDestroyFileStoreActor::HandleDescribeSessionsResponse(
     }
 
     if (msg->Record.SessionsSize() != 0) {
-        ReplyAndDie(
-            ctx,
-            MakeError(E_REJECTED, "FileStore has active sessions"));
+        TStringBuilder message;
+        message << "FileStore has active sessions with client ids:";
+        for (const auto& sessionInfo: msg->Record.GetSessions()) {
+            message << " " << sessionInfo.GetClientId();
+        }
+        ReplyAndDie(ctx, MakeError(E_REJECTED, message));
         return;
     }