glusterfs.git, branch v3.4.2qa2

gfapi: backport handles to 3.4 for nfs-ganesha

2013-12-04T11:34:13+00:00

nfs-ganesha-2.0 will ship shortly. It depends on the gfapi handle support,
which won't otherwise be released until 3.5 sometime in 2014.

Change-Id: I104d6fb275bb2c710790340fdc7d998446403026
Signed-off-by: Kaleb S. KEITHLEY 
Reviewed-on: http://review.gluster.org/6394
Tested-by: Gluster Build System 
Reviewed-by: Shyamsundar Ranganathan 
Reviewed-by: Vijay Bellur

cluster/afr: handle NULL check before strlen/strcmp in fgetxattr

2013-12-04T11:33:46+00:00

xattr name can legally be NULL. Handle that case without crashing.

Change-Id: Ie214cb05ccd52565dc247a9234ad83ae799d3866
BUG: 1036879
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/6420
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

glusterd: submit RPC requests without holding big lock

2013-12-04T08:35:23+00:00

If the endpoint of an RPC is not connected, the callback is called
synchronously within rpc_clnt_submit(). Since callbacks typically
hold the big lock, give up the big lock before calling rpc_clnt_submit
and acquire it freshly after the call.

Change-Id: Id89d8dd86c1a4012739ef4af7ea0935492b1a02b
BUG: 1037849
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/6414
Tested-by: Gluster Build System

protocol/client: handle network disconnect/reconnect properly

2013-12-03T19:35:23+00:00

if client/server state versions match, we still need to notify
parent xlators of reconnection (CHILD_UP) because they were
notified of CHILD_DOWN at the time of disconnection.

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

cluster/dht: handle NULL check before strlen/strcmp in fgetxattr

2013-12-03T10:23:18+00:00

@key can legally be NULL. Handle that case without crashing.

Change-Id: Iaae293caa7eeb24afc9cd2580799173e2ce00911
BUG: 1036879
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/6402
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

glusterfsd: fix small memory leaks in glusterfsd-mgmt.c

2013-12-02T04:38:04+00:00

E.g. In glusterfs_volfile_fetch(), req.xdata.xdata_val is allocated
in dict_allocate_and_serialize() but not freed after mgmt_submit_request().

A survey of dict_allocate_and_serialize/_submit_request in
glusterfsd-mgmt.c shows no consistent pattern of freeing the xdata_val
and also the dict, which is a little disturbing. (Yes, clearly not
every place this occurs needs to be freed the same way.)

Change-Id: I8f31179ae12c05c9b5406d8e9e28110fcbfac1c7
BUG: 1036102
Signed-off-by: Kaleb S. KEITHLEY 
Reviewed-on: http://review.gluster.org/6381
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

tests: Change 'volume create' to 'volume create force'

2013-11-27T05:39:30+00:00

Using 'force' when creating volumes prevents errors when creating
bricks in the root partition.
This fixes test bug-823081.t for bug-962226

Change-Id: I210ffda6bdc2cced4bd54755a2b44fdfd05b813b
Original-author: John Smith 
BUG: 962226
Signed-off-by: John Smith 
Signed-off-by: Krutika Dhananjay 
Reviewed-on: http://review.gluster.org/6356
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

glusterd: changes in 'volume create' behaviour

2013-11-27T05:39:16+00:00

        Backport of http://review.gluster.org/4740

Note: This patch is needed by oVirt's Gluster Integration Project

BUG: 948729
Change-Id: I7a9d7c36b7e15269f95637cd9061abac6f8a97de
Signed-off-by: Krutika Dhananjay 
Reviewed-on: http://review.gluster.org/6355
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

cli: write 'volume rebalance' error message in xml format when

2013-11-27T00:47:03+00:00

--xml is specified

When 'volume rebalance' encounters an error the cli prints the
error message in plain text independent of whether --xml is
specified. This throws off client application that expect xml
output (as mentioned in bz1026143).

Now, if the --xml flag is supplied, the cli print 'volume
rebalance' error messages in xml format.

(cherry picked from commit
 cbb47056ab09ae09eed2550584f190fd5a42fb12)

Change-Id: Iec266d56cb2ac34be0ac05a473ae3a08988cca38
BUG: 1026143
Signed-off-by: Dawit Alemu 
Reviewed-on: http://review.gluster.org/6250
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

cluster/dht: set layout in inode ctx even if linkfile fails

2013-11-26T20:52:25+00:00

Creating linkfile could have failed, but we dont care about linkfile
for setting layout in the inode ctx (could be EEXIST etc.)

So ignore @inode in cbk and pick it up from local->loc.inode

Change-Id: I2952799d7ae0d3441b84b2ca2981afd75d7576e2
BUG: 1032859
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/6358
Tested-by: Gluster Build System