From: Ryan Haasken Date: Wed, 18 Dec 2013 22:01:51 +0000 (-0600) Subject: LU-4396 tests: Remove unnecessary skip in sanity test_31n X-Git-Tag: 2.5.53~20 X-Git-Url: https://git.whamcloud.com/?p=fs%2Flustre-release.git;a=commitdiff_plain;h=73e816e57167eb92425b6cf29fc570e56c88f6bd;ds=sidebyside LU-4396 tests: Remove unnecessary skip in sanity test_31n Since test_31n now uses the function free_fd to find an unused file descriptor, it no longer depends on fd 173 being free. This change removes that test on whether fd 173 is in use. Signed-off-by: Ryan Haasken Change-Id: I3b68e278d4f3a8af42e57b073ccbe45b80e76ecd Reviewed-on: http://review.whamcloud.com/8622 Reviewed-by: Chris Horn Reviewed-by: Bob Glossman Reviewed-by: Patrick Farrell Tested-by: Jenkins Tested-by: Maloo Reviewed-by: Andreas Dilger --- diff --git a/lustre/tests/sanity.sh b/lustre/tests/sanity.sh index 33919d9..e2e2b96 100644 --- a/lustre/tests/sanity.sh +++ b/lustre/tests/sanity.sh @@ -2120,7 +2120,6 @@ test_31m() { run_test 31m "link to file: the same, non-existing, dir===============" test_31n() { - [ -e /proc/self/fd/173 ] && echo "skipping, fd 173 is in use" && return touch $DIR/$tfile || error "cannot create '$DIR/$tfile'" nlink=$(stat --format=%h $DIR/$tfile) [ ${nlink:--1} -eq 1 ] || error "nlink is $nlink, expected 1"