glusterfs.git/xlators/cluster/dht, branch v3.4.3

dht: dht_lookup_dir_cbk should set op_errno as local->op_errno

2014-03-11T16:31:57+00:00

Two glusterfs clients return inconsistent errnos when the bricks of the volume
were down. Consider two gluster mounts. Mount 1 was done when the bricks were
online. Mount 2 was done after the bricks were killed, (using the 'glusterfs'
command instead of the mount script).

For any request, mount 1 will return ENOTCONN, where as mount 2 will return
ENOENT.

This happens because for the 2nd mount, a fuse would send a lookup on '/' for
any request, as it hadn't been done yet. The client xlator returns ENOTCONN,
but the dht_lookup_dir_cbk changed this to ENOENT unconditionally when
aggregating. So, fuse returned ENOENT, even though the errno should have been
ENOTCONN.

backporting  http://review.gluster.org/6072

BUG: 1019095
Change-Id: Iaa40dffefddfcaf1ab7736f5423d7f9d2ece1363
Original-author: Kaushal M 
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.org/6471
Tested-by: Gluster Build System 
Reviewed-by: Harshavardhana 
Reviewed-by: Anand Avati

cluster/dht: Make sure loc has gfid

2014-01-13T12:58:45+00:00

Problem:
In some code paths neither loc->gfid nor loc->inode->gfid
is populated which leads to EINVAL for linkfile setattr
in dht_linkfile_attr_heal.

Fix:
Populate loc->gfid before dht_linkfile_attr_heal.

BUG: 971805
Change-Id: I8e4b7510ee5c38aa9ccf5283c7165c7df25ec62b
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.org/6691
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/dht: Ignore ENOENT errors for unlink of linkfiles

2013-12-24T11:32:23+00:00

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

If unlink of linkfile returns ENOENT, do not fail unlink.
Proceed with unlinking of cached file.

Change-Id: If7cec92b40c39d68dd9c3606c6c2c3a6bd67d27b
BUG: 966848
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/6586
Reviewed-by: Harshavardhana 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

Revert "core: fix errno for non-existent GFID"

2013-12-24T11:31:53+00:00

This reverts commit 837422858c2e4ab447879a4141361fd382645406

Change-Id: I0909f26ce088454bb14b3694b489c672286a4ae6
Reviewed-on: http://review.gluster.org/6575
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/dht: interim fix for reverting 837422858c

2013-12-24T09:53:48+00:00

Change-Id: I74818a03f7c5d7891561515af2fa35ea3775255c
BUG: 1032894
Signed-off-by: Vijay Bellur 
Reviewed-on: http://review.gluster.org/6582
Tested-by: Gluster Build System

cluster/dht: Ignore decommissioned subvol in overlap optimization

2013-12-17T07:15:06+00:00

Change-Id: Ib727948c6e21b19fd509f258ff0aea1c5d1a84d1
BUG: 966845
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.org/5056
Reviewed-by: Amar Tumballi 
Reviewed-by: Jeff Darcy 
Tested-by: Gluster Build System 
Reviewed-on: http://review.gluster.org/6517
Reviewed-by: Shishir Gowda 
Reviewed-by: Vijay Bellur

cluster/dht: Fix anomaly check

2013-12-14T14:25:21+00:00

We were wrongly detecting holes/overlaps for already accounted
errors. Additionally, sort should also handle zero'ed out layout

Change-Id: Ic3d13e1d735b914f9acc01fe919bc90656baea48
BUG: 1003851
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.org/5762
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Anand Avati 
Reviewed-on: http://review.gluster.org/6469
Reviewed-by: Vijay Bellur

cluster/dht: Del GF_READDIR_SKIP_DIRS key from dict for first_up

2013-12-14T13:28:32+00:00

Currently, we sent GF_READDIR_SKIP_DIRS for all subvolumes if
first_subvol != first_up_subvolume.

Also first_up_subvolume can change with-in the life of a call and
cbk. Saving the first_up_subvol in dht_local for checks.

Back porting fix http://review.gluster.org/5577

BUG: 996474
Change-Id: I67b5bbe781e12812557b569b7d0a0beba4224159
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.org/6468
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/dht: Do migration inprog/complete check only if ENOENT

2013-12-14T07:37:13+00:00

Additionally, update op_errno to the lasted failure. If failures
found in complete_check, error returned would be EUCLEAN instead
of the right failure (in this case ENOENT)

Change-Id: Ib813867f4b817af651627b9ea07b0b09fa2b26ce
BUG: 966852
Original-author: shishir gowda 
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/6495
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

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

2013-12-10T17:44:58+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

Backporting  http://review.gluster.org/6319
BUG: 1032859

Change-Id: Ic95e303a4c060900d041820d4faa68d1c4685b6a
Original-author: Anand Avati 
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.org/6470
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati