cephfs: Add some check befor we want to add new data pool

Signed-off-by: Vicente Cheng <vicente_cheng@bigtera.com>

diff --git a/src/mon/FSCommands.cc b/src/mon/FSCommands.cc
index b36443411d3f94a596c9da65ef5c97bd1e820987..b5b1146385700059e35f885845e939e9768c2d95 100644 (file)
--- a/src/mon/FSCommands.cc
+++ b/src/mon/FSCommands.cc
@@ -488,6 +488,12 @@ class AddDataPoolHandler : public FileSystemCommandHandler
       return r;
     }
 
+    // no-op when the data_pool already on fs
+    if (fs->mds_map.is_data_pool(poolid)) {
+      ss << "data pool " << poolid << " is already on fs " << fs_name;
+      return 0;
+    }
+
     fsmap.modify_filesystem(
         fs->fscid,
         [poolid](std::shared_ptr<Filesystem> fs)