osd: check fsid is normal before osd mkfs

Signed-off-by: song baisen <song.baisen@zte.com.cn>

diff --git a/src/ceph_osd.cc b/src/ceph_osd.cc
index 3565e2496798e68a30b1287545761b98ef57f8f0..eba88a86bedf54154411dc7a7667a42345ddbdf5 100644 (file)
--- a/src/ceph_osd.cc
+++ b/src/ceph_osd.cc
@@ -268,6 +268,11 @@ int main(int argc, const char **argv)
     if (mc.get_monmap_privately() < 0)
       return -1;
 
+    if (mc.monmap.fsid.is_zero()) {
+      derr << "must specify cluster fsid" << dendl;
+      return -EINVAL;
+    }
+
     int err = OSD::mkfs(g_ceph_context, store, g_conf->osd_data,
                        mc.monmap.fsid, whoami);
     if (err < 0) {

diff --git a/src/osd/OSD.cc b/src/osd/OSD.cc
index 2ac89d559fb2d51de444c0a95319a881ffc64945..8589df9cc8e81c3ffea30c5fae66bb65dfb49109 100644 (file)
--- a/src/osd/OSD.cc
+++ b/src/osd/OSD.cc
@@ -1531,12 +1531,6 @@ int OSD::mkfs(CephContext *cct, ObjectStore *store, const string &dev,
     }
   } else {
     // create superblock
-    if (fsid.is_zero()) {
-      derr << "must specify cluster fsid" << dendl;
-      ret = -EINVAL;
-      goto umount_store;
-    }
-
     sb.cluster_fsid = fsid;
     sb.osd_fsid = store->get_fsid();
     sb.whoami = whoami;