From 1fc2a841ae90b0f29bf8aa39351bd92879ec1f44 Mon Sep 17 00:00:00 2001 From: Junaid Date: Mon, 13 Jun 2011 00:51:10 +0000 Subject: mgmt/glusterd: Set the generic options in the graph before merging it with the parent graph while building nfs-server volfile. Signed-off-by: Junaid Signed-off-by: Anand Avati BUG: 3003 (Modification of one of the options of volume is reflected in all the volumes in the nfs-server-volfile) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=3003 --- xlators/mgmt/glusterd/src/glusterd-volgen.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/xlators/mgmt/glusterd/src/glusterd-volgen.c b/xlators/mgmt/glusterd/src/glusterd-volgen.c index 252c4907d7b..58282aecab9 100644 --- a/xlators/mgmt/glusterd/src/glusterd-volgen.c +++ b/xlators/mgmt/glusterd/src/glusterd-volgen.c @@ -1861,21 +1861,21 @@ build_nfs_graph (volgen_graph_t *graph, dict_t *mod_dict) ret = dict_set_str (set_dict, "client-transport-type", nfs_xprt); ret = build_client_graph (&cgraph, voliter, set_dict); - if (ret) - goto out;; - ret = volgen_graph_merge_sub (graph, &cgraph); if (ret) goto out; if (mod_dict) { dict_copy (mod_dict, set_dict); - ret = volgen_graph_set_options_generic (graph, set_dict, voliter, - nfs_spec_option_handler); + ret = volgen_graph_set_options_generic (&cgraph, set_dict, voliter, + nfs_spec_option_handler); + } else { + ret = volgen_graph_set_options_generic (&cgraph, voliter->dict, voliter, + nfs_spec_option_handler); } - else - ret = volgen_graph_set_options_generic (graph, voliter->dict, voliter, - nfs_spec_option_handler); + ret = volgen_graph_merge_sub (graph, &cgraph); + if (ret) + goto out; } out: -- cgit