7290 ZFS test suite needs to control what utilities it can run
Reviewed by: Dan Kimmel <dan.kimmel@delphix.com>
Reviewed by: Matthew Ahrens <mahrens@delphix.com>

          --- old/usr/src/test/zfs-tests/tests/functional/reservation/reservation_015_pos.sh
          +++ new/usr/src/test/zfs-tests/tests/functional/reservation/reservation_015_pos.sh

   1    1  #!/usr/bin/bash -p
   2    2  #
   3    3  # CDDL HEADER START
   4    4  #
   5    5  # The contents of this file are subject to the terms of the
   6    6  # Common Development and Distribution License (the "License").
   7    7  # You may not use this file except in compliance with the License.
   8    8  #
   9    9  # You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE
  10   10  # or http://www.opensolaris.org/os/licensing.
  11   11  # See the License for the specific language governing permissions
  12   12  # and limitations under the License.
  13   13  #
  14   14  # When distributing Covered Code, include this CDDL HEADER in each
  15   15  # file and include the License file at usr/src/OPENSOLARIS.LICENSE.
  16   16  # If applicable, add the following below this CDDL HEADER, with the
  17   17  # fields enclosed by brackets "[]" replaced with your own identifying
  18   18  # information: Portions Copyright [yyyy] [name of copyright owner]

  19   19  #
  20   20  # CDDL HEADER END
  21   21  #
  22   22  
  23   23  #
  24   24  # Copyright 2009 Sun Microsystems, Inc.  All rights reserved.
  25   25  # Use is subject to license terms.
  26   26  #
  27   27  
  28   28  #
  29      -# Copyright (c) 2013 by Delphix. All rights reserved.
       29 +# Copyright (c) 2013, 2016 by Delphix. All rights reserved.
  30   30  #
  31   31  
  32   32  . $STF_SUITE/include/libtest.shlib
  33   33  . $STF_SUITE/tests/functional/reservation/reservation.shlib
  34   34  
  35   35  #
  36   36  # DESCRIPTION:
  37   37  #
  38   38  # In pool with a full filesystem and a regular volume with an implicit
  39   39  # reservation, setting the reservation on the volume to 'none' should allow

  40   40  # more data to be written to the filesystem.
  41   41  #
  42   42  #
  43   43  # STRATEGY:
  44   44  # 1) Create a regular non-sparse volume (which implicitly sets the reservation
  45   45  #    property to a value equal to the volume size)
  46   46  # 2) Create a filesystem at the same level
  47   47  # 3) Fill up the filesystem
  48   48  # 4) Set the reservation on the volume to 'none'
  49   49  # 5) Verify can write more data to the filesystem

  50   50  #
  51   51  
  52   52  verify_runnable "global"
  53   53  
  54   54  log_assert "Setting volume reservation to 'none' allows more data to be " \
  55   55      "written to top level filesystem"
  56   56  
  57   57  function cleanup
  58   58  {
  59   59          datasetexists $TESTPOOL/$TESTVOL && \
  60      -        log_must $ZFS destroy $TESTPOOL/$TESTVOL
       60 +        log_must zfs destroy $TESTPOOL/$TESTVOL
  61   61  
  62      -        [[ -e $TESTDIR/$TESTFILE1 ]] && log_must $RM -rf $TESTDIR/$TESTFILE1
  63      -        [[ -e $TESTDIR/$TESTFILE2 ]] && log_must $RM -rf $TESTDIR/$TESTFILE2
       62 +        [[ -e $TESTDIR/$TESTFILE1 ]] && log_must rm -rf $TESTDIR/$TESTFILE1
       63 +        [[ -e $TESTDIR/$TESTFILE2 ]] && log_must rm -rf $TESTDIR/$TESTFILE2
  64   64  }
  65   65  log_onexit cleanup
  66   66  
  67   67  space_avail=$(largest_volsize_from_pool $TESTPOOL)
  68   68  
  69   69  #
  70   70  # To make sure this test doesn't take too long to execute on
  71   71  # large pools, we calculate a volume size which when applied
  72   72  # to the volume will ensure we have RESV_FREE_SPACE
  73   73  # left free in the pool which we can quickly fill.
  74   74  #
  75   75  ((resv_size_set = space_avail - RESV_FREE_SPACE))
  76   76  resv_size_set=$(floor_volsize $resv_size_set)
  77   77  
  78      -log_must $ZFS create -V $resv_size_set $TESTPOOL/$TESTVOL
       78 +log_must zfs create -V $resv_size_set $TESTPOOL/$TESTVOL
  79   79  
  80   80  space_avail_still=`get_prop available $TESTPOOL`
  81   81  
  82   82  fill_size=$((space_avail_still + $RESV_TOLERANCE))
  83   83  write_count=$((fill_size / BLOCK_SIZE))
  84   84  
  85   85  # Now fill up the filesystem (which doesn't have a reservation set
  86   86  # and thus will use up whatever free space is left in the pool).
  87      -$FILE_WRITE -o create -f $TESTDIR/$TESTFILE1 -b $BLOCK_SIZE -c $write_count -d 0
       87 +file_write -o create -f $TESTDIR/$TESTFILE1 -b $BLOCK_SIZE -c $write_count -d 0
  88   88  ret=$?
  89   89  if (($ret != $ENOSPC)); then
  90   90          log_fail "Did not get ENOSPC as expected (got $ret)."
  91   91  fi
  92   92  
  93      -log_must $ZFS set refreservation=none $TESTPOOL/$TESTVOL
       93 +log_must zfs set refreservation=none $TESTPOOL/$TESTVOL
  94   94  
  95      -log_must $FILE_WRITE -o create -f $TESTDIR/$TESTFILE2 -b $PAGESIZE \
       95 +log_must file_write -o create -f $TESTDIR/$TESTFILE2 -b pagesize \
  96   96      -c 1000 -d 0
  97   97  
  98   98  log_pass "Setting top level volume reservation to 'none' allows more " \
  99   99      "data to be written to the top level filesystem"
    

XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX