mon/MDSMonitor: do not generate mdsmaps from already-laggy mds

There is one path where a mds that is not sending its beacon (e.g.,
because it is not running at all) will lead to proposal of new mdsmaps.
Fix it.

Backport: emperor, dumpling
Signed-off-by: Sage Weil <sage@inktank.com>

diff --git a/src/mon/MDSMonitor.cc b/src/mon/MDSMonitor.cc
index 31151848a86bf2a02f4b5eaee4ab543d1715f369..6876c1e6eb60cc65785f95bee73158f9cd0e5185 100644 (file)
--- a/src/mon/MDSMonitor.cc
+++ b/src/mon/MDSMonitor.cc
@@ -1256,14 +1256,15 @@ void MDSMonitor::tick()
                   << " " << ceph_mds_state_name(info.state)
                   << " (laggy)" << dendl;
          pending_mdsmap.mds_info.erase(gid);
+         do_propose = true;
        } else if (!info.laggy()) {
          dout(10) << " marking " << gid << " " << info.addr << " mds." << info.rank << "." << info.inc
                   << " " << ceph_mds_state_name(info.state)
                   << " laggy" << dendl;
          info.laggy_since = now;
+         do_propose = true;
        }
        last_beacon.erase(gid);
-       do_propose = true;
       }
     }