Re: Review Request: BigSwitch networking plugin update

[Chiradeep Vittal <ch...@gmail.com>]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9671/#review17342
-----------------------------------------------------------


Kanzhe, this is too late for 4.1
Please re-submit against master
Also, please fix 13445 in 4.1 by remving references to VNS

- Chiradeep Vittal


On Feb. 28, 2013, 7:33 a.m., Kanzhe Jiang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9671/
> -----------------------------------------------------------
> 
> (Updated Feb. 28, 2013, 7:33 a.m.)
> 
> 
> Review request for cloudstack and Murali Reddy.
> 
> 
> Description
> -------
> 
> Update BigSwitch Plugin:
> . UI support for adding BigSwitch controller as a network service provider
> . Correct the message format between the plugin and the controller
> . Add Health check API
> 
> 
> This addresses bugs CLOUDSTACK-1345 and CLOUDSTACK-733.
> 
> 
> Diffs
> -----
> 
>   client/WEB-INF/classes/resources/messages.properties 9c4fcb3 
>   client/tomcatconf/componentContext.xml.in c45ab1b 
>   client/tomcatconf/nonossComponentContext.xml.in 5532bec 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/api/commands/DeleteBigSwitchVnsDeviceCmd.java f26bdde 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/api/commands/ListBigSwitchVnsDevicesCmd.java 002d750 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/api/response/BigSwitchVnsDeviceResponse.java ede5adf 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/network/BigSwitchVnsDeviceVO.java 7e64833 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/Attachment.java 9a13467 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/AttachmentData.java PRE-CREATION 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/BigSwitchVnsApi.java c409dce 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/ControlClusterStatus.java 9181afd 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/Network.java c89c693 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/NetworkData.java PRE-CREATION 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/Port.java bf749d0 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/network/bigswitch/PortData.java PRE-CREATION 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/network/element/BigSwitchVnsElement.java 95c9e18 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/network/guru/BigSwitchVnsGuestNetworkGuru.java b96ea56 
>   plugins/network-elements/bigswitch-vns/src/com/cloud/network/resource/BigSwitchVnsResource.java 8d070b5 
>   plugins/network-elements/bigswitch-vns/test/com/cloud/network/bigswitch/BigSwitchApiTest.java cab06ba 
>   plugins/network-elements/bigswitch-vns/test/com/cloud/network/resource/BigSwitchVnsResourceTest.java 514385f 
>   ui/dictionary.jsp 5081356 
>   ui/scripts/system.js e51bf90 
> 
> Diff: https://reviews.apache.org/r/9671/diff/
> 
> 
> Testing
> -------
> 
> Test to add a BigSwitch controller as a network service provider and view.
> Test to implementation of guest network on the controller
> Test to associate VM to its guest network
> 
> Test procedure:
> 1. Create an advanced zone with isolation = VNS
> 2. Add BigSwitch controller as the network service provider for the physical network
> 3. Configure OVS on the host to point to the controller
> 4. Create a guest network
> 5. Create a VM instance and make sure it belongs to the right network.
> 
> 
> Thanks,
> 
> Kanzhe Jiang
> 
>