Thanks for the review Karen

I ran the full suite of slim unit tests and no regressions were found.

cheers

Matt

On 04/05/2012 17:51, Karen Tung wrote:
Hi Matt,

The changes look good to me.

As for testing, did you run the slim unit tests also?

Thanks,

--Karen

On 05/ 4/12 08:56 AM, Matt Keenan wrote:
Hi,

Can I get a pair of eyes on this relatively simple CR :
7166325 : http://monaco.us.oracle.com/detail.jsf?cr=7166325

Webrev
https://cr.opensolaris.org/action/browse/caiman/mattman/7166325/


When creating zones, instantiation is achieved by calling BE.init()
directly, and passing in the dataset names and options associated with
each dataset.

Mountpoints are completely ignored, and be.init(), and the mountpoint
defaults to the name of the dataset being created.

Solution is to ensure mountpoint if set is passed into be.init() as a
dataset option.

Tested zone install in and boots successfuly

cheers

Matt
_______________________________________________
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

Reply via email to