Whamcloud - gitweb
LU-9332 tests: fix pool_list issue in conf-sanity test_82b 82/26582/3
authorEmoly Liu <emoly.liu@intel.com>
Fri, 14 Apr 2017 06:18:49 +0000 (14:18 +0800)
committerOleg Drokin <oleg.drokin@intel.com>
Sun, 23 Apr 2017 03:11:34 +0000 (03:11 +0000)
In conf-sanity.sh test_82b, if we use separated MGS and MDS nodes,
MDS needs more time to list the OSTs in pool, so use wait_update
instead of the pool_list command.

Signed-off-by: Emoly Liu <emoly.liu@intel.com>
Change-Id: I8f7281d44a12731d7ae2d1f8fa9bad163086cecc
Reviewed-on: https://review.whamcloud.com/26582
Tested-by: Jenkins
Reviewed-by: Parinay Kondekar <parinay.kondekar@seagate.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/tests/conf-sanity.sh

index abf125c..363dfc8 100755 (executable)
@@ -5881,8 +5881,8 @@ test_82b() { # LU-4665
        wait_update $HOSTNAME "$LCTL get_param -n lov.$FSNAME-*.pools.$TESTNAME|
                               sort -u | tr '\n' ' ' " "$ost_targets_uuid" ||
                                        error "wait_update $ost_pool failed"
-       [[ -z $(list_pool $ost_pool) ]] &&
-                       error "list OST pool $ost_pool failed"
+       wait_update_facet $SINGLEMDS "$LCTL pool_list $ost_pool | wc -l" 4 ||
+                               error "wait_update pool_list $ost_pool failed"
 
        # If [--pool|-p <pool_name>] is set with [--ost-list|-o <ost_indices>],
        # then the OSTs must be the members of the pool.