Skip to content

Commit

Permalink
7027 zfs_written_property_001_pos makes unreasonable assumptions abou…
Browse files Browse the repository at this point in the history
…t metadata space usage

Reviewed by: Prakash Surya <prakash.surya@delphix.com>
Reviewed by: John Kennedy <john.kennedy@delphix.com>
Reviewed by: Matthew Ahrens <mahrens@delphix.com>
Reviewed by: Igor Kozhukhov <ikozhukhov@gmail.com>
Approved by: Gordon Ross <gordon.ross@nexenta.com>
  • Loading branch information
aayare-dlpx authored and ahrens committed Jun 3, 2016
1 parent a2f72b6 commit 48cb8b9
Showing 1 changed file with 5 additions and 5 deletions.
Expand Up @@ -11,7 +11,7 @@
#

#
# Copyright (c) 2012 by Delphix. All rights reserved.
# Copyright (c) 2012, 2015 by Delphix. All rights reserved.
#

#
Expand Down Expand Up @@ -63,11 +63,8 @@ typeset -l total=0
typeset -l snap1_size=0
typeset -l snap2_size=0
typeset -l snap3_size=0
typeset -l metadata=0
typeset -l mb_block=0
((mb_block = 1024 * 1024))
# approximate metadata on dataset when empty is 32KB
((metadata = 32 * 1024))

log_note "verify written property statistics for dataset"
log_must $ZFS create -p $TESTPOOL/$TESTFS1/$TESTFS2/$TESTFS3
Expand All @@ -89,7 +86,10 @@ blocks=0
for i in 1 2 3; do
written=$(get_prop written $TESTPOOL/$TESTFS1@snap$i)
if [[ $blocks -eq 0 ]]; then
expected_written=$metadata
# Written value for the frist non-clone snapshot is
# expected to be equal to the referenced value.
expected_written=$( \
get_prop referenced $TESTPOOL/$TESTFS1@snap$i)
else
((expected_written = blocks * mb_block))
fi
Expand Down

0 comments on commit 48cb8b9

Please sign in to comment.