]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
test/librbd: fix required features for ViaLockOwner tests 20628/head
authorMykola Golub <mgolub@suse.com>
Wed, 21 Feb 2018 08:46:21 +0000 (10:46 +0200)
committerPrashant D <pdhange@redhat.com>
Wed, 28 Feb 2018 03:38:00 +0000 (22:38 -0500)
The rename operation is RPCed only when journaling is enabled.

The snap_remove operation is RPCed only when either fast-diff or
journaling is enabled.

The rename, snap_create, and snap_remove do not require layering.

Signed-off-by: Mykola Golub <mgolub@suse.com>
(cherry picked from commit 56e762b13aae302e6ab3cddb43f277d13ba6f43c)

src/test/librbd/test_librbd.cc

index 2f5d761a2ea60ab51d9bc11b7a3a17bf9b817433..4781dac99eadcdc215ae0420b78d32462d02d2eb 100644 (file)
@@ -4146,7 +4146,7 @@ TEST_F(TestLibRBD, RebuildObjectMapViaLockOwner)
 
 TEST_F(TestLibRBD, RenameViaLockOwner)
 {
-  REQUIRE_FEATURE(RBD_FEATURE_LAYERING | RBD_FEATURE_EXCLUSIVE_LOCK);
+  REQUIRE_FEATURE(RBD_FEATURE_JOURNALING);
 
   librados::IoCtx ioctx;
   ASSERT_EQ(0, _rados.ioctx_create(m_pool_name.c_str(), ioctx));
@@ -4178,7 +4178,7 @@ TEST_F(TestLibRBD, RenameViaLockOwner)
 
 TEST_F(TestLibRBD, SnapCreateViaLockOwner)
 {
-  REQUIRE_FEATURE(RBD_FEATURE_LAYERING | RBD_FEATURE_EXCLUSIVE_LOCK);
+  REQUIRE_FEATURE(RBD_FEATURE_EXCLUSIVE_LOCK);
 
   librados::IoCtx ioctx;
   ASSERT_EQ(0, _rados.ioctx_create(m_pool_name.c_str(), ioctx));
@@ -4222,7 +4222,7 @@ TEST_F(TestLibRBD, SnapCreateViaLockOwner)
 
 TEST_F(TestLibRBD, SnapRemoveViaLockOwner)
 {
-  REQUIRE_FEATURE(RBD_FEATURE_LAYERING | RBD_FEATURE_EXCLUSIVE_LOCK);
+  REQUIRE_FEATURE(RBD_FEATURE_FAST_DIFF);
 
   librados::IoCtx ioctx;
   ASSERT_EQ(0, _rados.ioctx_create(m_pool_name.c_str(), ioctx));