]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mds: do not try to diagnose cause of MDSMap removal
authorPatrick Donnelly <pdonnell@redhat.com>
Wed, 6 Nov 2019 08:26:51 +0000 (00:26 -0800)
committerPatrick Donnelly <pdonnell@redhat.com>
Fri, 8 Nov 2019 18:02:11 +0000 (10:02 -0800)
The Monitors send an empty MDSMap to an MDS it is removing. The MDS
can't diagnose the cause. Instead suggest looking at the cluster/monitor
logs.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
src/mds/MDSDaemon.cc

index 6e32a588d5bc39d0ed02d9a03b18dbfbfaebac84..6e65323ce02735159e1092690ba3bda427d8ad04 100644 (file)
@@ -794,25 +794,8 @@ void MDSDaemon::handle_mds_map(const cref_t<MMDSMap> &m)
   using DS = MDSMap::DaemonState;
   if (old_state != DS::STATE_NULL && new_state == DS::STATE_NULL) {
     const auto& oldinfo = oldmap->get_info_gid(mygid);
-    const auto existing = mdsmap->find_mds_gid_by_name(name);
-    if (g_conf()->mds_enforce_unique_name && existing != MDS_GID_NONE) {
-      const auto& info = mdsmap->get_info_gid(existing);
-      dout(1) << "Map replaced me " << oldinfo
-              << " with another MDS of the same name: " << info
-              << "; quitting!" << dendl;
-
-      /* Monitors should never go backwards! */
-      ceph_assert(info.global_id > myid);
-
-      // Call suicide() rather than respawn() because if someone else has
-      // taken our ID, we don't want to keep restarting and fighting them for
-      // the ID.
-      suicide();
-      return;
-    }
-
     dout(1) << "Map removed me " << oldinfo
-            << " from cluster due to lost contact; respawning" << dendl;
+            << " from cluster; respawning! See cluster/monitor logs for details." << dendl;
     respawn();
   }