I've updated the master webrev:
http://kebe.com/~danmcd/webrevs/4986
As follows:
1.) I've combined the tests to accompany the bugfix back upstream.
2.) I've moved the test into cli_root/zfs_receive as zfs_receive_011_pos.ksh
3.) I've added omnios.run and openindiana.run changes to include
zfs_receive_010. If this is fixed by others, it'll merge out cleanly.
I've also RUN these tests both with and without 4986's fix, and got these diffs
in failures:
danmcd-os(~)[1]% diff /tmp/{stock,4986}
29d28
< /opt/zfs-tests/tests/functional/cli_root/zfs_receive/zfs_receive_011_pos
32a32
> /opt/zfs-tests/tests/functional/cli_root/zfs_set/ro_props_001_pos
76,77d75
< /opt/zfs-tests/tests/functional/replacement/replacement_001_pos
< /opt/zfs-tests/tests/functional/replacement/replacement_002_pos
99d96
< /opt/zfs-tests/tests/functional/xattr/xattr_013_pos
104c101
< 103
---
> 100
The last two (counts) you can ignore, but notice that the ro_props_001_pos
fails on the new bits, but apparently that's been addressed elsewhere:
http://www.listbox.com/member/archive/182191/2015/11/sort/time_rev/page/1/entry/7:29/20151114081413:9572EF60-8AD1-11E5-BF6F-9F27E897D8C2/
Please let me know what you think. I'm still concerned I can make the bugfix
itself cleaner (esp. in libzfs), and would appreciate feedback.
Thanks!
Dan
_______________________________________________
developer mailing list
[email protected]
http://lists.open-zfs.org/mailman/listinfo/developer
