Skip to content

Commit dabba94

Browse files
committed
ZTS: Remove ashift setting from dedup_quota test
The test writes 1M of 1K blocks, which may produce up to 1GB of dirty data. On top of that ashift=12 likely produces additional 4GB of ZIO buffers during sync process. On top of that we likely need some page cache since the pool reside on files. And finally we need to cache the DDT. Not surprising that the test regularly ends up in OOMs, possibly depending on TXG size variations. While here, remove compression. It has nothing to do here, but waste CI CPU time. Signed-off-by: Alexander Motin <[email protected]> Sponsored by: iXsystems, Inc.
1 parent 155847c commit dabba94

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

tests/zfs-tests/tests/functional/dedup/dedup_quota.ksh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ function do_setup
7979
{
8080
log_must truncate -s 5G $VDEV_GENERAL
8181
# Use 'xattr=sa' to prevent selinux xattrs influencing our accounting
82-
log_must zpool create -o ashift=12 -f -O xattr=sa -m $MOUNTDIR $POOL $VDEV_GENERAL
82+
log_must zpool create -f -O xattr=sa -m $MOUNTDIR $POOL $VDEV_GENERAL
8383
log_must zfs set compression=off dedup=on $POOL
8484
}
8585

@@ -197,7 +197,6 @@ function ddt_dedup_vdev_limit
197197
log_must zpool set dedup_table_quota=auto $POOL
198198

199199
log_must zfs set recordsize=1K $POOL
200-
log_must zfs set compression=zstd $POOL
201200

202201
# Generate a working set to fill up the dedup/special allocation class
203202
log_must fio --directory=$MOUNTDIR --name=dedup-filler-1 \

0 commit comments

Comments
 (0)