summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRaghavendra Bhat <raghavendrabhat@gluster.com>2012-04-27 14:34:07 +0530
committerVijay Bellur <vijay@gluster.com>2012-05-01 10:25:39 -0700
commit1c825e5f2498ec586c0e9123c472f6d134706e08 (patch)
tree9abd19e1b35b9f252bf1db8f44d655fbdb230217
parent3dd38a4a35885adf6ac2e0c6c4e38d36ab862441 (diff)
mgmt/glusterd: take the statedumps of running bricks if one of the bricks is down
Till now if one of the bricks of the volume is down, then statedump of that volume (bricks) did not take the statedumps of running bricks. This patch fixes that issue and cli command will take the statedumps of running brick processes. Change-Id: Id9b11905d3d759202e505c2ecfa6ccd5dce6a9ff BUG: 816884 Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com> Reviewed-on: http://review.gluster.com/3237 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Amar Tumballi <amarts@redhat.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-volume-ops.c8
1 files changed, 7 insertions, 1 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-volume-ops.c b/xlators/mgmt/glusterd/src/glusterd-volume-ops.c
index 85e7abba853..4ee9582fd4f 100644
--- a/xlators/mgmt/glusterd/src/glusterd-volume-ops.c
+++ b/xlators/mgmt/glusterd/src/glusterd-volume-ops.c
@@ -1596,8 +1596,14 @@ glusterd_op_statedump_volume (dict_t *dict, char **op_errstr)
ret = glusterd_brick_statedump (volinfo, brickinfo,
options, option_cnt,
op_errstr);
+ /* Let us take the statedump of other bricks instead of
+ * exiting, if statedump of this brick fails.
+ */
if (ret)
- goto out;
+ gf_log (THIS->name, GF_LOG_WARNING, "could not "
+ "take the statedump of the brick %s:%s."
+ " Proceeding to other bricks",
+ brickinfo->hostname, brickinfo->path);
}
}