From 6a565219fb1631e9b14c676458c8c04251886494 Mon Sep 17 00:00:00 2001 From: Mohammed Rafi KC Date: Mon, 30 Nov 2015 19:02:54 +0530 Subject: tier:unlink during migration files deleted during promotion were not deleting as the files are moving from hashed to non-hashed On deleting a file that is undergoing promotion, the unlink call is not sent to the dst file as the hashed subvol == cached subvol. This causes the file to reappear once the migration is complete. This patch also fixes a problem with stale linkfile deleting. Backport of> >Change-Id: I4b02a498218c9d8eeaa4556fa4219e91e7fa71e5 >BUG: 1282390 >Signed-off-by: Mohammed Rafi KC >Reviewed-on: http://review.gluster.org/12829 >Tested-by: NetBSD Build System >Tested-by: Gluster Build System >Reviewed-by: Dan Lambright >Tested-by: Dan Lambright (cherry picked from commit b5de382afa8c5777e455c7a376fc4f1f01d782d1) Change-Id: I951adb4d929926bcd646dd7574f7a2d41d57479d BUG: 1282388 Signed-off-by: Mohammed Rafi KC Reviewed-on: http://review.gluster.org/12991 Smoke: Gluster Build System NetBSD-regression: NetBSD Build System CentOS-regression: Gluster Build System Reviewed-by: Dan Lambright --- tests/basic/tier/unlink-during-migration.t | 91 ++++++++++++++++++++++++++++++ 1 file changed, 91 insertions(+) create mode 100755 tests/basic/tier/unlink-during-migration.t (limited to 'tests/basic') diff --git a/tests/basic/tier/unlink-during-migration.t b/tests/basic/tier/unlink-during-migration.t new file mode 100755 index 00000000000..78887433ccb --- /dev/null +++ b/tests/basic/tier/unlink-during-migration.t @@ -0,0 +1,91 @@ +#!/bin/bash + +. $(dirname $0)/../../include.rc +. $(dirname $0)/../../volume.rc +. $(dirname $0)/../../tier.rc + + +DEMOTE_FREQ=5 +PROMOTE_FREQ=5 + +function create_dist_rep_vol () { + mkdir $B0/cold + mkdir $B0/hot + TEST $CLI volume create $V0 replica 2 $H0:$B0/cold/${V0}{0..3} + TEST $CLI volume set $V0 performance.quick-read off + TEST $CLI volume set $V0 performance.io-cache off + TEST $CLI volume set $V0 features.ctr-enabled on + TEST $CLI volume start $V0 +} + +function attach_dist_rep_tier () { + TEST $CLI volume attach-tier $V0 replica 2 $H0:$B0/hot/${V0}{0..3} + 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.read-freq-threshold 0 + TEST $CLI volume set $V0 cluster.write-freq-threshold 0 + TEST $CLI volume set $V0 cluster.tier-mode test +} + +cleanup; + +#Basic checks +TEST glusterd +TEST pidof glusterd +TEST $CLI volume info + + +#Create and start a volume +create_dist_rep_vol + +# Mount FUSE +TEST glusterfs -s $H0 --volfile-id $V0 $M0 + +# Create a large file (320MB), so that rebalance takes time +TEST dd if=/dev/zero of=$M0/foo bs=64k count=5120 + +# Get the path of the file on the cold tier +CPATH=`find $B0/cold/ -name foo` +echo "File path on cold tier: "$CPATH + +#Now attach the tier +attach_dist_rep_tier + +#Write into the file to promote it +echo "good morning">>$M0/foo + +# Wait for the tier process to promote the file +EXPECT_WITHIN $REBALANCE_TIMEOUT "yes" is_sticky_set $CPATH + +# Get the path of the file on the hot tier +HPATH=`find $B0/hot/ -name foo` + +echo "File path on hot tier: "$HPATH +TEST rm -rf $M0/foo +TEST ! stat $HPATH +TEST ! stat $CPATH + +#unlink during demotion +HPATH=""; +CPATH=""; + +# Create a large file (320MB), so that rebalance takes time +TEST dd if=/dev/zero of=$M0/foo1 bs=64k count=5120 + +# Get the path of the file on the hot tier +HPATH=`find $B0/hot/ -name foo1` +echo "File path on hot tier : "$HPATH + +EXPECT_WITHIN $REBALANCE_TIMEOUT "yes" is_sticky_set $HPATH + +# Get the path of the file on the cold tier +CPATH=`find $B0/cold/ -name foo1` +echo "File path on cold tier : "$CPATH + +TEST rm -rf $M0/foo1 + +TEST ! stat $HPATH +TEST ! stat $CPATH + +cleanup; + -- cgit