From: Patrick Donnelly <pdonnell@redhat.com>
Date: Thu, 9 Jun 2022 16:03:30 +0000 (-0400)
Subject: mds: note weird balancer debug behavior
X-Git-Tag: v18.0.0~580^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=0113aebdc22f7a21c8c771838b7ea5ee44f3d80a;p=ceph.git

mds: note weird balancer debug behavior

Fixes: https://tracker.ceph.com/issues/55980
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
---

diff --git a/src/mds/MDBalancer.cc b/src/mds/MDBalancer.cc
index cb63f6054442..0e078a3fc4ee 100644
--- a/src/mds/MDBalancer.cc
+++ b/src/mds/MDBalancer.cc
@@ -37,6 +37,11 @@ using namespace std;
 #include "common/config.h"
 #include "common/errno.h"
 
+/* Note, by default debug_mds_balancer is 1/5. For debug messages 1<lvl<=5,
+ * should_gather (below) will be true; so, debug_mds will be ignored even if
+ * set to 20/20. For this reason, some messages may not appear in the log.
+ * Increase both debug levels to get expected output!
+ */
 #define dout_context g_ceph_context
 #undef dout_prefix
 #define dout_prefix *_dout << "mds." << mds->get_nodeid() << ".bal " << __func__ << " "
diff --git a/src/mds/Mantle.cc b/src/mds/Mantle.cc
index 653d47d6665e..e9033bbd94c1 100644
--- a/src/mds/Mantle.cc
+++ b/src/mds/Mantle.cc
@@ -19,6 +19,11 @@
 #include "common/Clock.h"
 #include "CInode.h"
 
+/* Note, by default debug_mds_balancer is 1/5. For debug messages 1<lvl<=5,
+ * should_gather (below) will be true; so, debug_mds will be ignored even if
+ * set to 20/20. For this reason, some messages may not appear in the log.
+ * Increase both debug levels to get expected output!
+ */
 #define dout_context g_ceph_context
 #undef dout_prefix
 #define dout_prefix *_dout << "mds.mantle "