Hi Murali and Hugo, For L2 isolation, there isn't much difference between Nicira NVP and BigSwitch VNS. Obviously, the differences are on the controllers. Nicira NVP works with OVS with Nicira extension, while BigSwitch Controller works with stock OVS and all physical OpenFlow switches. I referenced Hugo's NVP plugin to implement the BigSwitch one. Hence, it should be familiar to you, Hugo.
I am still testing the plugin with the controller and will fill in the functional testing details shortly. Given the deadline in 3 days to be included in 4.1, I would like to start the review process first if it is ok. The current version uses the 1.1 version of the controller API, which is not available on floodlight yet. So the plugin won't work with floodlight for now. The v1.1 API will be supported by floodlight shortly and I will try to document the step as well. In summary, this is the bulk of the plugin. I will provide update on functional testing, documentation, and integration with floodlight shortly. Thanks, Kanzhe On Tue, Jan 29, 2013 at 1:13 AM, Murali Reddy <[email protected]>wrote: > > Kanze, > > Thanks for the patch. On first look patch look good, but I will do a > detailed review tomorrow. > > Meanwhile could you help me understand how the network is > provisioned/de-provisioned through the contrloller? I understand that you > plan to support Vlan isolation only for 4.1, so what is the role of > controller in this case? So when CreateNetwork, DeleteNetwork commands are > sent by the Guru in network design/destroy phases, does controller setup's > Vlans on the open flow enabled switches and hypervisors? > > Thanks, > Murali > > > On 29/01/13 11:29 AM, "Kanzhe Jiang" <[email protected]> wrote: > > > > >----------------------------------------------------------- > >This is an automatically generated e-mail. To reply, visit: > >https://reviews.apache.org/r/9131/ > >----------------------------------------------------------- > > > >Review request for cloudstack. > > > > > >Description > >------- > > > >This is the first patch for BigSwitch Network Plugin to CloudStack. The > >patch follows the design spec that can be found at > > > https://cwiki.apache.org/confluence/display/CLOUDSTACK/BigSwitch+Networkin > >g+Plugin. > > > > > >Diffs > >----- > > > > api/src/com/cloud/api/ApiConstants.java 78a3ded > > api/src/com/cloud/event/EventTypes.java e69e4a4 > > api/src/com/cloud/network/Network.java d38f740 > > api/src/com/cloud/network/PhysicalNetwork.java be4b1d0 > > build/build-cloud-plugins.xml 207ef71 > > build/developer.xml fdda171 > > client/pom.xml 1673429 > > client/tomcatconf/bigswitch-vns_commands.properties.in PRE-CREATION > > client/tomcatconf/components.xml.in 5957b61 > > cloud.spec 9f46dd6 > > debian/cloud-server.install 9cd1eeb > > docs/en-US/Release_Notes.xml c8cc686 > > plugins/network-elements/bigswitch-vns/pom.xml PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/agent/api/CreateNetwo > >rkAnswer.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/agent/api/CreateNetwo > >rkCommand.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/agent/api/CreatePortA > >nswer.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/agent/api/CreatePortC > >ommand.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/agent/api/DeleteNetwo > >rkAnswer.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/agent/api/DeleteNetwo > >rkCommand.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/agent/api/DeletePortA > >nswer.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/agent/api/DeletePortC > >ommand.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/agent/api/StartupBigS > >witchVnsCommand.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/agent/api/UpdatePortA > >nswer.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/agent/api/UpdatePortC > >ommand.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/api/commands/AddBigSw > >itchVnsDeviceCmd.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/api/commands/DeleteBi > >gSwitchVnsDeviceCmd.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/api/commands/ListBigS > >witchVnsDevicesCmd.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/api/response/BigSwitc > >hVnsDeviceResponse.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/network/BigSwitchVnsD > >eviceVO.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/Att > >achment.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/Big > >SwitchVnsApi.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/Big > >SwitchVnsApiException.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/Con > >trolClusterStatus.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/Net > >work.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/Por > >t.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/network/dao/BigSwitch > >VnsDao.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/network/dao/BigSwitch > >VnsDaoImpl.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/network/element/BigSw > >itchVnsElement.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/network/element/BigSw > >itchVnsElementService.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/network/guru/BigSwitc > >hVnsGuestNetworkGuru.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/src/com/cloud/network/resource/BigS > >witchVnsResource.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/test/com/cloud/network/bigswitch/Bi > >gSwitchApiTest.java PRE-CREATION > > > >plugins/network-elements/bigswitch-vns/test/com/cloud/network/resource/Big > >SwitchVnsResourceTest.java PRE-CREATION > > plugins/pom.xml 2009302 > > server/src/com/cloud/network/ExternalNetworkDeviceManager.java b1de86f > > setup/db/create-schema.sql fff084e > > tools/apidoc/gen_toc.py eeaf2a2 > > tools/apidoc/pom.xml b75ee82 > > wscript_configure 3b9377b > > > >Diff: https://reviews.apache.org/r/9131/diff/ > > > > > >Testing > >------- > > > >Verify the builds: mvn clean install; > >The database can be deployed; > >CloudStack can be launched with the plugin modules. > > > >JUnit tests of all api commands to a mock controller. > > > > > >Thanks, > > > >Kanzhe Jiang > > > > > > > -- Kanzhe Jiang MTS at BigSwitch
