]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
cls/rgw: fix bilog trim tests in ceph_test_cls_rgw 30268/head
authorCasey Bodley <cbodley@redhat.com>
Mon, 9 Sep 2019 13:42:42 +0000 (09:42 -0400)
committerCasey Bodley <cbodley@redhat.com>
Mon, 9 Sep 2019 14:14:51 +0000 (10:14 -0400)
the bucket index prepare step no longer writes a bilog entry, so the
tests were expecting more entries

Fixes: https://tracker.ceph.com/issues/41719
Signed-off-by: Casey Bodley <cbodley@redhat.com>
src/test/cls_rgw/test_cls_rgw.cc

index f824a29807df79d85c66d5bcbb4edf936631cb6b..e92a19b1681bc932cb6ee0b77ed96f934f89cd5f 100644 (file)
@@ -963,8 +963,8 @@ TEST_F(cls_rgw, bi_log_trim)
   {
     cls_rgw_bi_log_list_ret bilog;
     ASSERT_EQ(0, bilog_list(ioctx, bucket_oid, &bilog));
-    // prepare/complete/olh entry for each
-    EXPECT_EQ(30u, bilog.entries.size());
+    // complete/olh entry for each
+    EXPECT_EQ(20u, bilog.entries.size());
 
     bilog1.assign(std::make_move_iterator(bilog.entries.begin()),
                   std::make_move_iterator(bilog.entries.end()));
@@ -976,19 +976,19 @@ TEST_F(cls_rgw, bi_log_trim)
     ASSERT_EQ(0, bilog_trim(ioctx, bucket_oid, from, to));
     cls_rgw_bi_log_list_ret bilog;
     ASSERT_EQ(0, bilog_list(ioctx, bucket_oid, &bilog));
-    EXPECT_EQ(29u, bilog.entries.size());
+    EXPECT_EQ(19u, bilog.entries.size());
     EXPECT_EQ(bilog1[1].id, bilog.entries.begin()->id);
     ASSERT_EQ(-ENODATA, bilog_trim(ioctx, bucket_oid, from, to));
   }
   // trim back of bilog
   {
-    const std::string from = bilog1[28].id;
+    const std::string from = bilog1[18].id;
     const std::string to = "9";
     ASSERT_EQ(0, bilog_trim(ioctx, bucket_oid, from, to));
     cls_rgw_bi_log_list_ret bilog;
     ASSERT_EQ(0, bilog_list(ioctx, bucket_oid, &bilog));
-    EXPECT_EQ(28u, bilog.entries.size());
-    EXPECT_EQ(bilog1[28].id, bilog.entries.rbegin()->id);
+    EXPECT_EQ(18u, bilog.entries.size());
+    EXPECT_EQ(bilog1[18].id, bilog.entries.rbegin()->id);
     ASSERT_EQ(-ENODATA, bilog_trim(ioctx, bucket_oid, from, to));
   }
   // trim middle of bilog
@@ -998,7 +998,7 @@ TEST_F(cls_rgw, bi_log_trim)
     ASSERT_EQ(0, bilog_trim(ioctx, bucket_oid, from, to));
     cls_rgw_bi_log_list_ret bilog;
     ASSERT_EQ(0, bilog_list(ioctx, bucket_oid, &bilog));
-    EXPECT_EQ(27u, bilog.entries.size());
+    EXPECT_EQ(17u, bilog.entries.size());
     ASSERT_EQ(-ENODATA, bilog_trim(ioctx, bucket_oid, from, to));
   }
   // trim full bilog