glusterfs.git/cli/src/cli-cmd-parser.c, branch v3.3.1qa3

cli: Fix double free in cli_add_key_group

2012-09-14T08:30:30+00:00

Change-Id: I3c2f030ac7c53913612a3fbac5e582c47b005621
BUG: 851237
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.org/3944
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

cli: Added special key "group" for bulk volume set.

2012-09-12T19:26:15+00:00

gluster volume set VOLNAME group group_name

- where group_name is a file under /var/lib/glusterd/groups containing one
  key, value pair per line as below,
  key1=value1
  key2=value2
  [...]
- the command sets key1 to value1 and so on.

Change-Id: Ic4c8dedb98d013b29a74e57f8ee7c1d3573137d2
BUG: 851237
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.org/3896
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

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: 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

glusterd/remove-brick: Replace ABORT with STOP

2012-04-13T07:46:59+00:00

Remove-brick stop now invokes rebalance stop. This leads
to a graceful stop of decommissioning.
The volfile is also updated (removal of decommission)

Change-Id: I5a8f725c0f54439b810ce32d988c21c02229c703
BUG: 811513
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.com/3126
Reviewed-by: Amar Tumballi 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

glusterd/remove-brick: Remove support for pause option

2012-04-13T07:26:33+00:00

Decommissioning through rebalance has no pause option.

Change-Id: I90f165cdb2eccfaefc99365ae4b48d81320fb753
BUG: 811459
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.com/3123
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

cli: Limit value of bs * count in "volume top"

2012-04-02T18:31:21+00:00

The maximum value of bs * count is limited to 10GB. This prevents gluster from
returning 0 throughput with large values of bs * count. If throughput for values
larger than 10GB is required, use dd.

Change-Id: Ia4ea70cb20f3f32b6cb81fd451ad56933f438452
BUG: 803322
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/3059
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cli,glusterd: more volume status improvements

2012-03-29T14:47:53+00:00

The major changes are,
* "volume status" now supports getting details of the self-heal daemon processes
for replica volumes. A new cli options "shd", similar to "nfs", has been
introduced for this. "detail", "fd" and "clients" status ops are not supported
for self-heal daemons.
* The default/normal ouput of "volume status" has been enhanced to contain
information about nfs-server and self-heal daemon processes as well. Some tweaks
have been done to the cli output to show appropriate output.

Also, changes have been done to rebalance/remove-brick status, so that hostnames
are displayed instead of uuids.

Change-Id: I3972396dcf72d45e14837fa5f9c7d62410901df8
BUG: 803676
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/3016
Tested-by: Gluster Build System 
Reviewed-by: Krishnan Parthasarathi 
Reviewed-by: Vijay Bellur

mgmt/glusterd : volume set validation fixes

2012-03-18T07:52:40+00:00

This is the new version of the patch by Kaushik at review.gluster.com/699

The following new option types have been introduced:
 * GF_OPTION_TYPE_INTERNET_ADDRESS_LIST
 * GF_OPTION_TYPE_PRIORITY_LIST
 * GF_OPTION_TYPE_SIZE_LIST
and option types of several options in translators have been updated to use the
new types.

valid_internet_address(), valid_ipv4_address() & valid_ipv6_address() functions
has been updated for * wildcard matching. Previously used standalone wildcard
address checking functions have been removed.

Changes have been done to stripe translator to correctly set, update and use
stripe-blocksize. Also minimum value for block-size has been set to 16KB.

Change-Id: I2aa484ff695f6a915a8fc9a9f965cf0344f41d59
BUG: 765248
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/2899
Tested-by: Gluster Build System 
Reviewed-by: Shishir Gowda 
Reviewed-by: Anand Avati

core, cli : Changes to statedump

2012-03-14T09:57:49+00:00

This patch,
* Enables missing inodectx, fdctx dumpops for cli statedump command.
* Reintroduce changes introduced in statedump.c by 489a7a10 which were
  overwritten by db20a0f8.
* Change strncasecmp() in gf_proc_dump_parse_set_option() to strcasecmp()
  to properly parse similar options like fd-fdctx, inode-inodectx

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