cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dave Cahill" <>
Subject Re: Review Request: MidoNet Networking Plugin [2/2]
Date Mon, 25 Mar 2013 04:02:18 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated March 25, 2013, 4:02 a.m.)

Review request for cloudstack, Hugo Trippaers and Chiradeep Vittal.


Rebased from master and updated diff to fix review comments:
* Serving jar from publicly available Maven repo (not Maven central), so plugin is now part
of OSS build
* Moving the MidoNet Provider into plugin code

We don't seem to have consensus on Kawai san's question around ordering of NetworkElement
calls, so no changes made there.

This is still patch #2, where review board 9897 is the first (simple, file rename) patch.

Description (updated)

Feature spec:

Jira ticket:


* 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
"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.


This addresses bug CLOUDSTACK-996.

Diffs (updated)

  api/src/com/cloud/network/ c2ab655 
  api/src/com/cloud/network/ e3d2158 
  api/src/com/cloud/network/ 343a2b1 
  api/src/org/apache/cloudstack/network/ bc22804 
  client/pom.xml 7ad2eff 
  plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/ 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/ 48833b3

  plugins/network-elements/midonet/src/com/cloud/network/element/ PRE-CREATION

  plugins/network-elements/midonet/src/com/cloud/network/resource/ PRE-CREATION

  plugins/pom.xml 39d9907 
  server/src/com/cloud/configuration/ 9db7dbd 
  server/src/com/cloud/network/ b1236cc 
  ui/scripts/system.js c0a5d14 



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


Dave Cahill

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