Hey Dave,

My "ship it" is waiting for a last compile and test run on my dev platform. 
I'll try to do that over the weekend.

 Cheers,

Hugo

Sent from my iPhone

On 29 mrt. 2013, at 02:17, "Dave Cahill" 
<dcah...@midokura.com<mailto:dcah...@midokura.com>> wrote:

Hi all,

I think all review comments have been addressed on this, but review progress 
seems to have stalled - anything I should be doing to keep things moving?

Thanks,
Dave.


On Wed, Mar 27, 2013 at 3:33 PM, Dave Cahill 
<dcah...@midokura.com<mailto:dcah...@midokura.com>> wrote:
This is an automatically generated e-mail. To reply, visit: 
https://reviews.apache.org/r/9898/

Review request for cloudstack, Hugo Trippaers and Chiradeep Vittal.
By Dave Cahill.

Updated March 27, 2013, 6:33 a.m.

Changes

Fixed Hiroaki's exception-related review comment.


Description

Feature spec:
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Midokura+Networking+Plugin

Jira ticket:
https://issues.apache.org/jira/browse/CLOUDSTACK-996

Notes:

* Documentation will follow as a separate commit

* One main difference from existing networking plugins is the lack of a 
Resource class; we didn't feel it was necessary in this case. As mentioned in 
Extending CloudStack Networking [1]:
"Just like managers, resources are not strictly necessary. In theory a Network 
Element could implement a client for the API of the new controller and 
therefore be completely self-contained."

* We allow overriding Public traffic via the MidoNetPublicNetworkGuru. We 
checked this approach with the list [2] and received no comments, so we're 
going with it for now.

[1] https://cwiki.apache.org/CLOUDSTACK/extending-cloudstack-networking.html
[2] http://markmail.org/message/k5qse63eyylszm3i


Testing

Built and deployed, spun up Advanced Isolated network with two VMs, verified 
internal and external connectivity via MidoNet.


Bugs: CLOUDSTACK-996
Diffs (updated)

  *   api/src/com/cloud/network/Network.java (c2ab655)
  *   api/src/com/cloud/network/Networks.java (e3d2158)
  *   api/src/com/cloud/network/PhysicalNetwork.java (343a2b1)
  *   api/src/org/apache/cloudstack/network/ExternalNetworkDeviceManager.java 
(bc22804)
  *   client/pom.xml (7ad2eff)
  *   
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtDomainXMLParser.java
 (b622b6d)
  *   
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java 
(c93aeeb)
  *   plugins/network-elements/midokura-midonet/pom.xml (7f2e2d3)
  *   plugins/network-elements/midonet/pom.xml (PRE-CREATION)
  *   
plugins/network-elements/midonet/src/com/cloud/network/element/MidoNetElement.java
 (48833b3)
  *   
plugins/network-elements/midonet/src/com/cloud/network/element/SimpleFirewallRule.java
 (PRE-CREATION)
  *   
plugins/network-elements/midonet/src/com/cloud/network/guru/MidoNetGuestNetworkGuru.java
 (ed0eb3c)
  *   
plugins/network-elements/midonet/src/com/cloud/network/guru/MidoNetPublicNetworkGuru.java
 (PRE-CREATION)
  *   
plugins/network-elements/midonet/src/com/cloud/network/resource/MidoNetVifDriver.java
 (PRE-CREATION)
  *   
plugins/network-elements/midonet/test/com/cloud/network/element/MidoNetElementTest.java
 (PRE-CREATION)
  *   plugins/pom.xml (39d9907)
  *   server/src/com/cloud/configuration/Config.java (9db7dbd)
  *   server/src/com/cloud/network/NetworkManagerImpl.java (b1236cc)
  *   ui/scripts/system.js (c0a5d14)

View Diff<https://reviews.apache.org/r/9898/diff/>


Reply via email to