From 13f1fff6da4b2b9ee30f43346cb382abfc1bc304 Mon Sep 17 00:00:00 2001 From: Kaushik BV Date: Fri, 8 Oct 2010 10:34:18 +0000 Subject: mgmt/Glusterd : Volume info fixes Signed-off-by: Kaushik BV Signed-off-by: Vijay Bellur BUG: 1872 (gluster volume info is not working with qa42-) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1872 --- xlators/mgmt/glusterd/src/glusterd-handler.c | 27 +++++++++++++-------------- 1 file changed, 13 insertions(+), 14 deletions(-) (limited to 'xlators/mgmt/glusterd/src') diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c index dcc408eb071..ba9aeb8f95c 100644 --- a/xlators/mgmt/glusterd/src/glusterd-handler.c +++ b/xlators/mgmt/glusterd/src/glusterd-handler.c @@ -269,6 +269,7 @@ glusterd_add_volume_detail_to_dict (glusterd_volinfo_t *volinfo, char reconfig_key[256] = {0, }; dict_t *dict = NULL; data_t *value = NULL; + int opt_count = 0; GF_ASSERT (volinfo); @@ -318,34 +319,32 @@ glusterd_add_volume_detail_to_dict (glusterd_volinfo_t *volinfo, dict = volinfo->dict; if (!dict) { - ret = -1; + ret = 0; goto out; } pairs = dict->members_list; - if (!pairs) { - ret = -1; - goto out; - } while (pairs) { if (1 == glusterd_check_option_exists (pairs->key, NULL)) { value = pairs->value; - if (!value) { - ret = -1; - goto out; - } + if (!value) + continue; + snprintf (reconfig_key, 256, "volume%d.option.%s", count, pairs->key); - gf_log ("", GF_LOG_DEBUG, - "Setting dict with key=%s, value=%s", - reconfig_key, pairs->value->data); - ret = dict_set_str (volumes, gf_strdup (reconfig_key), - value->data); + ret = dict_set_str (volumes, reconfig_key, value->data); + if (!ret) + opt_count++; } pairs = pairs->next; } + snprintf (key, 256, "volume%d.opt_count", count); + ret = dict_set_int32 (volumes, key, opt_count); + if (ret) + goto out; + out: return ret; } -- cgit