From 120e9293bf73c4fcb2dbef43ec4525d846899673 Mon Sep 17 00:00:00 2001 From: Yingxin Cheng Date: Mon, 11 Apr 2022 23:42:47 +0800 Subject: [PATCH] crimson/os/seastore: minor debugging improvements Signed-off-by: Yingxin Cheng --- src/crimson/os/seastore/device.cc | 2 +- src/crimson/os/seastore/segment_cleaner.cc | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/crimson/os/seastore/device.cc b/src/crimson/os/seastore/device.cc index c1b5617206a26..0b21cac8ef2c3 100644 --- a/src/crimson/os/seastore/device.cc +++ b/src/crimson/os/seastore/device.cc @@ -25,7 +25,7 @@ std::ostream& operator<<(std::ostream& out, const device_config_t& conf) << ", secondary("; for (const auto& [k, v] : conf.secondary_devices) { out << device_id_printer_t{k} - << ": " << v; + << ": " << v << ", "; } return out << "))"; } diff --git a/src/crimson/os/seastore/segment_cleaner.cc b/src/crimson/os/seastore/segment_cleaner.cc index 416099ca65952..3eceee2bffc95 100644 --- a/src/crimson/os/seastore/segment_cleaner.cc +++ b/src/crimson/os/seastore/segment_cleaner.cc @@ -503,7 +503,7 @@ SegmentCleaner::gc_reclaim_space_ret SegmentCleaner::gc_reclaim_space() SegmentCleaner::mount_ret SegmentCleaner::mount() { const auto& sms = sm_group->get_segment_managers(); - logger().debug( + logger().info( "SegmentCleaner::mount: {} segment managers", sms.size()); init_complete = false; stats = {}; @@ -526,7 +526,7 @@ SegmentCleaner::mount_ret SegmentCleaner::mount() metrics.clear(); register_metrics(); - logger().debug("SegmentCleaner::mount: {} segments", segments.size()); + logger().info("SegmentCleaner::mount: {} segments", segments.size()); return seastar::do_with( std::vector>(), [this](auto& segment_set) { @@ -538,7 +538,7 @@ SegmentCleaner::mount_ret SegmentCleaner::mount() return sm_group->read_segment_header( segment_id ).safe_then([segment_id, this, &segment_set](auto header) { - logger().debug( + logger().info( "SegmentCleaner::mount: segment_id={} -- {}", segment_id, header); auto s_type = header.get_type(); -- 2.39.5