Hi Jonathan,

Thanks for your review comments. Please see my replies inline.

On 06/27/09 03:31, Jonathan Mellors wrote:
> Hi Hemachandran,
>
> The Makefile and packaging changes look fine. The source files should 
> not be executable so you need to run chmod -x on:
> usr/src/cmd/ha-services/gds-agents/xvm/SUNW.ldom
> usr/src/cmd/ha-services/gds-agents/xvm/validate_xvm.ksh
> usr/src/cmd/ha-services/gds-agents/xvm/lib/libgetpassphrase.c
>
> and their SCCS/s.files.
>
Fixed it.
> You appear to have mixed the CDDL header from the gds template with 
> the actual one we use for our source tree. They should all refer to 
> usr/src/CDDL.txt, and the SUNW.* files should contain it as well.
>
Fixed it. Please find the updated webrev at 
http://cr.opensolaris.org/~hnamachi/ha-ldom/
> usr/src/pkgdefs/jas.dvdimage.ksh:
>   xvm for S10/i386?
>
The HA-xVM for "domU"s were targeted only for SXCE editions. Maybe Neil 
can comment on this.

-Thanks and Regards
Hemachandran
> Thanks
> Jonathan
>
> Hemachandran Namachivayam wrote:
>> Hi Folks,
>>
>> I'd welcome the code review for the "Solaris cluster failover agent 
>> for LDoms Guest domains"
>>
>> The failover agent for LDoms Guest domains is an extension of the 
>> HA-xVM agent and introduces a new resource type SUNW.ldom. The agent 
>> is redesigned to use the Generic Data Service (SUNW.gds) binaries and 
>> hence the resource type (SUNW.xvm) have been reworked upon. The 
>> HA-xVM agent makes use of CCR to store the xVM/LDom guest domain 
>> configuration; thereby a need to configure a separate admin directory 
>> is eliminated.
>>
>> To refer to the requirements specification and design document, 
>> please visit http://opensolaris.org/os/project/ha-xvm/
>>
>> The webrev, for review, is located at 
>> http://cr.opensolaris.org/~hnamachi/ha-ldom/
>>
>> -Thanks and Regards
>> Hemachandran
>>
>> _______________________________________________
>> ha-clusters-discuss mailing list
>> ha-clusters-discuss at opensolaris.org
>> http://mail.opensolaris.org/mailman/listinfo/ha-clusters-discuss


Reply via email to