Skip to content

Commit

Permalink
tests: rsend/send-c_props: make it chooch
Browse files Browse the repository at this point in the history
Original error:
23:47:40.59 SUCCESS: eval zfs receive -dFv testpool2 < /mnt/testroot/backdir-rsend/pool-final-p
23:47:40.61 1,23d0
23:47:40.61 < type      filesystem      -
23:47:40.61 < origin    POOL@psnap      -
23:47:40.61 < volblocksize      -       -
23:47:40.61 < acltype   nfsv4   inherited from POOL
23:47:40.61 < dnodesize legacy  inherited from POOL
23:47:40.61 < atime     off     local
23:47:40.61 < canmount  off     local
23:47:40.61 < checksum  off     local
23:47:40.61 < compression       off     local
23:47:40.61 < copies    3       local
23:47:40.61 < devices   off     local
23:47:40.61 < exec      off     local
23:47:40.61 < quota     none    default
23:47:40.61 < readonly  on      local
23:47:40.61 < recordsize        128K    local
23:47:40.61 < reservation       none    default
23:47:40.61 < setuid    off     local
23:47:40.61 < snapdir   hidden  local
23:47:40.61 < version   5       -
23:47:40.61 < volsize   -       -
23:47:40.61 < xattr     off     local
23:47:40.61 < mountpoint        /PREFIX inherited from POOL
23:47:40.61 < jailed    on      local
23:47:40.62 cannot open 'testpool2/pclone': dataset does not exist
23:47:40.62 ERROR: cmp_ds_prop testpool/pclone testpool2/pclone exited 1

So: (a) actually send all the datasets in -p mode and
    (b) drop origin for clones sent with -p:

00:38:05.46 SUCCESS: eval zfs receive -dFv testpool2 < /mnt/testroot/backdir-rsend/pool-final-p
00:38:05.48 2c2
00:38:05.48 < origin    POOL@psnap
00:38:05.48 ---
00:38:05.48 > origin    POOL
00:38:05.49 ERROR: cmp_ds_prop testpool/pclone testpool2/pclone nosource exited 1

Closes openzfs#13250
Signed-off-by: Ahelenia Ziemiańska <[email protected]>
  • Loading branch information
nabijaczleweli committed Mar 24, 2022
1 parent d1e2c83 commit 8f738a4
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 7 deletions.
10 changes: 7 additions & 3 deletions tests/zfs-tests/tests/functional/rsend/rsend.kshlib
Original file line number Diff line number Diff line change
Expand Up @@ -207,14 +207,18 @@ function cmp_ds_cont
#
# $1 dataset 1
# $2 dataset 2
# $3 -n == don't track property source
# $4 -n == don't track the origin property
#
function cmp_ds_prop
{
typeset dtst1=$1
typeset dtst2=$2
typeset source=",source"
[ -n "$3" ] && source=
typeset props="type,origin,volblocksize,acltype,dnodesize"
typeset nosource=$3
typeset noorigin=$4
typeset source=",source"; [ -n "$nosource" ] && source=
typeset origin=",origin"; [ -n "$noorigin" ] && origin=
typeset props="type$origin,volblocksize,acltype,dnodesize"
props+=",atime,canmount,checksum,compression,copies,devices"
props+=",exec,quota,readonly,recordsize,reservation,setuid"
props+=",snapdir,version,volsize,xattr,mountpoint"
Expand Down
19 changes: 15 additions & 4 deletions tests/zfs-tests/tests/functional/rsend/send-c_props.ksh
Original file line number Diff line number Diff line change
Expand Up @@ -52,16 +52,27 @@ for opt in "-p" "-R"; do
randomize_ds_props $POOL$ds
done

log_must eval "zfs send -c $opt $POOL@final > $BACKDIR/pool-final$opt"
log_must eval "zfs receive -d -F $POOL2 < $BACKDIR/pool-final$opt"
if [ $opt = "-p" ]; then
for ds in ${datasets[@]}; do
log_must eval "zfs send -c $opt $POOL$ds@final > $BACKDIR/pool-final$opt"
log_must eval "zfs receive -dF $POOL2 < $BACKDIR/pool-final$opt"
done
else
log_must eval "zfs send -c $opt $POOL@final > $BACKDIR/pool-final$opt"
log_must eval "zfs receive -dF $POOL2 < $BACKDIR/pool-final$opt"
fi

for ds in ${datasets[@]}; do
log_must cmp_ds_prop $POOL$ds $POOL2$ds
typeset origin=
if [ $opt = "-p" ] && [ ${ds/clone//} != $ds ]; then
origin=noorigin
fi
log_must cmp_ds_prop $POOL$ds $POOL2$ds nosource $origin
log_must cmp_ds_prop $POOL$ds@final $POOL2$ds@final
done

# Don't cleanup the second time, since we do that on exit anyway.
[[ $opt = "-p" ]] && cleanup
[ $opt = "-p" ] && cleanup
done

log_pass "Compressed send doesn't interfere with preservation of properties"

0 comments on commit 8f738a4

Please sign in to comment.