On 1/26/10 9:22 AM, John Fischer wrote:
> Evan,
>
> Sorry I meant to reply yesterday.

Not a problem!

>
> This looks good.

Thanks for taking a look at it!

-evan

>
> Thanks,
>
> John
>
> On 01/23/10 09:29 PM, Evan Layton wrote:
>> Requesting a Code review for bug 5711:
>> 5711 - libbe autonaming for zones datasets should use the same
>> method as the BE autonaming
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=5711
>>
>> Webrev:
>> http://cr.opensolaris.org/~evanl/5711/
>>
>> This fix changes the way the auto-naming for zone BE's works so that
>> it matches that of the auto-naming for BE's. This eliminates possible
>> name collisions due to multiple processes as well as keeping the
>> auto-naming from reusing a name that may sill have a snapshot around
>> from before bug 10990 was fixed.
>>
>> The main name collision portion of the bug was fixed with bug 10990.
>>
>> Testing:
>> Ran stc libbe test suite - no unexpected test failures (any failures
>> were due to known ZFS unmount failure,
>> bugster bug 6872581)
>> Built build 131 iso and tested an install - all worked as expected.
>>
>>
>> Thanks,
>>
>> -evan
>> _______________________________________________
>> caiman-discuss mailing list
>> caiman-discuss at opensolaris.org
>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss


Reply via email to