Hi,

On 03/23/2011 11:59 AM, [email protected] wrote:
From: Matt Clark<[email protected]>

Added all_opt parameters to aid in the fencing of virtual machines. 
Specifically this is for the fencing of machines through the XenAPI (Citrix 
XenServer or Xen Cloud Platform). New parameters are session_url, vm_name and 
uuid.

session_url and uuid works for me.

vm_name should be replaced by existing port (-n) that is already used for this purpose. I can fix this and push your patch to upstream if you want.

Updated logic to now check if either an IP address (-a) or a session URL (-s) 
is set.

good idea
Added a section to ensure that either a vm_name or uuid is specified. Because you can 
specify either, the "required" field in all_opt array can't be used and a 
separate explicit check is required.

ok

m,

Reply via email to