From: Elena Gryaznova <elena.gryaznova@seagate.com>
Date: Thu, 14 Jan 2016 12:13:55 +0000 (+0200)
Subject: LU-7664 test: set_conf_param_and_check() defect
X-Git-Tag: 2.8.56~125
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F95%2F17995%2F2;p=fs%2Flustre-release.git

LU-7664 test: set_conf_param_and_check() defect

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>
---

diff --git a/lustre/tests/test-framework.sh b/lustre/tests/test-framework.sh
index 732eaff..a0a4f66 100755
--- a/lustre/tests/test-framework.sh
+++ b/lustre/tests/test-framework.sh
@@ -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!"
 }