diff options
author | Paul Zuchowski <[email protected]> | 2019-01-13 13:06:13 -0500 |
---|---|---|
committer | Brian Behlendorf <[email protected]> | 2019-01-13 10:06:13 -0800 |
commit | 83c796c5e9d9ecb28e9553338f079a5d6b015b10 (patch) | |
tree | 7b6661d3070441ad1472111fe2c92879a1a3d057 /tests | |
parent | a13392a6a1a0e3858a11f79cf3fc2f2026b4dcc0 (diff) |
zfs filesystem skipped by df -h
On full pool when pool root filesystem references very few bytes,
the f_blocks returned to statvfs is 0 but should be at least 1.
Reviewed by: Tom Caputi <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Paul Zuchowski <[email protected]>
Closes #8253
Closes #8254
Diffstat (limited to 'tests')
-rw-r--r-- | tests/runfiles/linux.run | 3 | ||||
-rw-r--r-- | tests/zfs-tests/tests/functional/no_space/Makefile.am | 3 | ||||
-rwxr-xr-x | tests/zfs-tests/tests/functional/no_space/enospc_df.ksh | 73 |
3 files changed, 77 insertions, 2 deletions
diff --git a/tests/runfiles/linux.run b/tests/runfiles/linux.run index 0b41d087b..d3ecf6274 100644 --- a/tests/runfiles/linux.run +++ b/tests/runfiles/linux.run @@ -656,7 +656,8 @@ tests = ['nestedfs_001_pos'] tags = ['functional', 'nestedfs'] [tests/functional/no_space] -tests = ['enospc_001_pos', 'enospc_002_pos', 'enospc_003_pos'] +tests = ['enospc_001_pos', 'enospc_002_pos', 'enospc_003_pos', + 'enospc_df'] tags = ['functional', 'no_space'] [tests/functional/nopwrite] diff --git a/tests/zfs-tests/tests/functional/no_space/Makefile.am b/tests/zfs-tests/tests/functional/no_space/Makefile.am index a5b042b23..c2e42bc2a 100644 --- a/tests/zfs-tests/tests/functional/no_space/Makefile.am +++ b/tests/zfs-tests/tests/functional/no_space/Makefile.am @@ -4,7 +4,8 @@ dist_pkgdata_SCRIPTS = \ cleanup.ksh \ enospc_001_pos.ksh \ enospc_002_pos.ksh \ - enospc_003_pos.ksh + enospc_003_pos.ksh \ + enospc_df.ksh dist_pkgdata_DATA = \ enospc.cfg diff --git a/tests/zfs-tests/tests/functional/no_space/enospc_df.ksh b/tests/zfs-tests/tests/functional/no_space/enospc_df.ksh new file mode 100755 index 000000000..b3df69141 --- /dev/null +++ b/tests/zfs-tests/tests/functional/no_space/enospc_df.ksh @@ -0,0 +1,73 @@ +#!/bin/ksh -p +# +# CDDL HEADER START +# +# This file and its contents are supplied under the terms of the +# Common Development and Distribution License ("CDDL"), version 1.0. +# You may only use this file in accordance with the terms of version +# 1.0 of the CDDL. +# +# A full copy of the text of the CDDL should have accompanied this +# source. A copy of the CDDL is also available via the Internet at +# http://www.illumos.org/license/CDDL. +# +# CDDL HEADER END +# + +# +# Copyright (c) 2014, 2016 by Delphix. All rights reserved. +# Copyright (c) 2019 by Datto Inc. All rights reserved. +# + +. $STF_SUITE/include/libtest.shlib +. $STF_SUITE/tests/functional/no_space/enospc.cfg + +# +# DESCRIPTION: +# After filling a filesystem, the df command produces the +# expected result against the pool root filesystem. +# +# STRATEGY: +# 1. Write a file until the child file system is full. +# 2. Ensure that ENOSPC is returned. +# 3. Unmount the child file system. +# 4. Issue df -h command. +# 5. Ensure pool root filesystem is included (issue #8253). +# 6. Issue df -h <filesystem>. +# 7. Ensure size and used are non-zero. +# + +verify_runnable "both" + +log_onexit default_cleanup_noexit + +log_assert "Correct df output is returned when file system is full." + +default_setup_noexit $DISK_SMALL +log_must zfs set compression=off $TESTPOOL/$TESTFS + +log_note "Writing file: $TESTFILE0 until ENOSPC." +file_write -o create -f $TESTDIR/$TESTFILE0 -b $BLOCKSZ \ + -c $NUM_WRITES -d $DATA +ret=$? + +(( $ret != $ENOSPC )) && \ + log_fail "$TESTFILE0 returned: $ret rather than ENOSPC." + +log_must zfs umount $TESTPOOL/$TESTFS + +# Ensure the pool root filesystem shows in df output. +# If the pool was full (available == 0) and the pool +# root filesytem had very little in it (used < 1 block), +# the size reported to df was zero (issue #8253) and +# df skipped the filesystem in its output. +log_must eval "df -h | grep $TESTPOOL" + +# Confirm df size and used are non-zero. +size=$(df -h /$TESTPOOL | grep $TESTPOOL | awk '{print $2}') +used=$(df -h /$TESTPOOL | grep $TESTPOOL | awk '{print $3}') +if [[ "$size" = "0" ]] || [[ "$used" = "0" ]] +then + log_fail "df failed with size $size and used $used." +fi +log_pass "df after ENOSPC works as expected." |