cls/rbd: deprecate mirror_image_status_remove method

It has never been used by RBD mirroring and should be removed.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>

diff --git a/src/cls/rbd/cls_rbd.cc b/src/cls/rbd/cls_rbd.cc
index f09749399733ecb8f0af7808858a512dcd6db26a..fc6ad0df719307ddb5041f29173a18ae38bb133f 100644 (file)
--- a/src/cls/rbd/cls_rbd.cc
+++ b/src/cls/rbd/cls_rbd.cc
@@ -6021,6 +6021,8 @@ int mirror_image_status_set(cls_method_context_t hctx, bufferlist *in,
  *
  * Output:
  * @returns 0 on success, negative error code on failure
+ *
+ * NOTE: deprecated - remove this method after Octopus is unsupported
  */
 int mirror_image_status_remove(cls_method_context_t hctx, bufferlist *in,
                               bufferlist *out) {

diff --git a/src/cls/rbd/cls_rbd_client.cc b/src/cls/rbd/cls_rbd_client.cc
index 1dc1e7c7f2a61690988d7b68aa2b1443ba6f1e42..a8df333e3f79f59b694204cd92ec3b6a27c4da6b 100644 (file)
--- a/src/cls/rbd/cls_rbd_client.cc
+++ b/src/cls/rbd/cls_rbd_client.cc
@@ -2086,20 +2086,6 @@ void mirror_image_status_set(librados::ObjectWriteOperation *op,
   op->exec("rbd", "mirror_image_status_set", bl);
 }
 
-int mirror_image_status_remove(librados::IoCtx *ioctx,
-                               const std::string &global_image_id) {
-  librados::ObjectWriteOperation op;
-  mirror_image_status_remove(&op, global_image_id);
-  return ioctx->operate(RBD_MIRRORING, &op);
-}
-
-void mirror_image_status_remove(librados::ObjectWriteOperation *op,
-                                const std::string &global_image_id) {
-  bufferlist bl;
-  encode(global_image_id, bl);
-  op->exec("rbd", "mirror_image_status_remove", bl);
-}
-
 int mirror_image_status_get(librados::IoCtx *ioctx,
                             const std::string &global_image_id,
                             cls::rbd::MirrorImageStatus *status) {

diff --git a/src/cls/rbd/cls_rbd_client.h b/src/cls/rbd/cls_rbd_client.h
index 9ab63141d04f1c7ad6ed4bd7d7b086a45b4bc4bc..29337e7293bba0b98766ce62aae1ee722adb31f0 100644 (file)
--- a/src/cls/rbd/cls_rbd_client.h
+++ b/src/cls/rbd/cls_rbd_client.h
@@ -438,10 +438,6 @@ int mirror_image_status_set(librados::IoCtx *ioctx,
 void mirror_image_status_set(librados::ObjectWriteOperation *op,
                              const std::string &global_image_id,
                              const cls::rbd::MirrorImageStatus &status);
-int mirror_image_status_remove(librados::IoCtx *ioctx,
-                               const std::string &global_image_id);
-void mirror_image_status_remove(librados::ObjectWriteOperation *op,
-                                const std::string &global_image_id);
 int mirror_image_status_get(librados::IoCtx *ioctx,
                             const std::string &global_image_id,
                             cls::rbd::MirrorImageStatus *status);

diff --git a/src/test/cls_rbd/test_cls_rbd.cc b/src/test/cls_rbd/test_cls_rbd.cc
index e1f3834656a3ed4e7a4ff026ad5a1bc3fb4b3794..7be6a642a6c7100e2e364ffd8c426d6337e170c5 100644 (file)
--- a/src/test/cls_rbd/test_cls_rbd.cc
+++ b/src/test/cls_rbd/test_cls_rbd.cc
@@ -1939,24 +1939,6 @@ TEST_F(TestClsRbd, mirror_image_status) {
   ASSERT_EQ(1U, states.size());
   ASSERT_EQ(3, states[cls::rbd::MIRROR_IMAGE_STATUS_STATE_REPLAYING]);
 
-  // Test remove
-
-  ASSERT_EQ(0, mirror_image_status_remove(&ioctx, "uuid3"));
-  ASSERT_EQ(-ENOENT, mirror_image_status_get(&ioctx, "uuid3", &read_status));
-  images.clear();
-  statuses.clear();
-  ASSERT_EQ(0, mirror_image_status_list(&ioctx, "", 1024, &images, &statuses));
-  ASSERT_EQ(3U, images.size());
-  ASSERT_EQ(2U, statuses.size());
-  ASSERT_EQ(statuses["image_id1"], status1);
-  ASSERT_EQ(statuses["image_id2"], status2);
-
-  states.clear();
-  ASSERT_EQ(0, mirror_image_status_get_summary(&ioctx, &states));
-  ASSERT_EQ(2U, states.size());
-  ASSERT_EQ(1, states[cls::rbd::MIRROR_IMAGE_STATUS_STATE_UNKNOWN]);
-  ASSERT_EQ(2, states[cls::rbd::MIRROR_IMAGE_STATUS_STATE_REPLAYING]);
-
   // Test statuses are down after removing watcher
 
   ASSERT_EQ(0, mirror_image_status_set(&ioctx, "uuid1", status1));