Merge pull request ceph#53611 from Patty8122/adding-useful-print

rishabh-d-dave · web-flow · commit 23b3a55f3a0a · 2023-11-03T16:32:41.000+05:30
mds: Include more information in print statement in MDSRank.cc

Reviewed-by: Greg Farnum &lt;gfarnum@redhat.com&gt;
Reviewed-by: Anthony D'Atri &lt;anthony.datri@gmail.com&gt;
Reviewed-by: Dhairya Parmar &lt;dparmar@redhat.com&gt;
Reviewed-by: Rishabh Dave &lt;ridave@redhat.com&gt;
diff --git a/src/mds/MDSRank.cc b/src/mds/MDSRank.cc
@@ -3187,7 +3187,7 @@ void MDSRank::command_dump_tree(const cmdmap_t &cmdmap, std::ostream &ss, Format
   std::lock_guard l(mds_lock);
   CInode *in = mdcache->cache_traverse(filepath(root.c_str()));
   if (!in) {
-    ss << "root inode is not in cache";
+    ss << "inode for path '" << filepath(root.c_str()) << "' is not in cache";
     return;
   }
   f->open_array_section("inodes");

Original file line number	Diff line number	Diff line change
`@@ -3187,7 +3187,7 @@ void MDSRank::command_dump_tree(const cmdmap_t &cmdmap, std::ostream &ss, Format`
`3187`	`3187`	`std::lock_guard l(mds_lock);`
`3188`	`3188`	`CInode *in = mdcache->cache_traverse(filepath(root.c_str()));`
`3189`	`3189`	`if (!in) {`
`3190`		`- ss << "root inode is not in cache";`
	`3190`	`+ ss << "inode for path '" << filepath(root.c_str()) << "' is not in cache";`
`3191`	`3191`	`return;`
`3192`	`3192`	`}`
`3193`	`3193`	`f->open_array_section("inodes");`