From 6a0ac27dd2cd8a27e9511ad0b89ef34e9476c439 Mon Sep 17 00:00:00 2001 From: Igor Fedotov Date: Wed, 25 Aug 2021 15:37:19 +0300 Subject: [PATCH] test/store_test: fix broken test case After merging NCB stuff missed shared blob detection doesn't reveal extent leak any more - which is expected. Hence we need to adjust the relevant UT. Fixes:https://tracker.ceph.com/issues/52398 Signed-off-by: Igor Fedotov --- src/test/objectstore/store_test.cc | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/src/test/objectstore/store_test.cc b/src/test/objectstore/store_test.cc index 950a326d127d..36f8efece999 100644 --- a/src/test/objectstore/store_test.cc +++ b/src/test/objectstore/store_test.cc @@ -8624,8 +8624,16 @@ TEST_P(StoreTestSpecificAUSize, BluestoreBrokenNoSharedBlobRepairTest) { { cerr << "fscking/fixing" << std::endl; bstore->umount(); - ASSERT_EQ(bstore->fsck(false), 3); - ASSERT_LE(bstore->repair(false), 3); + // depending on the allocation map's source we can + // either observe or don't observe an additional + // extent leak detection. Hence adjusting the expected + // value + size_t expected_error_count = + g_ceph_context->_conf->bluestore_allocation_from_file ? + 2 : + 3; + ASSERT_EQ(bstore->fsck(false), expected_error_count); + ASSERT_LE(bstore->repair(false), expected_error_count); ASSERT_EQ(bstore->fsck(false), 0); } -- 2.47.3