Hi Marty,

Thanks for your comments. I have added an additional check to test the 
return status of cl_exec_client command at line 324.

Please find the updated webrev @ 
http://cr.opensolaris.org/~hnamachi/ha-ldom/.

-Regards
Hemachandran

Martin Rattner wrote:
> Hemachandran,
>
> The changes look good.
>
> The lines you inserted at line 314 check for the node being up to fix 
> CR 6861658 . This should work fine almost always, but there is still a 
> slight chance that the node goes down immediately after you check it 
> at line 314, but before executing ${CL_EXEC_CLIENT} at line 320. To 
> make this bulletproof, perhaps you could check for an error return 
> from ${CL_EXEC_CLIENT} that would catch the case of the node being 
> down? If it is feasible to do that check, and if it finds that the 
> node has died, you could 'continue' to the next node in the list at 
> that point.
>
> --Marty
>
> On 07/21/09 05:09, Hemachandran Namachivayam wrote:
>> Hi Folks,
>>
>> Can you please review the changes made for the below CR's. The webrev 
>> is @ http://cr.opensolaris.org/~hnamachi/ha-ldom/.
>>
>> CR 6861658 /opt/SUNWscgds/bin/../../SUNWscxvm/bin/validate_xvm[46]: 
>> is: bad number if node is down
>>
>> Description:
>> If one of the cluster node is down the validate method throws the ksh 
>> error message ": is: bad number ".
>>
>> Solution:
>> The domain related checks have to be skipped if the node is down.
>>
>> CR 6861119 Validation should check for failure-policy=reset on master 
>> instead of guest domain
>>
>> Description:
>> The validate method should check if the failure-policy is to "reset" 
>> on the primary instead of the guest domain. The 'failure-policy' 
>> property should be set on the master as this determines what the 
>> slaves will do when the master fails.
>>
>> Solution:
>> Validate method will query the failure-policy setting on control 
>> domain (i.e primary).
>>
>> -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