Skip to content

Commit 8a182df

Browse files
committed
test/crimson/seastore/transaction_manager_test_state: fix compilation error
Introduced in: ceph@898efa7 Followup: ceph#53433 ``` FAILED: src/crimson/tools/CMakeFiles/perf-staged-fltree.dir/perf_staged_fltree.cc.o error: ‘GetParam’ was not declared in this scope 287 | if (std::get<1>(GetParam()) == integrity_check_t::FULL_CHECK) { | ^~~~~~~~ /home/yogisha/ceph/src/test/crimson/seastore/transaction_manager_test_state.h: In member function ‘virtual seastar::future<> SeaStoreTestState::_init()’: /home/yogisha/ceph/src/test/crimson/seastore/transaction_manager_test_state.h:439:21: error: ‘GetParam’ was not declared in this scope 439 | if (std::get<1>(GetParam()) == integrity_check_t::FULL_CHECK) { | ^~~~~~~~ ``` Signed-off-by: Matan Breizman <[email protected]>
1 parent e03f7be commit 8a182df

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

src/test/crimson/seastore/transaction_manager_test_state.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -284,13 +284,15 @@ class TMTestState : public EphemeralTestState {
284284
auto sec_devices = devices->get_secondary_devices();
285285
auto p_dev = devices->get_primary_device();
286286
auto fut = seastar::now();
287+
#ifdef UNIT_TESTS_BUILT
287288
if (std::get<1>(GetParam()) == integrity_check_t::FULL_CHECK) {
288289
fut = crimson::common::local_conf().set_val(
289290
"seastore_full_integrity_check", "true");
290291
} else {
291292
fut = crimson::common::local_conf().set_val(
292293
"seastore_full_integrity_check", "false");
293294
}
295+
#endif
294296
tm = make_transaction_manager(p_dev, sec_devices, true);
295297
epm = tm->get_epm();
296298
lba_manager = tm->get_lba_manager();
@@ -436,13 +438,15 @@ class SeaStoreTestState : public EphemeralTestState {
436438

437439
virtual seastar::future<> _init() final {
438440
auto fut = seastar::now();
441+
#ifdef UNIT_TESTS_BUILT
439442
if (std::get<1>(GetParam()) == integrity_check_t::FULL_CHECK) {
440443
fut = crimson::common::local_conf().set_val(
441444
"seastore_full_integrity_check", "true");
442445
} else {
443446
fut = crimson::common::local_conf().set_val(
444447
"seastore_full_integrity_check", "false");
445448
}
449+
#endif
446450
seastore = make_test_seastore(
447451
std::make_unique<TestMDStoreState::Store>(mdstore_state.get_mdstore()));
448452
return fut.then([this] {

0 commit comments

Comments
 (0)