incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tomoe Sugihara" <to...@midokura.com>
Subject Re: Review Request: Introduce pluggable vif driver support on KVM
Date Thu, 02 Aug 2012 23:18:03 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6285/
-----------------------------------------------------------

(Updated Aug. 2, 2012, 11:18 p.m.)


Review request for cloudstack, edison su and Chiradeep Vittal.


Changes
-------

fix typo in commit log


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 <tomoe@midokura.com>

Add unplug() hook

Signed-off-by: Tomoe Sugihara <tomoe@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 <tomoe@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
-------

Manually tested launching VM on all-in-one KVM box with Advanced networking.


Thanks,

Tomoe Sugihara


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message