The updated webrev looks good to me. -evan
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
