diff options
author | N Balachandran <nbalacha@redhat.com> | 2015-11-09 15:49:50 +0530 |
---|---|---|
committer | Dan Lambright <dlambrig@redhat.com> | 2015-11-18 07:16:12 -0800 |
commit | a1e4be4d277c8150657f7b9fd4b72dcdd639523d (patch) | |
tree | 068b9e1924b2138cbc37175b219e94ded90ae4be /tests/bugs | |
parent | cb76371a56eb90a6f2ac126c876d7a2dc203e16c (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
BUG: 1279376
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>
Diffstat (limited to 'tests/bugs')
-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; + |