test/librbd: Remove crimson skip from TestDeepCopy

knrt10 · knrt10 · commit f5ed57afc9cc · 2025-11-06T11:53:45.000Z
The TestDeepCopy.Stress and TestDeepCopy.Stress_SmallerDstObjSize tests were previously skipped for the crimson store. This commit removes the SKIP_IF_CRIMSON() calls, indicating that the tests should now pass with the crimson osd. Fixes: https://tracker.ceph.com/issues/72042 Signed-off-by: Kautilya Tripathi <kautilya.tripathi@ibm.com>
diff --git a/src/test/librbd/test_DeepCopy.cc b/src/test/librbd/test_DeepCopy.cc
@@ -510,8 +510,6 @@ TEST_F(TestDeepCopy, CloneFlatten)
 
 TEST_F(TestDeepCopy, Stress)
 {
-  // https://tracker.ceph.com/issues/72042
-  SKIP_IF_CRIMSON();
   test_stress();
 }
 
@@ -609,8 +607,6 @@ TEST_F(TestDeepCopy, CloneFlatten_SmallerDstObjSize)
 
 TEST_F(TestDeepCopy, Stress_SmallerDstObjSize)
 {
-  // https://tracker.ceph.com/issues/72042
-  SKIP_IF_CRIMSON();
   uint64_t order = m_src_ictx->order - 1 - rand() % 2;
   ASSERT_EQ(0, m_opts.set(RBD_IMAGE_OPTION_ORDER, order));
   uint64_t stripe_unit = m_src_ictx->stripe_unit >> 2;

Original file line number	Diff line number	Diff line change
`@@ -510,8 +510,6 @@ TEST_F(TestDeepCopy, CloneFlatten)`
`510`	`510`
`511`	`511`	`TEST_F(TestDeepCopy, Stress)`
`512`	`512`	`{`
`513`		`- // https://tracker.ceph.com/issues/72042`
`514`		`- SKIP_IF_CRIMSON();`
`515`	`513`	`test_stress();`
`516`	`514`	`}`
`517`	`515`
`@@ -609,8 +607,6 @@ TEST_F(TestDeepCopy, CloneFlatten_SmallerDstObjSize)`
`609`	`607`
`610`	`608`	`TEST_F(TestDeepCopy, Stress_SmallerDstObjSize)`
`611`	`609`	`{`
`612`		`- // https://tracker.ceph.com/issues/72042`
`613`		`- SKIP_IF_CRIMSON();`
`614`	`610`	`uint64_t order = m_src_ictx->order - 1 - rand() % 2;`
`615`	`611`	`ASSERT_EQ(0, m_opts.set(RBD_IMAGE_OPTION_ORDER, order));`
`616`	`612`	`uint64_t stripe_unit = m_src_ictx->stripe_unit >> 2;`