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> 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/> >