Looks ok. Filing another CR sounds good.
thx,
ginnie
On 10/27/08 11:52, Ethan Quach wrote:
> Ethan Quach wrote:
>
>> I've updated the webrev with a TODO. To track this, I can file
>> another defect to track removal of this code. That defect will
>> hinge on 617643.
>>
>>
>
> That's 6176743.
>
>
>> -ethan
>>
>>
>> Virginia Wray wrote:
>>
>>
>>> Hi Ethan -
>>>
>>> The changes look ok to me also. Evan echoed my thought, which
>>> was...how will the fix for 617643 be tracked and then implemented.
>>> g
>>>
>>> On 10/27/08 11:07, Evan Layton wrote:
>>>
>>>
>>>> Hi Ethan,
>>>>
>>>> The changes look fine to me.
>>>>
>>>> One nit/idea on the comments though. Since we're doing this fix as a work
>>>> around
>>>> for 6176743 do we want to add a "TODO:" here to state that this needs to
>>>> be
>>>> revisited when 6176743 gets fixed?
>>>>
>>>> Thanks!
>>>> -evan
>>>>
>>>>
>>>>
>>>> Ethan Quach wrote:
>>>>
>>>>
>>>>
>>>>> Can I get a review one liner (plus comments)
>>>>>
>>>>> Webrev:
>>>>> ------------
>>>>> http://cr.opensolaris.org/~equach/webrev.4041
>>>>>
>>>>>
>>>>> Defect:
>>>>> ----------
>>>>> 4041 Creating a BE with zones fails when an alternate BE is currently
>>>>> mounted
>>>>> http://defect.opensolaris.org/bz/show_bug.cgi?id=4041
>>>>>
>>>>>
>>>>> With this fix, new boot environments can be created while
>>>>> some other boot environment is mounted with zones.
>>>>>
>>>>>
>>>>> thanks,
>>>>> -ethan
>>>>>
>>>>> _______________________________________________
>>>>> caiman-discuss mailing list
>>>>> caiman-discuss at opensolaris.org
>>>>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
>>>>>
>>>>>
>>>>>
>>>> _______________________________________________
>>>> caiman-discuss mailing list
>>>> caiman-discuss at opensolaris.org
>>>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
>>>>
>>>>
>>>>
>>> --
>>>
>>> Ginnie
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>> _______________________________________________
>> caiman-discuss mailing list
>> caiman-discuss at opensolaris.org
>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
>>
>>
> _______________________________________________
> caiman-discuss mailing list
> caiman-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
>
--
Ginnie
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://mail.opensolaris.org/pipermail/caiman-discuss/attachments/20081027/ed264f87/attachment.html>
