As a stop gap measure I propose to mark the test as Bad to unblock patches [1][2]. Are maintainers of snapshot in agreement with this?
regards,On Wed, Aug 1, 2018 at 10:28 AM, Raghavendra Gowdappa <rgowdapp@xxxxxxxxxx> wrote:
Sunny/Rafi,I was trying to debug regression failures on [1]. Note that patch [1] only disables usage of anonymous fds on readv. So, I tried the same test disabling performance.open-behind
[root@rhs-client27 glusterfs]# git diff
diff --git a/tests/bugs/snapshot/bug-1167580-set-proper-uid-and- gid-during-nfs-access.t b/tests/bugs/snapshot/bug- 1167580-set-proper-uid-and- gid-during-nfs-access.t
index 3776451..cedf96b 100644
--- a/tests/bugs/snapshot/bug-1167580-set-proper-uid-and- gid-during-nfs-access.t
+++ b/tests/bugs/snapshot/bug-1167580-set-proper-uid-and- gid-during-nfs-access.t
@@ -79,6 +79,7 @@ TEST $CLI volume start $V0
EXPECT_WITHIN $NFS_EXPORT_TIMEOUT "1" is_nfs_export_available
TEST glusterfs -s $H0 --volfile-id $V0 $M0
TEST mount_nfs $H0:/$V0 $N0 nolock
+TEST $CLI volume set $V0 performance.open-behind off
# Create 2 user
user1=$(get_new_user)With the above change, I can see consistent failures of the test just like observed in [1].TEST 23 (line 154): Y check_if_permitted eeefadc /mnt/glusterfs/0/.snaps/snap2/file3 cat
su: warning: cannot change directory to /tmp/tmp.eaKBKS0lfM/eeefadc: No such file or directory
cat: /mnt/glusterfs/0/.snaps/snap2/file3: Permission denied
su: warning: cannot change directory to /tmp/tmp.eaKBKS0lfM/eeefadc: No such file or directory
cat: /mnt/glusterfs/0/.snaps/snap2/file3: Permission denied
su: warning: cannot change directory to /tmp/tmp.eaKBKS0lfM/eeefadc: No such file or directory
cat: /mnt/glusterfs/0/.snaps/snap2/file3: Permission denied
su: warning: cannot change directory to /tmp/tmp.eaKBKS0lfM/eeefadc: No such file or directory
cat: /mnt/glusterfs/0/.snaps/snap2/file3: Permission denied
Test Summary Report
-------------------
./tests/bugs/snapshot/bug-1167580-set-proper-uid-and- gid-during-nfs-access.t (Wstat: 0 Tests: 46 Failed: 1)
Failed test: 23I had a feeling this test fails spuriously and the spurious nature is tied with whether open-behind uses an anonymous fd or a regular fd for read.@Sunny,This test is blocking two of my patches - [1] and [2]. Can I mark this test as bad and proceed with my work on [1] and [2]?regards,Raghavendra
_______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx https://lists.gluster.org/mailman/listinfo/gluster-devel