diff options
| author | Raghavendra Bhat <raghavendrabhat@gluster.com> | 2012-04-27 14:34:07 +0530 |
|---|---|---|
| committer | Vijay Bellur <vijay@gluster.com> | 2012-05-01 10:25:39 -0700 |
| commit | 1c825e5f2498ec586c0e9123c472f6d134706e08 (patch) | |
| tree | 9abd19e1b35b9f252bf1db8f44d655fbdb230217 /xlators/mgmt/glusterd/src/glusterd-volume-ops.c | |
| parent | 3dd38a4a35885adf6ac2e0c6c4e38d36ab862441 (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>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-volume-ops.c')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-volume-ops.c | 8 |
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); } } |