glusterfs.git/xlators/cluster/dht/src, branch v3.7.9

cluster/tier: break out of iterating query file once cycle time ends

2016-03-17T04:37:48+00:00

When iterating the query file during migration, tiering should
break out of the loop once cycle time completes. Otherwise it
may be possible to stay in the loop for a long time. If that
happens updates to files will become stale and have not impact
migration.

This is a backport of 13284

> Change-Id: Ib60cf74bc84e8646e6a0da21ff04954b1b83c414
> BUG: 1301227
> Signed-off-by: Dan Lambright 
> Reviewed-on: http://review.gluster.org/13284
> Smoke: Gluster Build System 
> NetBSD-regression: NetBSD Build System 
> CentOS-regression: Gluster Build System 
Signed-off-by: Dan Lambright 

Change-Id: Ia305784b0aa570b70e54f532bf5d68a91b0cf305
BUG: 1318203
Reviewed-on: http://review.gluster.org/13750
Smoke: Gluster Build System 
Reviewed-by: hari gowtham 
Tested-by: Dan Lambright 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Dan Lambright

cluster/tier: Additional details in error messages

2016-03-07T22:15:53+00:00

Added file path/gfid when available to the tier log
messages to make debugging easier.

> Change-Id: I22dda329367df2b846dcf254594312c997b66083
> BUG: 1273043
> Signed-off-by: N Balachandran 
> Reviewed-on: http://review.gluster.org/13114
> Reviewed-by: mohammed rafi  kc 
> Tested-by: Gluster Build System 
> Reviewed-by: Dan Lambright 
> Tested-by: Dan Lambright 

Change-Id: I5cbf8407fb7c493c4be1dda65c0f722ede33487f
BUG: 1311865
Signed-off-by: N Balachandran 
Reviewed-on: http://review.gluster.org/13518
Smoke: Gluster Build System 
Reviewed-by: mohammed rafi  kc 
Reviewed-by: Dan Lambright 
CentOS-regression: Gluster Build System 
NetBSD-regression: NetBSD Build System 
Tested-by: Dan Lambright

tier: Fix unused-but-set-variable warning

2016-03-04T03:44:40+00:00

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

Change-Id: I20c205849f511acf4797a8a53ea59bfaad60bd35
BUG: 1314548
Signed-off-by: Raghavendra Talur 
Reviewed-on: http://review.gluster.org/13598
Smoke: Gluster Build System 
CentOS-regression: Gluster Build System 
NetBSD-regression: NetBSD Build System 
Reviewed-by: Pranith Kumar Karampuri

dht: mkdir must unwind with latest ctime

2016-03-01T04:54:12+00:00

Currently fops like mkdir used the the ctime it gets
after creating the directory entry. But setting layout
also updates the ctime of a directory. Hence DHT must
get the ctime after the setxattr call and unwind with
the latest ctime to avoid mismatch in time seen by
applications like tar.

Backport of http://review.gluster.org/#/c/13352/

> Change-Id: Iecbbe3aac5244af5da9788b48ccf299ca56b4bae
> BUG: 1302948
> Signed-off-by: Sakshi Bansal 
> Reviewed-on: http://review.gluster.org/13352
> Smoke: Gluster Build System 
> CentOS-regression: Gluster Build System 
> NetBSD-regression: NetBSD Build System 
> Reviewed-by: Raghavendra G 

BUG: 1312721
Change-Id: Ie09342d3705b40ce98b2935f05ad4402f74ba069
Signed-off-by: Sakshi Bansal 
Reviewed-on: http://review.gluster.org/13536
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Raghavendra G

cluster/tier: Create linkfiles to hardlinks correctly

2016-02-27T18:48:50+00:00

There is a bug in the way hardlinks are handled in tiered volumes.
Ideally, the tier linkto files on the cold tier to files that are
hardlinks to each other on the hot tier, should themselves be
hardlinks of each other. As they are not, they end up being files
with the same gfid but different names for the cold tier dht,
and end up overwriting the cached-subvol information stored in the dht inode-ctx.

> Change-Id: Ic658a316836e6a1729cfea848b7d212674b0edd2
> BUG: 1305277
> Signed-off-by: N Balachandran 
> Reviewed-on: http://review.gluster.org/13391
> Smoke: Gluster Build System 
> NetBSD-regression: NetBSD Build System 
> CentOS-regression: Gluster Build System 
> Reviewed-by: Dan Lambright 
> Tested-by: Dan Lambright 
(cherry picked from commit eb362c74db84d95aac07febf0d888bd98b3fb2b9)

Change-Id: I5dd98e6d248619147974a630d7d72e1942a1cc83
BUG: 1311836
Signed-off-by: N Balachandran 
Reviewed-on: http://review.gluster.org/13517
Smoke: Gluster Build System 
Reviewed-by: mohammed rafi  kc 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Dan Lambright

cluster/tier : Fixed wrong variable comparison

2016-02-27T18:48:12+00:00

The wrong variable was being checked to determine
the watermark value.

> Change-Id: If4c97fa70b772187f1fcbdf5193e077cb356a8b1
> BUG: 1303895
> Signed-off-by: N Balachandran 
> Reviewed-on: http://review.gluster.org/13357
> Smoke: Gluster Build System 
> Reviewed-by: Dan Lambright 
> NetBSD-regression: NetBSD Build System 
> CentOS-regression: Gluster Build System 
Signed-off-by: N Balachandran 

Change-Id: I0a98e0efbc093a727912107038477239e6d85765
BUG: 1306129
Reviewed-on: http://review.gluster.org/13516
Smoke: Gluster Build System 
Reviewed-by: mohammed rafi  kc 
Tested-by: mohammed rafi  kc 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Dan Lambright

dht: cleanup dict and free memory in rename code path

2016-02-26T03:19:23+00:00

Backport of http://review.gluster.org/#/c/13392/

> Change-Id: I2458e18197bdf7565563a85e9021b5b2850c1825
> BUG: 1303945
> Signed-off-by: Sakshi Bansal 
> Reviewed-on: http://review.gluster.org/13392
> Smoke: Gluster Build System 
> Reviewed-by: Xavier Hernandez 
> CentOS-regression: Gluster Build System 
> NetBSD-regression: NetBSD Build System 
> Reviewed-by: Raghavendra G 

BUG: 1304889
Change-Id: Ibda96769be2bd5b2708066d83363633fd7286929
Signed-off-by: Sakshi Bansal 
Reviewed-on: http://review.gluster.org/13513
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Raghavendra G

dht: loc should store proper gfid

2016-02-23T08:28:08+00:00

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

> Change-Id: Ic1393d44a9ed4aaba23d7c9ddea45977b9dae5e4
> BUG: 1281265
> Signed-off-by: Sakshi Bansal 
> Reviewed-on: http://review.gluster.org/12574
> Tested-by: NetBSD Build System 
> Tested-by: Gluster Build System 
> Reviewed-by: Raghavendra G 

BUG: 1308410
Change-Id: I6577971f3db139eccf3b8ca3b25e66d292a6a150
Signed-off-by: Sakshi Bansal 
Reviewed-on: http://review.gluster.org/13443
Smoke: Gluster Build System 
CentOS-regression: Gluster Build System 
NetBSD-regression: NetBSD Build System 
Reviewed-by: Raghavendra G

dht: file rename must take blocking inode locks

2016-02-22T18:12:07+00:00

Backport of http://review.gluster.org/#/c/13366

Currently DHT takes non-blocking locks for file rename.
Due to this during parallel renames some clients fail
with EBUSY or ESTALE errors. Hence to avoid application
discontinuity file rename must take blocking inode locks.

> Change-Id: I986e9d08b3be359f20b1a3e1564e049b0f3dffd3
> BUG: 1304966
> Signed-off-by: Sakshi Bansal 
> Reviewed-on: http://review.gluster.org/13366
> Smoke: Gluster Build System 
> NetBSD-regression: NetBSD Build System 
> CentOS-regression: Gluster Build System 
> Reviewed-by: Raghavendra G 

Change-Id: I986e9d08b3be359f20b1a3e1564e049b0f3dffd3
BUG: 1310544
Signed-off-by: Sakshi Bansal 
Reviewed-on: http://review.gluster.org/13481
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Raghavendra G

tier/create: store TIER_LINKFILE_GFID in xattr dictionary

2016-02-22T14:54:15+00:00

In tier_create, a new key TIER_LINKFILE_GFID was introduced to
avoid a race in stale linkfile deletion. Storing this key in
xattr dictionary instead of using local->params dictionary.

Because local->params dictionary was also used to create the file
before stale linkfile deletion, that leads posix_create to fail,
trying to set the added key as extended attributes

Back port of>
>Change-Id: I24fecb62b47bee65a1e86103925a67d13304c5df
>BUG: 1290677
>Signed-off-by: Mohammed Rafi KC 
>Reviewed-on: http://review.gluster.org/13130
>Tested-by: Gluster Build System 
>Reviewed-by: N Balachandran 
>Reviewed-by: Dan Lambright 
>Tested-by: Dan Lambright 

Change-Id: I301c159ee51c77d3876e30f216d0375d761eb91e
BUG: 1295359
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/13410
Smoke: Gluster Build System 
CentOS-regression: Gluster Build System 
NetBSD-regression: NetBSD Build System 
Reviewed-by: Dan Lambright