mds: deprecation-warn on numeric mds names

To be turned into strict check in post-hammer
release, so that we can always unambiguously
discriminate a string into one of name, rank, gid.

Signed-off-by: John Spray <john.spray@redhat.com>

diff --git a/src/ceph_mds.cc b/src/ceph_mds.cc
index b1ab386cc6762b446c52051f66c0ee69c8416bdc..7f4e082de97f69d0126185bfadf4174c34e9efed 100644 (file)
--- a/src/ceph_mds.cc
+++ b/src/ceph_mds.cc
@@ -142,6 +142,12 @@ int main(int argc, const char **argv)
     usage();
   }
 
+  if (g_conf->name.get_id().empty() || (g_conf->name.get_id()[0] >= '0' && g_conf->name.get_id()[0] <= '9')) {
+    derr << "deprecation warning: MDS id '" << g_conf->name
+      << "' is invalid and will be forbidden in a future version.  "
+      "MDS names may not start with a numeric digit." << dendl;
+  }
+
   Messenger *messenger = Messenger::create(g_ceph_context,
                                           entity_name_t::MDS(-1), "mds",
                                           getpid());