Giuseppe Vallarelli has posted comments on this change.

Change subject: Update device hooks: Added before and after update device hooks.
......................................................................


Patch Set 7: (4 inline comments)

....................................................
File vdsm/vdsmd.8.in
Line 46:     before_vm_migrate_source, after_vm_migrate_source,
Line 47:     before_vm_migrate_destination, after_vm_migrate_destination,
Line 48:     before_vm_destroy, after_vm_destroy,
Line 49:     before_vm_set_ticket, after_vm_set_ticket,
Line 50:     before_update_device, after_update_device, 
after_update_device_fail,
Ok.
Line 51:     before_device_create, after_device_create,
Line 52:     before_device_destroy, after_device_destroy,
Line 53:     before_nic_hotplug, after_nic_hotplug, after_nic_hotplug_fail,
Line 54:     before_nic_hotunplug, after_nic_hotunplug, 
after_nic_unhotplug_fail,


Line 71: available as the environment variable
Line 72: .B vmId.
Line 73: Hooks that handle NIC hotplug and hotunplug have the _hook_domxml 
variable but
Line 74: it contains the representation of the NIC rather than the VM.
Line 75: Hotplug/hotunplug disk hooks also have the _hook_dom_xml variable,
Thanks!
Line 76: which contains the drive definition (not the VM).
Line 77: 
Line 78: On top of these, @ENGINENAME@ allows to set a collection of "custom 
parameters" for
Line 79: each virtual machine.  Each of these parameters is provided to hooks 
as an


Line 86: The environment of before_vm_set_ticket and after_vm_set_ticket hooks 
is augmented
Line 87: with a set of params passed by the caller of setVmTicket.
Line 88: 
Line 89: The environment of hooks specific to devices:
Line 90:     before_nic_hotplug, after_nic_hotplug, after_nic_hotplug_fail,
OK.
Line 91:     before_nic_hotunplug, after_nic_hotunplug, 
after_nic_unhotplug_fail,
Line 92:     before_disk_hotplug, after_disk_hotplug,
Line 93:     before_disk_hotunplug, after_disk_hotunplug,
Line 94:     before_device_create, after_device_create,


Line 92:     before_disk_hotplug, after_disk_hotplug,
Line 93:     before_disk_hotunplug, after_disk_hotunplug,
Line 94:     before_device_create, after_device_create,
Line 95:     before_device_destroy, after_device_destroy.
Line 96: 
Sure, np
Line 97: Are all augmented by custom properties specific to those devices, 
Line 98: sent by the caller of the hook. For example if before_nic_hotplug is 
called
Line 99: with custom: {qos: '0.5', color: 'red'} then qos and color will be 
directly
Line 100: available as environment variables when before_nic_hotplug is called.


--
To view, visit http://gerrit.ovirt.org/13972
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8fe7f5e6d8291cc227a323b324c977bcd5bc8295
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Giuseppe Vallarelli <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Giuseppe Vallarelli <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to