diff options
| author | Vijay Bellur <vijay@gluster.com> | 2010-10-27 03:07:29 +0000 | 
|---|---|---|
| committer | Anand V. Avati <avati@dev.gluster.com> | 2010-10-27 02:57:12 -0700 | 
| commit | 38c91384861f29f97c3f564a36ee799f1eb57cc1 (patch) | |
| tree | 41125daa10a476d44b371eb773569c9b52769329 /xlators/mgmt/glusterd/src/glusterd-handler.c | |
| parent | c02cd37866ae22e3a433d737f3c80395bb72f002 (diff) | |
mgmt/glusterd: Unlock upon a cli request failure
Signed-off-by: Vijay Bellur <vijay@gluster.com>
Signed-off-by: Anand V. Avati <avati@dev.gluster.com>
BUG: 1873 (Stop volume gluster command execution failed)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1873
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-handler.c')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-handler.c | 14 | 
1 files changed, 7 insertions, 7 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c index 8f324802f5a..930c3a45755 100644 --- a/xlators/mgmt/glusterd/src/glusterd-handler.c +++ b/xlators/mgmt/glusterd/src/glusterd-handler.c @@ -1092,8 +1092,8 @@ glusterd_handle_add_brick (rpcsvc_request_t *req)                  if (!brick_count || !volinfo->sub_count)                          goto brick_val; -		/* If the brick count is less than sub_count then, allow add-brick only for  -		   plain replicate volume since in plain stripe brick_count becoming less than  +		/* If the brick count is less than sub_count then, allow add-brick only for +		   plain replicate volume since in plain stripe brick_count becoming less than  		   the sub_count is not allowed */                  if (volinfo->brick_count < volinfo->sub_count && (volinfo->type == GF_CLUSTER_TYPE_REPLICATE) ) {                          if ((volinfo->sub_count - volinfo->brick_count) == brick_count) @@ -2915,7 +2915,7 @@ out:                  rsp.op_ret = 1;          else                  rsp.op_ret = ret; -        if (!rsp.volname)  +        if (!rsp.volname)                  rsp.volname = "";          if (!rsp.op_errstr)                  rsp.op_errstr = "Error, Validation failed"; @@ -2933,7 +2933,7 @@ glusterd_reset_volume (rpcsvc_request_t *req, dict_t *dict)  {          int ret = -1; -         +          glusterd_op_set_op (GD_OP_RESET_VOLUME);          glusterd_op_set_ctx (GD_OP_RESET_VOLUME, dict); @@ -2945,7 +2945,7 @@ glusterd_reset_volume (rpcsvc_request_t *req, dict_t *dict)          glusterd_op_set_req (req);          ret = glusterd_op_txn_begin (); -         +          return ret;  } @@ -2959,11 +2959,11 @@ glusterd_set_volume (rpcsvc_request_t *req, dict_t *dict)          GF_ASSERT (req);          GF_ASSERT (dict); -         +          ret = dict_get_int32 (dict, "count", &dict_count);          if (ret)                 goto out; -         +          if (dict_count == 1) {                  if (dict_get (dict, "history")) {                          ret = glusterd_set_volume_history(req, dict);  | 
