diff options
| author | M S Vishwanath Bhat <vishwanath@gluster.com> | 2011-10-14 17:29:41 +0530 |
|---|---|---|
| committer | Vijay Bellur <vijay@gluster.com> | 2011-10-16 23:24:46 -0700 |
| commit | 90d691d049420f712f5bd1898c75cc66edd1b315 (patch) | |
| tree | 898f9b622b235ba89333d96697fce838f1e30aa8 /cli | |
| parent | f9e58c75f6bffabc29872f20688ac3a30a6a2bee (diff) | |
makes 'volume top' exit with exit status '0' upon success
volume top exit status was not zero even when command was successful
Change-Id: I12761cb6f00ad5c851bfee7cf0aaf0b92dfb94f4
BUG: 2882
testcase: run any valid 'volume top' command and check for exit status
Reviewed-on: http://review.gluster.com/587
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vijay@gluster.com>
Diffstat (limited to 'cli')
| -rw-r--r-- | cli/src/cli-rpc-ops.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c index 761c67cbbb5..29589591ea6 100644 --- a/cli/src/cli-rpc-ops.c +++ b/cli/src/cli-rpc-ops.c @@ -3238,7 +3238,7 @@ gf_cli3_1_top_volume_cbk (struct rpc_req *req, struct iovec *iov, ret = dict_get_int32 (dict, key, (int32_t*)&top_op); if (ret) goto out; - while (i <= brick_count) { + while (i < brick_count) { i++; snprintf (brick, sizeof (brick), "%d-brick", i); ret = dict_get_str (dict, brick, &bricks); |