osd: add missing formatter close_section() to scrub status

Also add braces to make the open/close matchups easier to see.  Broken
by f36617392710f9b3538bfd59d45fd72265993d57.

Signed-off-by: Sage Weil <sage@inktank.com>

diff --git a/src/osd/PG.cc b/src/osd/PG.cc
index 3d6dc85523ce02f098b09cae35c7b0d778120c77..14d415b6e47bd6b87085d3cf4641a0eb6a18439f 100644 (file)
--- a/src/osd/PG.cc
+++ b/src/osd/PG.cc
@@ -4342,19 +4342,24 @@ boost::statechart::result PG::RecoveryState::Active::react(const QueryState& q)
     q.f->close_section();
   }
 
-  q.f->open_object_section("scrub");
-  q.f->dump_stream("scrub_epoch_start") << pg->scrub_epoch_start;
-  q.f->dump_int("scrub_active", pg->scrub_active);
-  q.f->dump_int("scrub_block_writes", pg->scrub_block_writes);
-  q.f->dump_int("finalizing_scrub", pg->finalizing_scrub);
-  q.f->dump_int("scrub_waiting_on", pg->scrub_waiting_on);
-  q.f->open_array_section("scrub_waiting_on_whom");
-  for (set<int>::iterator p = pg->scrub_waiting_on_whom.begin();
-       p != pg->scrub_waiting_on_whom.end();
-       ++p) {
-    q.f->dump_int("osd", *p);
+  {
+    q.f->open_object_section("scrub");
+    q.f->dump_stream("scrub_epoch_start") << pg->scrub_epoch_start;
+    q.f->dump_int("scrub_active", pg->scrub_active);
+    q.f->dump_int("scrub_block_writes", pg->scrub_block_writes);
+    q.f->dump_int("finalizing_scrub", pg->finalizing_scrub);
+    q.f->dump_int("scrub_waiting_on", pg->scrub_waiting_on);
+    {
+      q.f->open_array_section("scrub_waiting_on_whom");
+      for (set<int>::iterator p = pg->scrub_waiting_on_whom.begin();
+          p != pg->scrub_waiting_on_whom.end();
+          ++p) {
+       q.f->dump_int("osd", *p);
+      }
+      q.f->close_section();
+    }
+    q.f->close_section();
   }
-  q.f->close_section();
 
   q.f->close_section();
   return forward_event();