geo-rep: Detailed JSON output for config

JSON output of `config-get` command now returns in the following
format

    {
        "name": CONFIG_NAME,
        "value": CONFIG_VALUE,
        "default_value": DEFAULT_VALUE,  # Only if modified == true
        "configurable": true|false,
        "modified": true|false
    }

Change-Id: I6193de48cd33655df7ecef5a0d83d7cb147089cf
Fixes: #361
Signed-off-by: Aravinda VK <avishwan@redhat.com>

1 files changed, 13 insertions, 5 deletions

diff --git a/geo-replication/syncdaemon/subcmds.py b/geo-replication/syncdaemon/subcmds.py
index 258dbb0b658..1b306ad5b62 100644
--- a/geo-replication/syncdaemon/subcmds.py
+++ b/geo-replication/syncdaemon/subcmds.py
@@ -243,21 +243,29 @@ def subcmd_config_get(args):
             sys.stderr.write("Invalid config name \"%s\"\n" % args.name)
             sys.exit(ERROR_CONFIG_INVALID)
 
-        print_config(args.name, val, only_value=args.only_value,
+        print_config(args.name, val["value"], only_value=args.only_value,
                      use_underscore=args.use_underscore)
         return
 
     if args.json:
-        out = {}
+        out = []
         # Convert all values as string
-        for k, v in all_config.items():
-            out[k] = str(v)
+        for k in sorted(all_config):
+            v = all_config[k]
+            out.append({
+                "name": k,
+                "value": str(v["value"]),
+                "default": str(v["default"]),
+                "configurable": v["configurable"],
+                "modified": v["modified"]
+            })
 
         print(json.dumps(out))
         return
 
     for k in sorted(all_config):
-        print_config(k, all_config[k], use_underscore=args.use_underscore)
+        print_config(k, all_config[k]["value"],
+                     use_underscore=args.use_underscore)
 
 
 def subcmd_config_check(args):