Ethan Quach wrote: > Joe, Jean, Jerry, > > > Thanks for providing your comments. We've built an incremental webrev > with just the changes from your comments. This webrev also includes a > couple of dev bugfixes that went in during the code review process - > 3776, 3772 > Any further comments are welcomed. > > http://cr.opensolaris.org/~equach/webrev.snap_zones.2 > > > As a reference, we've also refreshed the overall webrev here: > > http://cr.opensolaris.org/~equach/webrev.3686 > > > Thanks again for the review, > > -ethan
This all looks good. I only found one small nit. Joe usr/src/lib/libbe/be_create.c +-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+-=+ Issue: ------ Typo-s in comment: Change from: 1004 * root dataset of the new BE. The uuid is use to set the parentbe 1005 * property for the new zones datasets. To: 1004 * root dataset of the new BE. The uuid is used to set the parent be 1005 * property for the new zones datasets. _______________________________________________ zones-discuss mailing list zones-discuss@opensolaris.org