From owner-svn-src-projects@freebsd.org Wed Sep 6 22:48:44 2017 Return-Path: Delivered-To: svn-src-projects@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 0118DE1B94B for ; Wed, 6 Sep 2017 22:48:44 +0000 (UTC) (envelope-from asomers@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id C4B6175BE1; Wed, 6 Sep 2017 22:48:43 +0000 (UTC) (envelope-from asomers@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id v86MmgI9095614; Wed, 6 Sep 2017 22:48:42 GMT (envelope-from asomers@FreeBSD.org) Received: (from asomers@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id v86MmgnL095607; Wed, 6 Sep 2017 22:48:42 GMT (envelope-from asomers@FreeBSD.org) Message-Id: <201709062248.v86MmgnL095607@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: asomers set sender to asomers@FreeBSD.org using -f From: Alan Somers Date: Wed, 6 Sep 2017 22:48:42 +0000 (UTC) To: src-committers@freebsd.org, svn-src-projects@freebsd.org Subject: svn commit: r323247 - in projects/zfsd/head/tests/sys/cddl/zfs/tests: cli_root/zpool_get cli_user/zpool_list X-SVN-Group: projects X-SVN-Commit-Author: asomers X-SVN-Commit-Paths: in projects/zfsd/head/tests/sys/cddl/zfs/tests: cli_root/zpool_get cli_user/zpool_list X-SVN-Commit-Revision: 323247 X-SVN-Commit-Repository: base MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-projects@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "SVN commit messages for the src " projects" tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 06 Sep 2017 22:48:44 -0000 Author: asomers Date: Wed Sep 6 22:48:42 2017 New Revision: 323247 URL: https://svnweb.freebsd.org/changeset/base/323247 Log: Diff reduction versus Illumos' ZFS test suite Two of the zpool_get and one of the zpool_list tests have long been failing because they expected an outdated list of pool properties. Illumos has fixed this in their version of the test suite (though not in their zpool(8) man page). tests/sys/cddl/zfs/tests/cli_root/zpool_get/zpool_get.cfg tests/sys/cddl/zfs/tests/cli_user/zpool_list/zpool_list_001_pos.ksh Update the list of pool properties tests/sys/cddl/zfs/tests/cli_root/zpool_get/cleanup.ksh tests/sys/cddl/zfs/tests/cli_root/zpool_get/setup.ksh Remove checks for the current kernel's level of ZFS support. They're not necessary since the test suite now resides in the same codebase as the kernel. tests/sys/cddl/zfs/tests/cli_root/zpool_get/zpool_get_test.sh tests/sys/cddl/zfs/tests/cli_user/zpool_list/zpool_list_test.sh Remove atf_expect_fail statements Obtained from: Illumos Sponsored by: Spectra Logic Corp Modified: projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/cleanup.ksh projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/setup.ksh projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/zpool_get.cfg projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/zpool_get_test.sh projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_user/zpool_list/zpool_list_001_pos.ksh projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_user/zpool_list/zpool_list_test.sh Modified: projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/cleanup.ksh ============================================================================== --- projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/cleanup.ksh Wed Sep 6 21:38:07 2017 (r323246) +++ projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/cleanup.ksh Wed Sep 6 22:48:42 2017 (r323247) @@ -31,10 +31,4 @@ verify_runnable "both" -$ZPOOL upgrade -v 2>&1 | $GREP "bootfs pool property" > /dev/null -if [ $? -ne 0 ] -then - log_unsupported "Pool properties not supported on this release." -fi - default_cleanup Modified: projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/setup.ksh ============================================================================== --- projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/setup.ksh Wed Sep 6 21:38:07 2017 (r323246) +++ projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/setup.ksh Wed Sep 6 22:48:42 2017 (r323247) @@ -30,12 +30,6 @@ . ${STF_SUITE}/include/libtest.kshlib verify_runnable "both" -$ZPOOL upgrade -v 2>&1 | $GREP "bootfs pool property" > /dev/null -if [ $? -ne 0 ] -then - log_unsupported "Pool properties not supported on this release." -fi - DISK=${DISKS%% *} default_setup $DISK Modified: projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/zpool_get.cfg ============================================================================== --- projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/zpool_get.cfg Wed Sep 6 21:38:07 2017 (r323246) +++ projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/zpool_get.cfg Wed Sep 6 22:48:42 2017 (r323247) @@ -25,14 +25,44 @@ # ident "@(#)zpool_get.cfg 1.6 09/06/22 SMI" # -set -A properties "size" "used" "available" "capacity" "altroot" "health" \ - "guid" "version" "bootfs" "delegation" "autoreplace" \ - "cachefile" - -typeset -i i=${#properties[*]} -for prop in "failmode" "listsnapshots" "keysource" "keystatus" "autoexpand"; do - if pool_prop_exist $prop ; then - properties[$i]="$prop" - (( i = i + 1 )) - fi -done +# Set the expected properties of zpool +typeset -a properties=( + "size" + "capacity" + "altroot" + "health" + "guid" + "version" + "bootfs" + "delegation" + "autoreplace" + "cachefile" + "failmode" + "listsnapshots" + "autoexpand" + "dedupditto" + "dedupratio" + "free" + "allocated" + "readonly" + "comment" + "expandsize" + "freeing" + "fragmentation" + "leaked" + "feature@async_destroy" + "feature@empty_bpobj" + "feature@lz4_compress" + "feature@multi_vdev_crash_dump" + "feature@spacemap_histogram" + "feature@enabled_txg" + "feature@hole_birth" + "feature@extensible_dataset" + "feature@embedded_data" + "feature@bookmarks" + "feature@filesystem_limits" + "feature@large_blocks" + "feature@sha512" + "feature@skein" + # "feature@edonr" Edonr is not yet implemented on FreeBSD +) Modified: projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/zpool_get_test.sh ============================================================================== --- projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/zpool_get_test.sh Wed Sep 6 21:38:07 2017 (r323246) +++ projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_root/zpool_get/zpool_get_test.sh Wed Sep 6 22:48:42 2017 (r323247) @@ -55,7 +55,6 @@ zpool_get_002_pos_head() } zpool_get_002_pos_body() { - atf_expect_fail 'BUG26173: zpool man page and STF tests were never updated for removal of "used" property' . $(atf_get_srcdir)/../../../include/default.cfg . $(atf_get_srcdir)/zpool_get.cfg @@ -79,7 +78,6 @@ zpool_get_003_pos_head() } zpool_get_003_pos_body() { - atf_expect_fail 'BUG26173: zpool man page and STF tests were never updated for removal of "used" property' . $(atf_get_srcdir)/../../../include/default.cfg . $(atf_get_srcdir)/zpool_get.cfg Modified: projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_user/zpool_list/zpool_list_001_pos.ksh ============================================================================== --- projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_user/zpool_list/zpool_list_001_pos.ksh Wed Sep 6 21:38:07 2017 (r323246) +++ projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_user/zpool_list/zpool_list_001_pos.ksh Wed Sep 6 22:48:42 2017 (r323247) @@ -61,8 +61,8 @@ fi set -A args "list $TESTPOOL" "list -H $TESTPOOL" "list" "list -H" \ "list -H -o name $TESTPOOL" "list -o name $TESTPOOL" \ - "list -o name,size,used,available,capacity,health,altroot $TESTPOOL" \ - "list -H -o name,size,used,available,capacity,health,altroot $TESTPOOL" + "list -o name,size,capacity,health,altroot $TESTPOOL" \ + "list -H -o name,size,capacity,health,altroot $TESTPOOL" log_assert "zpool list [-H] [-o filed[,filed]*] [ ...]" Modified: projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_user/zpool_list/zpool_list_test.sh ============================================================================== --- projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_user/zpool_list/zpool_list_test.sh Wed Sep 6 21:38:07 2017 (r323246) +++ projects/zfsd/head/tests/sys/cddl/zfs/tests/cli_user/zpool_list/zpool_list_test.sh Wed Sep 6 22:48:42 2017 (r323247) @@ -34,8 +34,6 @@ zpool_list_001_pos_head() } zpool_list_001_pos_body() { - atf_expect_fail 'BUG26173: zpool man page and STF tests were never updated for removal of "used" property' - . $(atf_get_srcdir)/../../../include/default.cfg . $(atf_get_srcdir)/zpool_list.cfg