crimson/os/seastore: minor debugging improvements

Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>

diff --git a/src/crimson/os/seastore/device.cc b/src/crimson/os/seastore/device.cc
index c1b5617206a26da74ab9876f935c04610a1c6e60..0b21cac8ef2c3d9c4f41e447786e122697b7d8d4 100644 (file)
--- 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 416099ca65952f399ad644a34977c4a24b337dd8..3eceee2bffc959fb1350eb6ac003e383baf5462d 100644 (file)
--- 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<std::pair<segment_id_t, segment_header_t>>(),
     [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();