From 37bf4daca164cfcb260760ee2fd25d66f920dc7f Mon Sep 17 00:00:00 2001 From: Jiffin Tony Thottan Date: Wed, 22 Feb 2017 18:26:30 +0530 Subject: [PATCH 064/124] glusterd/ganesha : return proper value in pre_setup() Label: DOWNSTREAM ONLY Change-Id: I6f7ce82488904c7d418ee078162f26f1ec81e9d9 Signed-off-by: Jiffin Tony Thottan Reviewed-on: https://review.gluster.org/16733 Smoke: Gluster Build System Reviewed-by: Atin Mukherjee Reviewed-by: Raghavendra Talur Tested-by: Raghavendra Talur NetBSD-regression: NetBSD Build System CentOS-regression: Gluster Build System Signed-off-by: Jiffin Tony Thottan Reviewed-on: https://code.engineering.redhat.com/gerrit/167149 Reviewed-by: Soumya Koduri Tested-by: RHGS Build Bot Reviewed-by: Sunil Kumar Heggodu Gopala Acharya --- xlators/mgmt/glusterd/src/glusterd-ganesha.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/xlators/mgmt/glusterd/src/glusterd-ganesha.c b/xlators/mgmt/glusterd/src/glusterd-ganesha.c index 1c2ba7a..d9fdfc6 100644 --- a/xlators/mgmt/glusterd/src/glusterd-ganesha.c +++ b/xlators/mgmt/glusterd/src/glusterd-ganesha.c @@ -843,16 +843,15 @@ pre_setup(gf_boolean_t run_setup, char **op_errstr) { int ret = 0; - ret = check_host_list(); - - if (ret) { + if (check_host_list()) { ret = setup_cluster(run_setup); if (ret == -1) gf_asprintf(op_errstr, "Failed to set up HA " "config for NFS-Ganesha. " "Please check the log file for details"); - } + } else + ret = -1; return ret; } -- 1.8.3.1