Hi,

On 03/23/2011 11:59 AM, mattjclark0407@hotmail.com wrote:

From: Matt Clark<mattjclark0407@hotmail.com>

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,