X-Git-Url: https://git.whamcloud.com/?a=blobdiff_plain;f=lustre%2Ftests%2Fruntests;h=f6b3cd21cddb0ea0c68f49356305c31d072b7e08;hb=71eb67f041732093696781cc3ddddb7d9aac66a6;hp=50caa10521960088f3bfdce9e8137a8556c7901c;hpb=2e65c3da684da931fd7c8e0c97fec328188da995;p=fs%2Flustre-release.git

diff --git a/lustre/tests/runtests b/lustre/tests/runtests
index 50caa10..f6b3cd2 100755
--- a/lustre/tests/runtests
+++ b/lustre/tests/runtests
@@ -1,4 +1,4 @@
-#!/bin/sh
+#!/bin/bash
 #
 # Script which does some basic tests to ensure we haven't regressed.
 # Probably a good idea to run this before doing any checkins.
@@ -13,11 +13,11 @@ export NAME=${NAME:-local}
 . $LUSTRE/tests/test-framework.sh
 init_test_env $@
 . ${CONFIG:=$LUSTRE/tests/cfg/$NAME.sh}
+init_logging
 
-ERROR=
 RUNTESTS_SRC=${RUNTESTS_SRC:-"/etc /bin"}
-[ "$COUNT" ] || COUNT=1000
-[ "$SLOW" = "no" ] && COUNT=100
+[ "$COUNT" ] || COUNT=10000
+[ "$SLOW" = "no" ] && COUNT=1000
 
 [ "$MCREATE" ] || MCREATE=mcreate
 
@@ -54,7 +54,7 @@ if [ $COUNT -gt 10 -o $COUNT -eq 0 ]; then
 	rm $HOSTS || error "can't remove $HOSTS" 9
 	cp /etc/hosts $HOSTS.2 || error "can't cp /etc/hosts to $HOSTS.2" 7
 	log "truncating $HOSTS.2 to $TRUNCSIZE bytes"
-	truncate $HOSTS.2 $TRUNCSIZE
+	$TRUNCATE $HOSTS.2 $TRUNCSIZE
 fi
 
 DST=$MOUNT/runtest.$$
@@ -64,31 +64,36 @@ mkdir $DST || error "can't mkdir $DST" 10
 
 # ok, that hopefully worked, so let's do a little more, with files that
 # haven't changed in the last day (hopefully they don't change during test)
-FILES=`find $RUNTESTS_SRC -type f -mtime +1 | head -n $COUNT`
-[ -z "$FILES" ] && error "No unchanged files - is $RUNTESTS_SRC a new dir?"
+FILES=$TMP/runtests.files
+# use "NUL" instead of newline as filename terminator, bug 19702 
+find $RUNTESTS_SRC -type f -mtime +1 | head -n $COUNT | tr '\n' '\0' > $FILES
+[ -s "$FILES" ] ||
+	error "$RUNTESTS_SRC contains only files modifed less than 2 days ago"
+COUNT=$(xargs -0 -n 1 echo < $FILES | wc -l)
 
-log "copying files from $RUNTESTS_SRC to $DST$RUNTESTS_SRC at `date`"
-tar cf - $FILES | tar xvf - -C $DST > /dev/null || error "copying $RUNTESTS_SRC" 11
+log "copying $COUNT files from $RUNTESTS_SRC to $DST$RUNTESTS_SRC at `date`"
+tar cf - --null --files-from $FILES | tar xvf - -C $DST > /dev/null ||
+	error "copying $RUNTESTS_SRC" 11
 
-log "comparing newly copied files at `date`"
-for f in $FILES; do
+log "comparing $COUNT newly copied files at `date`"
+
+cat $FILES | tr "\0" "\n" | ( rc=0; while read f; do
 	[ $V ] && log "verifying $DST/$f"
-	diff -q $f $DST/$f || ERROR=11
+	diff -q "$f" "$DST/$f" || rc=11
 done
+[ "$rc" = 0 ] || error_exit "old and new files are different: rc=$rc" ) 
 
-[ "$ERROR" ] && error "old and new files are different" $ERROR
 log "finished at `date` ($(($(date +%s) - START)))"
 
 stopall || exit 19
 setupall || exit 20
 
-log "comparing previously copied files"
-for f in $FILES; do
+log "comparing $COUNT previously copied files"
+cat $FILES | tr "\0" "\n" | ( rc=0; while read f; do
 	[ $V ] && log "verifying $DST/$f"
-	diff -q $f $DST/$f || ERROR=22
+	diff -q "$f" "$DST/$f" || rc=22
 done
-
-[ "$ERROR" ] && error "old and new files are different on second diff" $ERROR
+[ "$rc" = 0 ] || error_exit "old and new files are different: rc=$rc" )
 
 stopall || exit 21
 setupall || exit 22
@@ -112,7 +117,7 @@ fi
 # mkdirmany test (bug 589)
 log "running $MKDIRMANY $MOUNT/base$$ 100"
 $MKDIRMANY $MOUNT/base$$ 100 || error "mkdirmany failed"
-log "removing mkdirmany directories"
+log "removing $MKDIRMANY directories"
 rmdir $MOUNT/base$$* || error "mkdirmany cleanup failed"
 
 log "done"
@@ -122,4 +127,7 @@ if [ `expr $NOWUSED - $USED` -gt 1024 ]; then
 	echo "Space not all freed: now ${NOWUSED}kB, was ${USED}kB." 1>&2
 fi
 
+complete $SECONDS
+rm -f $FILES
 check_and_cleanup_lustre
+exit_status