]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
qa: avoid using sudo in fsstress
authorJohn Spray <john.spray@redhat.com>
Wed, 23 Sep 2015 11:58:46 +0000 (12:58 +0100)
committerGreg Farnum <gfarnum@redhat.com>
Wed, 7 Oct 2015 17:44:43 +0000 (10:44 -0700)
This test required root in order to copy its built
binary into /usr (presumably to avoid rebuilding it).

That's not really a good thing anyway because there's
no guarantee that a binary in that path is the binary
we wanted, so just run the thing straight out of /tmp.  The
build is really quick anyway.

Signed-off-by: John Spray <john.spray@redhat.com>
Reviewed-by: Greg Farnum <gfarnum@redhat.com>
qa/workunits/suites/fsstress.sh

index 1951106ae621432c3ed7c79be202149059f813dd..92e123b99bb676d001a3e5401c161d2117d57b36 100755 (executable)
@@ -1,23 +1,19 @@
 #!/bin/bash
 
-if [ ! -f /usr/lib/ltp/testcases/bin/fsstress ]
-then
-    path=`pwd`
-    mkdir -p /tmp/fsstress
-    cd /tmp/fsstress
-    wget -q -O /tmp/fsstress/ltp-full.tgz http://download.ceph.com/qa/ltp-full-20091231.tgz
-    tar xzf /tmp/fsstress/ltp-full.tgz
-    rm /tmp/fsstress/ltp-full.tgz
-    cd /tmp/fsstress/ltp-full-20091231/testcases/kernel/fs/fsstress
-    make
-    sudo mkdir -p /usr/lib/ltp/testcases/bin
-    sudo cp -av --remove-destination /tmp/fsstress/ltp-full-20091231/testcases/kernel/fs/fsstress/fsstress /usr/lib/ltp/testcases/bin/fsstress
-    sudo chmod 755 /usr/lib/ltp/testcases/bin/fsstress
-    rm -Rf /tmp/fsstress
-    cd $path
-fi
+BIN_PATH=${TESTDIR}/fsstress/ltp-full-20091231/testcases/kernel/fs/fsstress/fsstress
 
-command="/usr/lib/ltp/testcases/bin/fsstress -d fsstress-`hostname`$$ -l 1 -n 1000 -p 10 -v"
+path=`pwd`
+trap "rm -rf ${TESTDIR}/fsstress" EXIT
+mkdir -p ${TESTDIR}/fsstress
+cd ${TESTDIR}/fsstress
+wget -q -O ${TESTDIR}/fsstress/ltp-full.tgz http://download.ceph.com/qa/ltp-full-20091231.tgz
+tar xzf ${TESTDIR}/fsstress/ltp-full.tgz
+rm ${TESTDIR}/fsstress/ltp-full.tgz
+cd ${TESTDIR}/fsstress/ltp-full-20091231/testcases/kernel/fs/fsstress
+make
+cd $path
+
+command="${BIN_PATH} -d fsstress-`hostname`$$ -l 1 -n 1000 -p 10 -v"
 
 echo "Starting fsstress $command"
 mkdir fsstress`hostname`-$$