]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
rgw: use rgw::BucketReshardState for reshard state change
authorShilpa Jagannath <smanjara@redhat.com>
Mon, 29 Jun 2020 19:17:53 +0000 (00:47 +0530)
committerAdam C. Emerson <aemerson@redhat.com>
Mon, 13 Sep 2021 16:27:49 +0000 (12:27 -0400)
Signed-off-by: Shilpa Jagannath <smanjara@redhat.com>
src/rgw/rgw_reshard.cc

index e1a13e14eee0e305251c0fdf22c403eb938a4794..9adc89d3541c6f444c514e2f9c1ce46cf4421069 100644 (file)
@@ -402,7 +402,7 @@ public:
       }
       set_status(rgw::BucketReshardState::None, dpp);
     }
-  }
+  } 
 
   int start() {
     int ret = set_status(rgw::BucketReshardState::InProgress, dpp);
@@ -668,12 +668,6 @@ int RGWBucketReshard::do_reshard(int num_shards,
       return ret;
   }
 
-  ret = bucket_info_updater.complete();
-  if (ret < 0) {
-    ldout(store->ctx(), 0) << __func__ << ": failed to update bucket info ret=" << ret << dendl;
-    /* don't error out, reshard process succeeded */
-  }
-
   return 0;
   // NB: some error clean-up is done by ~BucketInfoReshardUpdate
 } // RGWBucketReshard::do_reshard
@@ -707,14 +701,6 @@ int RGWBucketReshard::execute(int num_shards, int max_op_entries,
     }
   }
 
-  // set resharding status of current bucket_info & shards with
-  // information about planned resharding
-  ret = set_resharding_status(dpp, cls_rgw_reshard_status::IN_PROGRESS);
-  if (ret < 0) {
-    return ret;
-    goto error_out;
-  }
-
   ret = do_reshard(num_shards,
                   max_op_entries,
                    verbose, out, formatter, dpp);