mds: derr log and dump sessionmap when reclaim session uuid isn't found

vshankar · vshankar · commit 9f5ca61293cc · 2024-06-04T10:44:34.000+05:30
If the old session uuid isn't found in sessionmap, nothing is
logeed. The client however returns -ENOENT to the caller when
no reclaim addrs are returned.

Signed-off-by: Venky Shankar &lt;vshankar@redhat.com&gt;
diff --git a/src/mds/Server.cc b/src/mds/Server.cc
@@ -472,6 +472,9 @@ void Server::reclaim_session(Session *session, const cref_t<MClientReclaim> &m)
     ceph_assert(!session->reclaiming_from);
     session->reclaiming_from = target;
     reply->set_addrs(entity_addrvec_t(target->info.inst.addr));
+  } else {
+    derr << ": could not find session by uuid:" << m->get_uuid() << dendl;
+    mds->sessionmap.dump();
   }
 
   if (flags & CEPH_RECLAIM_RESET) {

Original file line number	Diff line number	Diff line change
`@@ -472,6 +472,9 @@ void Server::reclaim_session(Session *session, const cref_t<MClientReclaim> &m)`
`472`	`472`	`ceph_assert(!session->reclaiming_from);`
`473`	`473`	`session->reclaiming_from = target;`
`474`	`474`	`reply->set_addrs(entity_addrvec_t(target->info.inst.addr));`
	`475`	`+ } else {`
	`476`	`+ derr << ": could not find session by uuid:" << m->get_uuid() << dendl;`
	`477`	`+ mds->sessionmap.dump();`
`475`	`478`	`}`
`476`	`479`
`477`	`480`	`if (flags & CEPH_RECLAIM_RESET) {`