]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
tests: save 9 characters for asok paths 12151/head
authorLoic Dachary <ldachary@redhat.com>
Fri, 18 Nov 2016 07:06:02 +0000 (08:06 +0100)
committerLoic Dachary <ldachary@redhat.com>
Wed, 23 Nov 2016 07:53:18 +0000 (08:53 +0100)
For vstart.sh powered tests, save 9 characters in the path name
by replacing testdir/test- with td/t-

60 characters imposed by jenkins
9 characters for src/test
5 characters for td/t-

33 left (instead of 24) for the test to create asok such as out/client.admin.25327.asok

Moving these files outside of the build directory is a bad idea because
tests should only create/use files within the builddir and not write
outside of this directory. Doing so would make things more complicated
for cleanup in case the test fail and create other problems as a
consequence (filling out disk space, conflicting directories between
runs etc.).

For ceph-helpers.sh tests replace testdir with td, saving 5 characters.
This is not strictly necessary but keeps the directory names consistent:
if the developer wants to get rid of all the test leftovers, it is
enough to remove the a single directory: td.

Fixes: http://tracker.ceph.com/issues/16014
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit cd72ff9f746fe13a245324dce2aa457d3c876331)

Conflicts:
.gitignore : test directories are not ignored in jewel
src/test/vstart_wrapper.sh : trivial context resolution

qa/workunits/ceph-helpers.sh
src/test/vstart_wrapper.sh

index 4f5f8f802621386687dc4f62aa18009668de3615..9947217591141f2c1373d23c2253dc9f7998b612 100755 (executable)
@@ -1442,7 +1442,7 @@ function test_wait_background() {
 # @return 0 on success, 1 on error
 #
 function main() {
-    local dir=testdir/$1
+    local dir=td/$1
     shift
 
     shopt -s -o xtrace
@@ -1481,7 +1481,7 @@ function run_tests() {
     export CEPH_CONF=/dev/null
 
     local funcs=${@:-$(set | sed -n -e 's/^\(test_[0-9a-z_]*\) .*/\1/p')}
-    local dir=testdir/ceph-helpers
+    local dir=td/ceph-helpers
 
     for func in $funcs ; do
         $func $dir || return 1
index 5c5a3501e9285e16548924161e0f25dff2a4ea61..a555855c4ef690b8ddae406e558931666568aa28 100755 (executable)
@@ -19,7 +19,7 @@
 source $CEPH_ROOT/qa/workunits/ceph-helpers.sh
 
 export CEPH_VSTART_WRAPPER=1
-export CEPH_DIR="$PWD/testdir/test-$CEPH_PORT"
+export CEPH_DIR="$PWD/td/t-$CEPH_PORT"
 export CEPH_DEV_DIR="$CEPH_DIR/dev"
 export CEPH_OUT_DIR="$CEPH_DIR/out"