]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
cls/rbd: fix mirror_image_map_list object
authorN Balachandran <nibalach@redhat.com>
Wed, 4 Dec 2024 11:40:13 +0000 (17:10 +0530)
committerN Balachandran <nibalach@redhat.com>
Wed, 4 Dec 2024 11:45:06 +0000 (17:15 +0530)
The rbd-mirror image map is stored in the rbd_mirror_leader object.
Fixed mirror_image_map_list() to operate on the correct object.

Fixes: https://tracker.ceph.com/issues/69112
Signed-off-by: N Balachandran <nibalach@redhat.com>
src/cls/rbd/cls_rbd_client.cc
src/test/cls_rbd/test_cls_rbd.cc

index 559ac221f89ad4f84b485ad3c66db428b84231e4..fca2d0ea18b7a07e21f552b95d0c38fa0a28039d 100644 (file)
@@ -2538,7 +2538,7 @@ int mirror_image_map_list(
   mirror_image_map_list_start(&op, start_after, max_read);
 
   bufferlist out_bl;
-  int r = ioctx->operate(RBD_MIRRORING, &op, &out_bl);
+  int r = ioctx->operate(RBD_MIRROR_LEADER, &op, &out_bl);
   if (r < 0) {
     return r;
   }
index 7eb03cc42f5422c32ace0d0e808bd26daf7fb966..4f39a8e1eabf8841fde5d106a2244ae052bd9c8f 100644 (file)
@@ -2156,7 +2156,7 @@ TEST_F(TestClsRbd, mirror_image_map)
 {
   librados::IoCtx ioctx;
   ASSERT_EQ(0, _rados.ioctx_create(_pool_name.c_str(), ioctx));
-  ioctx.remove(RBD_MIRRORING);
+  ioctx.remove(RBD_MIRROR_LEADER);
 
   std::map<std::string, cls::rbd::MirrorImageMap> image_mapping;
   ASSERT_EQ(-ENOENT, mirror_image_map_list(&ioctx, "", 0, &image_mapping));
@@ -2177,7 +2177,7 @@ TEST_F(TestClsRbd, mirror_image_map)
 
       mirror_image_map_update(&op, global_image_id, mirror_image_map);
     }
-    ASSERT_EQ(0, ioctx.operate(RBD_MIRRORING, &op));
+    ASSERT_EQ(0, ioctx.operate(RBD_MIRROR_LEADER, &op));
   }
 
   ASSERT_EQ(0, mirror_image_map_list(&ioctx, "", 1000, &image_mapping));
@@ -2203,7 +2203,7 @@ TEST_F(TestClsRbd, mirror_image_map)
   librados::ObjectWriteOperation op;
   mirror_image_map_remove(&op, "1");
   mirror_image_map_update(&op, "10", expected_mirror_image_map);
-  ASSERT_EQ(0, ioctx.operate(RBD_MIRRORING, &op));
+  ASSERT_EQ(0, ioctx.operate(RBD_MIRROR_LEADER, &op));
 
   ASSERT_EQ(0, mirror_image_map_list(&ioctx, "0", 1, &image_mapping));
   ASSERT_EQ(1U, image_mapping.size());