osd: OSDMap: have osdmap json dump print valid boolean instead of string

Fixes: 8108
Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>

diff --git a/PendingReleaseNotes b/PendingReleaseNotes
index 9a4cc081bda7566fd224d430728ab5e4b9cb4e65..7494e6e9e45c2a0ce0024662a82c907d24c74f40 100644 (file)
--- a/PendingReleaseNotes
+++ b/PendingReleaseNotes
@@ -1,3 +1,7 @@
 
 v0.80
 -----
+
+* OSDMap's json-formatted dump changed for keys 'full' and 'nearfull'.
+  What was previously being outputted as 'true' or 'false' strings are
+  now being outputted 'true' and 'false' booleans according to json syntax.

diff --git a/src/osd/OSDMap.cc b/src/osd/OSDMap.cc
index a764f34cdbcc051b69c22c4398b0055d2b963b19..fc61d4356d41954f2da71879c6bf3743df39f39d 100644 (file)
--- a/src/osd/OSDMap.cc
+++ b/src/osd/OSDMap.cc
@@ -2426,9 +2426,8 @@ void OSDMap::print_summary(Formatter *f, ostream& out) const
     f->dump_int("num_osds", get_num_osds());
     f->dump_int("num_up_osds", get_num_up_osds());
     f->dump_int("num_in_osds", get_num_in_osds());
-    f->dump_string("full", test_flag(CEPH_OSDMAP_FULL) ? "true" : "false");
-    f->dump_string("nearfull", test_flag(CEPH_OSDMAP_NEARFULL) ?
-                  "true" : "false");
+    f->dump_bool("full", test_flag(CEPH_OSDMAP_FULL) ? true : false);
+    f->dump_bool("nearfull", test_flag(CEPH_OSDMAP_NEARFULL) ? true : false);
     f->close_section();
   } else {
     out << "     osdmap e" << get_epoch() << ": "