Whamcloud - gitweb
LU-2407 tests: remove the leading space in the fake nid 79/6779/2
authorJian Yu <jian.yu@intel.com>
Wed, 26 Jun 2013 09:22:56 +0000 (17:22 +0800)
committerOleg Drokin <oleg.drokin@intel.com>
Wed, 10 Jul 2013 02:59:50 +0000 (02:59 +0000)
This patch fixes conf-sanity test_35a() and test_35b() to remove
the leading space ahead of the fake failover nid.

Test-Parameters: envdefinitions=SLOW=yes,ENABLE_QUOTA=yes \
clientdistro=el6 serverdistro=el6 clientarch=x86_64 \
serverarch=x86_64 serverjob=lustre-b2_3 serverbuildno=41 \
testlist=conf-sanity

Signed-off-by: Jian Yu <jian.yu@intel.com>
Change-Id: I2dd5b0274abc2aea7a5e7ee3599fbef7bcb16037
Reviewed-on: http://review.whamcloud.com/6779
Tested-by: Hudson
Reviewed-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/tests/conf-sanity.sh

index c14debe..23b5eaa 100644 (file)
@@ -2008,9 +2008,10 @@ test_35a() { # bug 12459
 
        log "Set up a fake failnode for the MDS"
        FAKENID="127.0.0.2"
 
        log "Set up a fake failnode for the MDS"
        FAKENID="127.0.0.2"
-       local device=$(do_facet $SINGLEMDS "lctl get_param -n devices" | awk '($3 ~ "mdt" && $4 ~ "MDT") { print $4 }' | head -1)
-       do_facet mgs "$LCTL conf_param ${device}.failover.node=" \
-               "$(h2$NETTYPE $FAKENID)" || return 4
+       local device=$(do_facet $SINGLEMDS "lctl get_param -n devices" |
+               awk '($3 ~ "mdt" && $4 ~ "MDT") { print $4 }' | head -1)
+       do_facet mgs "$LCTL conf_param \
+               ${device}.failover.node=$(h2$NETTYPE $FAKENID)" || return 4
 
        log "Wait for RECONNECT_INTERVAL seconds (10s)"
        sleep 10
 
        log "Wait for RECONNECT_INTERVAL seconds (10s)"
        sleep 10
@@ -2062,10 +2063,10 @@ test_35b() { # bug 18674
 
        log "Set up a fake failnode for the MDS"
        FAKENID="127.0.0.2"
 
        log "Set up a fake failnode for the MDS"
        FAKENID="127.0.0.2"
-       local device=$(do_facet $SINGLEMDS "$LCTL get_param -n devices" | \
-                       awk '($3 ~ "mdt" && $4 ~ "MDT") { print $4 }' | head -1)
-       do_facet mgs "$LCTL conf_param ${device}.failover.node=" \
-               "$(h2$NETTYPE $FAKENID)" || return 1
+       local device=$(do_facet $SINGLEMDS "$LCTL get_param -n devices" |
+               awk '($3 ~ "mdt" && $4 ~ "MDT") { print $4 }' | head -1)
+       do_facet mgs "$LCTL conf_param \
+               ${device}.failover.node=$(h2$NETTYPE $FAKENID)" || return 1
 
        local at_max_saved=0
        # adaptive timeouts may prevent seeing the issue
 
        local at_max_saved=0
        # adaptive timeouts may prevent seeing the issue