glusterfs.git, branch v3.2.6p3

cluster/afr: Perform conservative merge on dir with xattr split-brain

2012-03-14T12:14:56+00:00

Change-Id: I314cf7bbc8a451e42b6addf367f55e431ab0a85c
BUG: 802637
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/2945
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/afr: Handle files w.o. xattrs and size mismatch.

2012-03-13T10:51:46+00:00

Change-Id: I31458538c738c85a026994584c937c99538fe069
BUG: 802637
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/2932
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/afr: Mark zero size file as sink in absense of xattrs.

2012-03-13T10:28:08+00:00

Change-Id: Ic9381f9caa7021a629afb7401e51a29686880bfe
BUG: 802637
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/2931
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

replicate: backport of 0783ca994d9ea95fd9ab3dd95d6407918f19f255

2012-03-13T02:37:45+00:00

This is a backport of patch 0783ca994d9ea95fd9ab3dd95d6407918f19f255
which is a fix for bug 798874

Change-Id: I0edac430b2f22dce121ff4889b563cbef14b09cd
BUG: 798874
Reviewed-on: http://review.gluster.com/2928
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

cluster/afr: Handle split-brain/all-fool xattrs for directory

2012-03-13T02:37:26+00:00

In case of split-brain/all-fool xattrs perform conservative merge.
Don't treat ignorant subvol as fool.

Change-Id: I3044d388d816d79268fec170d202ef23e7d5bf1c
BUG: 765528
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/2674
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

geo-rep / gsyncd: fix cleanup of temporary mounts

2012-03-09T14:14:22+00:00

The "finally" clause that was meant to cleanup after the
temp mount has not covered the case of getting signalled
(eg. by monitor, upon worker timing out).

So here we "outsource" the cleanup to an ephemeral child process.
Child calls setsid(2) so it won't be bothered by internal process
management. We use a pipe in between worker and the cleanup child;
when child sees the worker end getting closed, it performs the cleanup.
Worker end can get closed either because worker closes it (normal case),
or because worker has terminated (faulty case) -- thus as bonus, we get
a nice uniform handling with no need to differentiate between normal and
faulty cases.

The faulty case that was seen IRL -- ie., users of maintainance mounts
hang in chdir(2) to mount point -- can be simulated for testing purposes
by applying the following patch:

diff --git a/xlators/mount/fuse/src/fuse-bridge.c b/xlators/mount/fuse/src/fuse-bridge.c
index acd3c68..1ce5dc1 100644
--- a/xlators/mount/fuse/src/fuse-bridge.c
+++ b/xlators/mount/fuse/src/fuse-bridge.c
@@ -2918,7 +2918,7 @@ fuse_init (xlator_t *this, fuse_in_header_t *finh, void *msg)
         if (fini->minor < 9)
                 *priv->msg0_len_p = sizeof(*finh) + FUSE_COMPAT_WRITE_IN_SIZE;
 #endif
-        ret = send_fuse_obj (this, finh, &fino);
+        ret = priv->client_pid_set ? 0 : send_fuse_obj (this, finh, &fino);
         if (ret == 0)
                 gf_log ("glusterfs-fuse", GF_LOG_INFO,
                         "FUSE inited with protocol versions:"

Change-Id: I1172bf16ac1006bad48958655572155820e5138d
BUG: 786291
Signed-off-by: Csaba Henk 
Reviewed-on: http://review.gluster.com/2908
Tested-by: Gluster Build System 
Reviewed-by: Venky Shankar 
Reviewed-by: Vijay Bellur

gsyncd: geo-rep: Reduce excessive verbosity in logging

2012-03-06T17:46:21+00:00

Backport logging enhancement from master. This patch
reduces per-second logging to per-minute logging during
crawl

Change-Id: Ie51207e6dd8a0257e59f90239cc6960803673223
BUG: 789078
Signed-off-by: Venky Shankar 
Reviewed-on: http://review.gluster.com/2879
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

xlator/lib: Drain out sent requests to avoid multiple STACK_UNWINDs

2012-03-06T13:09:35+00:00

Change-Id: Id48c468934ac64ec52e279ded8949c2a7a760598
BUG: 789078
Signed-off-by: Vijay Bellur 
Reviewed-on: http://review.gluster.com/2871
Tested-by: Gluster Build System 
Reviewed-by: Venky Shankar

Refactor xlator/lib for better readability & maintainability.

2012-03-03T07:40:32+00:00

Change-Id: I26532f1bf7cab58b5870a6117f2a62288ee1f23d
BUG: 789078
Signed-off-by: Vijay Bellur 
Reviewed-on: http://review.gluster.com/2859
Reviewed-by: Pranith Kumar Karampuri 
Tested-by: Gluster Build System

geo-rep / syncdaemon: make the timeout for establishing the connection to slave configurable

2012-03-01T17:28:20+00:00

It can be set through the connection-timeout tunable but we keep it hidden,
intended as a workaround for some special scenarios not for general use.

Change-Id: I31f9fa3873afa7babc2106ee34484123a01bdc57
BUG: 789078
Signed-off-by: Csaba Henk 
Reviewed-on: http://review.gluster.com/2840
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur