Whamcloud - gitweb
LU-196 set $PTLDEBUG, $SUBSYSTEM and $DEBUG_SIZE values on every node
authorYu Jian <yujian@whamcloud.com>
Tue, 24 May 2011 08:23:06 +0000 (16:23 +0800)
committerJohann Lombardi <johann@whamcloud.com>
Tue, 24 May 2011 20:59:22 +0000 (13:59 -0700)
The current set_default_debug_nodes() could not pass the values of
$PTLDEBUG, $SUBSYSTEM and $DEBUG_SIZE to the remote nodes while they
are specified from the command line on the local node. This patch is
to fix this issue.

Signed-off-by: Yu Jian <yujian@whamcloud.com>
Change-Id: I85922c67e7c68a5ea3e9ba7fc1cbf6516907c285
Reviewed-on: http://review.whamcloud.com/593
Tested-by: Hudson
Reviewed-by: Chris Gearing <chris@whamcloud.com>
Reviewed-by: Yang Sheng <ys@whamcloud.com>
Reviewed-by: Johann Lombardi <johann@whamcloud.com>
lustre/tests/test-framework.sh

index feaff4e..ecd283e 100644 (file)
@@ -414,10 +414,14 @@ set_debug_size () {
 }
 
 set_default_debug () {
-    lctl set_param debug=$PTLDEBUG
-    lctl set_param subsystem_debug=${SUBSYSTEM# }
+    local debug=${1:-"$PTLDEBUG"}
+    local subsystem_debug=${2:-"$SUBSYSTEM"}
+    local debug_size=${3:-$DEBUG_SIZE}
 
-    set_debug_size $DEBUG_SIZE
+    lctl set_param debug="$debug"
+    lctl set_param subsystem_debug="${subsystem_debug# }"
+
+    set_debug_size $debug_size
     sync
 }
 
@@ -430,7 +434,8 @@ set_default_debug_nodes () {
     local nodes=$1
 
     if remote_node $nodes; then
-       do_rpc_nodes $nodes set_default_debug
+       do_rpc_nodes $nodes set_default_debug \
+            \\\"$PTLDEBUG\\\" \\\"$SUBSYSTEM\\\" $DEBUG_SIZE
     else
        set_default_debug
     fi
@@ -438,8 +443,10 @@ set_default_debug_nodes () {
 
 set_default_debug_facet () {
     local facet=$1
+    local node=$(facet_active_host $facet)
+    [ -z "$node" ] && echo "No host defined for facet $facet" && exit 1
 
-    set_default_debug_nodes $(facet_host ${facet})
+    set_default_debug_nodes $node
 }
 
 # Facet functions