]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
rgw: prevent bucket reshard scheduling if bucket is resharding 31299/head
authorJ. Eric Ivancich <ivancich@redhat.com>
Fri, 27 Sep 2019 00:54:42 +0000 (20:54 -0400)
committerNathan Cutler <ncutler@suse.com>
Thu, 31 Oct 2019 18:06:39 +0000 (19:06 +0100)
radosgw-admin should prevent a bucket from being added to the reshard
list if it is currently being resharded.

Signed-off-by: J. Eric Ivancich <ivancich@redhat.com>
(cherry picked from commit f24a7eecdea70ff971f24f6f12748db80b1a43b2)

src/rgw/rgw_admin.cc

index 367bf746514d8fc73b65e8b2c018f577f54da69e..068d900865fd325d630a43f6c044ce29f7b431e9 100644 (file)
@@ -23,6 +23,7 @@
 
 #include "include/util.h"
 
+#include "cls/rgw/cls_rgw_types.h"
 #include "cls/rgw/cls_rgw_client.h"
 
 #include "global/global_init.h"
@@ -2564,6 +2565,13 @@ int check_reshard_bucket_params(RGWRados *store,
     return ret;
   }
 
+  if (bucket_info.reshard_status != CLS_RGW_RESHARD_NOT_RESHARDING) {
+    // if in_progress or done then we have an old BucketInfo
+    cerr << "ERROR: the bucket is currently undergoing resharding and "
+      "cannot be added to the reshard list at this time" << std::endl;
+    return -EBUSY;
+  }
+
   int num_source_shards = (bucket_info.num_shards > 0 ? bucket_info.num_shards : 1);
 
   if (num_shards <= num_source_shards && !yes_i_really_mean_it) {