LU-434 Add comment for $SHARED_DIRECTORY variable

To try and avoid confusion, a comment was added to better explain how the
$SHARED_DIRECTORY variable is intended to be used. The explanation was taken
from the Lustre mailing list. Specifically, from this message by Andreas:
http://lists.lustre.org/pipermail/lustre-discuss/2011-June/015682.html

Signed-off-by: Prakash Surya <surya1@llnl.gov>
Change-Id: I594283a5963a0ac83bbe88ce8258e361408ec57a
Reviewed-on: http://review.whamcloud.com/940
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Tested-by: Hudson

diff --git a/lustre/tests/cfg/ncli.sh b/lustre/tests/cfg/ncli.sh
index 2249dd1..5e97660 100644 (file)
--- a/lustre/tests/cfg/ncli.sh
+++ b/lustre/tests/cfg/ncli.sh
@@ -14,7 +14,12 @@ init_clients_lists
 # for recovery scale tests
 # default boulder cluster iozone location
 export PATH=/opt/iozone/bin:$PATH
 # for recovery scale tests
 # default boulder cluster iozone location
 export PATH=/opt/iozone/bin:$PATH

+
+# This is used by a small number of tests to share state between the client
+# running the tests, or in some cases between the servers (e.g. lfsck.sh).
+# It needs to be a non-lustre filesystem that is available on all the nodes.

 SHARED_DIRECTORY=${SHARED_DIRECTORY:-""}       # bug 17839 comment 65
 SHARED_DIRECTORY=${SHARED_DIRECTORY:-""}       # bug 17839 comment 65

+

 LOADS=${LOADS:-"dd tar dbench iozone"}
 for i in $LOADS; do
     [ -f $LUSTRE/tests/run_${i}.sh ] || \
 LOADS=${LOADS:-"dd tar dbench iozone"}
 for i in $LOADS; do
     [ -f $LUSTRE/tests/run_${i}.sh ] || \

diff --git a/lustre/tests/test-framework.sh b/lustre/tests/test-framework.sh
index 392536a..f0218c0 100644 (file)
--- a/lustre/tests/test-framework.sh
+++ b/lustre/tests/test-framework.sh
@@ -124,6 +124,10 @@ init_test_env() {
 
     export LFSCK_ALWAYS=${LFSCK_ALWAYS:-"no"} # check fs after each test suite
     export FSCK_MAX_ERR=4   # File system errors left uncorrected
 
     export LFSCK_ALWAYS=${LFSCK_ALWAYS:-"no"} # check fs after each test suite
     export FSCK_MAX_ERR=4   # File system errors left uncorrected

+
+    # This is used by a small number of tests to share state between the client
+    # running the tests, or in some cases between the servers (e.g. lfsck.sh).
+    # It needs to be a non-lustre filesystem that is available on all the nodes.

     export SHARED_DIRECTORY=${SHARED_DIRECTORY:-"/tmp"}
     export MDSDB=${MDSDB:-$SHARED_DIRECTORY/mdsdb}
     export OSTDB=${OSTDB:-$SHARED_DIRECTORY/ostdb}
     export SHARED_DIRECTORY=${SHARED_DIRECTORY:-"/tmp"}
     export MDSDB=${MDSDB:-$SHARED_DIRECTORY/mdsdb}
     export OSTDB=${OSTDB:-$SHARED_DIRECTORY/ostdb}