]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
mon: print FSMap if standbys exist
authorPatrick Donnelly <pdonnell@redhat.com>
Tue, 17 Dec 2019 18:13:02 +0000 (10:13 -0800)
committerPatrick Donnelly <pdonnell@redhat.com>
Wed, 18 Dec 2019 00:10:48 +0000 (16:10 -0800)
Fixes: https://tracker.ceph.com/issues/42088
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
src/mon/MDSMonitor.h
src/mon/Monitor.cc

index 9f415fc24445bd87b79d2b2cdea3c1d7be35e453..1475e312d069928483c66d69a85260c8ba0eb9e4 100644 (file)
@@ -49,6 +49,13 @@ class MDSMonitor : public PaxosService, public PaxosFSMap, protected CommandHand
   bool prepare_update(MonOpRequestRef op) override;
   bool should_propose(double& delay) override;
 
+  bool should_print_status() const {
+    auto& fs = get_fsmap();
+    auto fs_count = fs.filesystem_count();
+    auto standby_count = fs.get_num_standby();
+    return fs_count > 0 || standby_count > 0;
+  }
+
   void on_active() override;
   void on_restart() override;
 
index 27ea54b0b327f3711ac81665bca4764719bb88e9..e8f3ab8fe2408b1193585c2aaab3777c36dd1790 100644 (file)
@@ -3005,8 +3005,8 @@ void Monitor::get_cluster_status(stringstream &ss, Formatter *f)
        mgrmon()->get_map().print_summary(nullptr, &ss);
        ss << "\n";
       }
-      if (mdsmon()->get_fsmap().filesystem_count() > 0) {
-       ss << "    mds: " << spacing << mdsmon()->get_fsmap() << "\n";
+      if (mdsmon()->should_print_status()) {
+        ss << "    mds: " << spacing << mdsmon()->get_fsmap() << "\n";
       }
       ss << "    osd: " << spacing;
       osdmon()->osdmap.print_summary(NULL, ss, string(maxlen + 6, ' '));