Whamcloud - gitweb
LU-7664 test: set_conf_param_and_check() defect 95/17995/2
authorElena Gryaznova <elena.gryaznova@seagate.com>
Thu, 14 Jan 2016 12:13:55 +0000 (14:13 +0200)
committerOleg Drokin <oleg.drokin@intel.com>
Wed, 22 Jun 2016 02:55:02 +0000 (02:55 +0000)
set_conf_param_and_check() should wait update on facet_active_host
instead of facet_host; othewise tests failed on configuration
facet_HOST != facetfailover_HOST

Seagate-bug-id: MRP-2846
Signed-off-by: Elena Gryaznova <elena.gryaznova@seagate.com>
Reviewed-by: Sergey Cheremencev <sergey.cheremencev@seagate.com>
Reviewed-by: Vitaly Fertman <vitaly.fertman@seagate.com>
Change-Id: I974f145b3783fec1adb2fc6219ef4bd025a31811
Reviewed-on: http://review.whamcloud.com/17995
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/tests/test-framework.sh

index 732eaff..a0a4f66 100755 (executable)
@@ -3848,7 +3848,7 @@ set_conf_param_and_check() {
        do_facet mgs "$LCTL conf_param $PARAM='$FINAL'" ||
                error "conf_param $PARAM failed"
 
-       wait_update $(facet_host $myfacet) "$TEST" "$FINAL" ||
+       wait_update_facet $myfacet "$TEST" "$FINAL" ||
                error "check $PARAM failed!"
 }