My nit has been hit and now it looks fit :)
My issue has been addressed.
Thanks,
Poet Jack
On 12/15/10 01:41 PM, Drew Fisher wrote:
Round two!
Full webrev against slim_source:
http://cr.opensolaris.org/~drewfish/dc_7006443/
Incremental webrev:
http://cr.opensolaris.org/~drewfish/dc_7006443-2/
I've addressed all the comments and have retested all of my scenarios
as listed below.
If I could get a couple of eyes on this, I would greatly appreciate it!
Thanks!
-Drew
On 12/14/10 11:01 AM, Drew Fisher wrote:
Could I please get a code review for:
7006443 transfer-ips-install checkpoint created image in wrong location
Bug:
http://monaco.sfbay.sun.com/detail.jsf?cr=7006443
Webrev:
http://cr.opensolaris.org/~drewfish/dc_7006443/
The way DC handled mountpoints of ZFS datasets and zpools was very
wrong. It made an assumption that everything would mount at "/".
Tests run:
- Create a new zpool and a new zfs dataset both with no mountpoint
information (zpool mounts at /, dataset mounts at
/zpool_name/dataset_name)
- Create a new zpool with an explicit mountpoint defined and a new
zfs dataset with no mountpoint information (zpool mounts at
/mountpoint, dataset mounts at /mountpoint/dataset_name *with the
name of the zpool stripped from the dataset_name* - this is how ZFS
currently works)
- Use an existing zpool, but explicitly set the mountpoint of the zfs
dataset to something outside of the mountpoint of the pool.
Thanks!
-Drew
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
