glusterfs.git/tests/bugs, branch v3.4.0qa4

afr: handle short writes in afr_writev_wind and self-heal to avoid corruption

2012-11-29T17:00:28+00:00

The current failure to handle short writes on writev fops leaves
us open to file corruption. A short write on a user request is
ignored and leaves replicas in an inconsistent state. A short write
during a self-heal is ignored and incorrectly marks the files as
consistent if the heal completes.

Modify user writev handling to return the best case return value
from each of the replicas. Short writes that occur relative to this
value are marked as failed and will require a heal. Modify
self-heal to set an error on a short write and abort the heal.

BUG: 853690
Change-Id: I18b30f58702326249230eeebb361b29e40b535f5
Signed-off-by: Brian Foster 
Reviewed-on: http://review.gluster.org/4150
Reviewed-by: Jeff Darcy 
Reviewed-by: Pranith Kumar Karampuri 
Tested-by: Gluster Build System

cluster/dht: send ACCESS call on dir to first_up_subvol if cached is down

2012-11-29T10:26:18+00:00

Change-Id: I4f518a969bbe3a11075e7c9ae10bd21bf059d5f3
BUG: 867253
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.org/4240
Reviewed-by: Jeff Darcy 
Tested-by: Gluster Build System

glusterd, cli: implement gluster system uuid reset command

2012-11-29T00:28:55+00:00

A commonly faced problem among glusterfs users is: after a fresh
installation of glusterfs in a virtual machine, the VM image is
cloned to make multiple instances of the server. This breaks
glusterd because right after glusterfs installation on the first
boot glusterd would have created the node UUID and this gets
inherited into the clone. The result is wierd behavior at the time
of peer probe where glusterd does not (yet) deal with UUID
collisions in a user friendly way.

To handle it gluster peer reset command is implemented which upon
execution changes the uuid of local glusterd.

Change-Id: If207dd2ad93ab94ef1a3253f409c21c442975f87
BUG: 811493
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.org/3637
Reviewed-by: Pranith Kumar Karampuri 
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

glusterd: Protected conf->xprt_list racy access.

2012-11-29T00:27:01+00:00

- epoll on RPCSVC_EVENT_ACCEPT would add corresponding xprt
  onto the xprt_list. Concurrently, synctask thread (volume op)
  would call into glusterd_fetchspec_notify which iterates on
  the xprt_list. Added a mutex to protect such a racy access of
  the list.

Change-Id: Idc51b4bdb1c814dfab7790e1c899d6977f7640f2
BUG: 878873
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.org/4241
Reviewed-by: Raghavendra G 
Reviewed-by: Jeff Darcy 
Tested-by: Gluster Build System

protocol/client: add an option to filter O_DIRECT flag in open

2012-11-28T11:37:03+00:00

with the option, the idea is all client-side caching will be disabled,
where as on server side process, the fd will be treated as a regular
fd, thus helping the performance better.

"gluster volume set  remote-dio enable" would set
this option in client protocol volumes.

Change-Id: Id2255a167137f8fee20849513e3011274dc829b4
Signed-off-by: Amar Tumballi 
BUG: 845213
Reviewed-on: http://review.gluster.org/4206
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

mount/fuse: add test cases for lock migration

2012-11-28T06:45:02+00:00

Change-Id: I1a48c63387c7803b36584070fd8f3684ccb696b1
BUG: 808400
Signed-off-by: Raghavendra G 
Reviewed-on: http://review.gluster.org/4209
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

tests/bug-865825: turn stat-prefetch off before doing any afr self-heal

2012-11-28T06:42:21+00:00

                  related tests.

Change-Id: I5c45bb4f4d4aa59965c6dda34ab884071bef9c6f
BUG: 865825
Signed-off-by: Raghavendra G 
Reviewed-on: http://review.gluster.org/4242
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cli: XML output for "gluster volume geo-replication status"

2012-11-28T06:29:10+00:00

Change-Id: I1a64bd5eb9b1040a2a2d9b97bfe9cc756835596e
BUG: 864506
Signed-off-by: Avra Sengupta 
Reviewed-on: http://review.gluster.org/4227
Reviewed-by: Kaushal M 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

tests: Replace mount -t glusterfs with glusterfs -s

2012-11-27T12:02:59+00:00

Change-Id: I9aa9bd1efb9ae56caec0fee596b3f21ca496f66e
BUG: 863068
Signed-off-by: Vijay Bellur 
Reviewed-on: http://review.gluster.org/4239
Tested-by: Gluster Build System 
Reviewed-by: Pranith Kumar Karampuri

replicate: don't stop checking xattrs because one was absent

2012-11-27T05:34:33+00:00

The functional issue is described by the subject line.  This patch also
addresses several efficiency/structure issues, such as...

* Calling dict_set_ptr once for each txn type, instead of once overall.

* Calling afr_index_for_transaction_type once per iteration instead of
  once per call (or better yet zero since the conversion is unnecessary).

* Implementation of inner functions in a different file than their one
  caller, creating a spurious header-file dependency.

Change-Id: I29e0df906a820533b66b9ced73e015dfe77267d2
BUG: 865825
Signed-off-by: Jeff Darcy 
Reviewed-on: http://review.gluster.org/4070
Tested-by: Gluster Build System 
Reviewed-by: Pranith Kumar Karampuri 
Reviewed-by: Anand Avati