diff options
author | N Balachandran <nbalacha@redhat.com> | 2015-11-09 15:49:50 +0530 |
---|---|---|
committer | Dan Lambright <dlambrig@redhat.com> | 2015-11-19 10:05:27 -0800 |
commit | 79103a0f9eea2af4c68710e602f78963471ecec7 (patch) | |
tree | 4ad86bfd1f7a49f03df3713548f401439e46d2d5 /tests | |
parent | e24fb2278bc0b0da88ec8c7b2d873c3e4a864d9d (diff) |
cluster/tier: Do not delete linkto file on demotion
The current DHT migration code will always delete the
src linkto file after migration as dht always moves
files to the hashed subvol. This is not the case in tiering.
The lack of linkto files causes rename to fail leaving 2 files
with the same name but different gfids on the volume.
Modified to leave the linkto file behind if the source
volume is the hashed subvolume.
> Change-Id: I2b99f7d34b4b719aee6232dc40c6a8f8ba88225d
> Signed-off-by: N Balachandran <nbalacha@redhat.com>
> Reviewed-on: http://review.gluster.org/12551
> Reviewed-by: Dan Lambright <dlambrig@redhat.com>
> Tested-by: Dan Lambright <dlambrig@redhat.com>
Change-Id: I210b94cdae0409c87af8ba198e3cd263a6c85190
BUG: 1283480
Signed-off-by: N Balachandran <nbalacha@redhat.com>
Reviewed-on: http://review.gluster.org/12655
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Dan Lambright <dlambrig@redhat.com>
Tested-by: Dan Lambright <dlambrig@redhat.com>
Diffstat (limited to 'tests')
-rwxr-xr-x | tests/bugs/tier/bug-1279376-rename-demoted-file.t | 88 |
1 files changed, 88 insertions, 0 deletions
diff --git a/tests/bugs/tier/bug-1279376-rename-demoted-file.t b/tests/bugs/tier/bug-1279376-rename-demoted-file.t new file mode 100755 index 00000000000..c75c398ce28 --- /dev/null +++ b/tests/bugs/tier/bug-1279376-rename-demoted-file.t @@ -0,0 +1,88 @@ +#!/bin/bash + +. $(dirname $0)/../../include.rc +. $(dirname $0)/../../volume.rc +. $(dirname $0)/../../tier.rc + + +NUM_BRICKS=2 +DEMOTE_FREQ=5 +DEMOTE_TIMEOUT=10 +PROMOTE_FREQ=500 + + +#Both src and dst files must hash to the same hot tier subvol +SRC_FILE="file1.txt" +DST_FILE="newfile1.txt" + + +# Creates a tiered volume with pure distribute hot and cold tiers +# Both hot and cold tiers will have an equal number of bricks. + +function create_dist_tier_vol () { + mkdir $B0/cold + mkdir $B0/hot + TEST $CLI volume create $V0 $H0:$B0/cold/${V0}{0..$1} + TEST $CLI volume set $V0 performance.quick-read off + TEST $CLI volume set $V0 performance.io-cache off + TEST $CLI volume start $V0 + TEST $CLI volume tier $V0 attach $H0:$B0/hot/${V0}{0..$1} + TEST $CLI volume set $V0 cluster.tier-demote-frequency $DEMOTE_FREQ + TEST $CLI volume set $V0 cluster.tier-promote-frequency $PROMOTE_FREQ + TEST $CLI volume set $V0 cluster.tier-mode test + +#We do not want any files to be promoted during this test + TEST $CLI volume set $V0 cluster.read-freq-threshold 50 + TEST $CLI volume set $V0 cluster.write-freq-threshold 50 +} + + +cleanup; + +#Basic checks +TEST glusterd +TEST pidof glusterd +TEST $CLI volume info + + +#Create and start a tiered volume +create_dist_tier_vol $NUM_BRICKS + +# Mount FUSE +TEST glusterfs -s $H0 --volfile-id $V0 $M0 + + +# The file will be created on the hot tier + +touch "$M0/$SRC_FILE" + +# Get the path of the file on the hot tier +HPATH=`find $B0/hot/ -name "$SRC_FILE"` +echo "File path on hot tier: "$HPATH + + +EXPECT "yes" exists_and_regular_file $HPATH + +# Wait for the tier process to demote the file +sleep $DEMOTE_TIMEOUT + +# Get the path of the file on the cold tier +CPATH=`find $B0/cold/ -name "$SRC_FILE"` +echo "File path on cold tier: "$CPATH + +EXPECT "yes" exists_and_regular_file $CPATH + +#This will be created on the hot tier + +touch "$M0/$DST_FILE" +HPATH=`find $B0/hot/ -name "$DST_FILE"` +echo "File path on hot tier: "$HPATH + +TEST mv $M0/$SRC_FILE $M0/$DST_FILE + +# We expect a single file to exist at this point +# when viewed on the mountpoint +EXPECT 1 echo $(ls -l $M0 | grep $DST_FILE | wc -l) + +cleanup; + |