----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6285/ -----------------------------------------------------------
(Updated Aug. 2, 2012, 11:11 p.m.) Review request for cloudstack and Chiradeep Vittal. Changes ------- Updated the patches for pluggable vif driver support on KVM. There are still a lot of places that can be cleaned up, but I think this is a good start. Summary (updated) ----------------- Introduce pluggable vif driver support on KVM Description (updated) ------- Remove methods that are no longer needed in LibvirtComputingResource.java Signed-off-by: Tomoe Sugihara <to...@midokura.com> Add unpug() hook Signed-off-by: Tomoe Sugihara <to...@midokura.com> Add vif driver support - define abstract class named LibvirtVifDriver - add LibvirtBridgeVifDriver to support current network implementation - unplug() will be added Signed-off-by: Tomoe Sugihara <to...@midokura.com> Diffs (updated) ----- plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtBridgeVifDriver.java PRE-CREATION plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java 73101a9896c4863bc75502ff8b0b6daceff52dc4 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVifDriver.java PRE-CREATION Diff: https://reviews.apache.org/r/6285/diff/ Testing (updated) ------- Manually tested launching VM on all-in-one KVM box with Advanced networking. Thanks, Tomoe Sugihara
