Sounds great, thanks Hugo!
On Fri, Mar 29, 2013 at 8:47 PM, Hugo Trippaers < htrippa...@schubergphilis.com> wrote: > 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> 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> 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/> >> > >