From d28ada9eaaa3676073cf3b3073597bedf70d11b1 Mon Sep 17 00:00:00 2001 From: Brian Behlendorf Date: Tue, 18 May 2010 16:15:57 -0700 Subject: [PATCH] Remove explicit import from zconfig test 1. Due to now resolved bug in the SPL you would need to explicitly import you zpools after module load. Now that is no longer the case. If a cache file is found your pool will be automatically loaded and available so I'm removing the explicit imports from the test case. --- scripts/zconfig.sh | 22 +++++++++------------- 1 file changed, 9 insertions(+), 13 deletions(-) diff --git a/scripts/zconfig.sh b/scripts/zconfig.sh index 52dfc6897a..55e4c46453 100755 --- a/scripts/zconfig.sh +++ b/scripts/zconfig.sh @@ -64,20 +64,16 @@ zconfig_test1() { ${ZPOOL_CREATE_SH} -p ${POOL_NAME} -c lo-raidz2 || fail 2 ${ZPOOL} status ${POOL_NAME} >${TMP_FILE1} || fail 3 - # Unload/load the module stack to clear any configuration state - # then verify the pool is defined in the cache file, it can be - # imported without error, and it matches the original pool. + # Unload/load the module stack and verify the pool persists. ${ZFS_SH} -u || fail 4 ${ZFS_SH} zfs="spa_config_path=${TMP_CACHE}" || fail 5 - ${ZPOOL} import -c ${TMP_CACHE} | grep ${POOL_NAME} >/dev/null||fail 6 - ${ZPOOL} import -c ${TMP_CACHE} ${POOL_NAME} || fail 7 - ${ZPOOL} status ${POOL_NAME} >${TMP_FILE2} || fail 8 - cmp ${TMP_FILE1} ${TMP_FILE2} || fail 9 + ${ZPOOL} status ${POOL_NAME} >${TMP_FILE2} || fail 6 + cmp ${TMP_FILE1} ${TMP_FILE2} || fail 7 # Cleanup the test pool and temporary files - ${ZPOOL_CREATE_SH} -p ${POOL_NAME} -c lo-raidz2 -d || fail 10 - rm -f ${TMP_FILE1} ${TMP_FILE2} ${TMP_CACHE} || fail 11 - ${ZFS_SH} -u || fail 12 + ${ZPOOL_CREATE_SH} -p ${POOL_NAME} -c lo-raidz2 -d || fail 8 + rm -f ${TMP_FILE1} ${TMP_FILE2} ${TMP_CACHE} || fail 9 + ${ZFS_SH} -u || fail 10 pass } @@ -97,9 +93,9 @@ zconfig_test2() { ${ZPOOL_CREATE_SH} -p ${POOL_NAME} -c lo-raidz2 || fail 2 ${ZPOOL} status ${POOL_NAME} >${TMP_FILE1} || fail 3 - # Unload/load the module stack to clear any configuration state - # then remove the cache file, probe the disks for pools, import - # the pool without error, and match it against the original pool. + # Unload the module stack, remove the cache file, load the module + # stack and attempt to probe the disks to import the pool. As + # a cross check verify the old pool state against the imported. ${ZFS_SH} -u || fail 4 rm -f ${TMP_CACHE} || fail 5 ${ZFS_SH} zfs="spa_config_path=${TMP_CACHE}" || fail 6