glusterfs.git/cli/src, branch v3.3.0qa40

cli: fix remove-brick output behavior in failure cases

2012-05-04T04:28:37+00:00

earlier it was dependant on server sending the dict for knowing
the command. Now it preserves the dictionary in frame->local,
which is used to understand the command.

Change-Id: Ica675b497e466f77878f67792ae96b71f0704d6b
Signed-off-by: Amar Tumballi 
BUG: 815745
Reviewed-on: http://review.gluster.com/3235
Tested-by: Gluster Build System 
Reviewed-by: Shishir Gowda

cli: Make use of cli_err()

2012-04-27T05:50:38+00:00

Use cli_err() in case of errors instead of cli_out().

Change-Id: Ib0766e59516a1ccd56a2133f33fac3877c4384d6
BUG: 815194
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/3229
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

cli: Show proper remove-brick usage

2012-04-25T11:05:53+00:00

Change-Id: I669736b34f50caa146eea47dcd8c081f6436a585
BUG: 816051
Signed-off-by: shylesh 
Reviewed-on: http://review.gluster.com/3224
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Shishir Gowda

cli: implement a fn 'cli_err()' to send error messages to 'stderr'

2012-04-25T07:45:51+00:00

we were using 'cli_out()' to send all the possible msgs, which is not
very friendly with scripts, because if one want to get only valid
output with " 2>/error.log 1>/proper-info.log"

Change-Id: I008ebcbd90935c41dbfc1bd2adeb094ed21116cb
Signed-off-by: Amar Tumballi 
BUG: 815194
Reviewed-on: http://review.gluster.com/3208
Tested-by: Gluster Build System 
Reviewed-by: Kaushal M 
Reviewed-by: Vijay Bellur

cli: Return -1 if fix-layout stop is issued

2012-04-24T16:42:03+00:00

Change-Id: If2e4b842029a1a720403cbec0c076f3dbf1fc6df
BUG: 803971
Signed-off-by: shylesh 
Reviewed-on: http://review.gluster.com/3216
Tested-by: Gluster Build System 
Reviewed-by: Shishir Gowda

NetBSD build fixes

2012-04-24T04:29:03+00:00

Signed-off-by: Emmanuel Dreyfus 

Change-Id: I8f9aabeadd2f842521a82e59594115bd80155d68
BUG: 2923
Signed-off-by: Emmanuel Dreyfus 
Reviewed-on: http://review.gluster.com/3053
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

cli: fix the output of 'gluster volume list'

2012-04-24T01:47:52+00:00

make it script friendly

Change-Id: I5b196cf5cf1744090a1360eef95c9a842d7be996
Signed-off-by: Amar Tumballi 
BUG: 815190
Reviewed-on: http://review.gluster.com/3207
Tested-by: Gluster Build System 
Reviewed-by: Kaushal M 
Reviewed-by: Anand Avati

core: coverity issues fixed

2012-04-23T21:52:57+00:00

this is not a complete set of issues getting fixed. Will
address other issues in another patch.

Change-Id: Ib01c7b11b205078cc4d0b3f11610751e32d14b69
Signed-off-by: Amar Tumballi 
BUG: 789278
Reviewed-on: http://review.gluster.com/3145
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy

cli: Strip whitespace from "volume set" option values

2012-04-20T16:32:17+00:00

Strips the whitespace from options values before sending to glusterd.
This prevents options containing whitespace to be written to the volfiles which
would cause the volfile parser to fail.

Change-Id: I46faee7b0853141fa102d06bb067c7ab499a2f6e
BUG: 813937
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/3192
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur

cli: do not put the frame->local to mempool since its a calloced structure

2012-04-19T06:53:51+00:00

In cli, frame->local is not allocated from the internal mempool, rather its
calloced. So while destroying the stack, we have to set frame->local to NULL.
Otherwise if frame->local is not NULL, STACK_DESTROY will try to put it back
to the mempool and thus segfault.

Change-Id: I5bd17d099e16fb8e73f3250cc022008e26e28f50
BUG: 812735
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.com/3167
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi