Murali please review and apply the patch if satisfied

> -----Original Message-----
> From: Kanzhe Jiang [mailto:nore...@reviews.apache.org] On Behalf Of
> Kanzhe Jiang
> Sent: Wednesday, March 06, 2013 5:30 PM
> To: Murali Reddy
> Cc: cloudstack; Kanzhe Jiang
> Subject: Review Request: BigSwitch networking plugin update
> 
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9789/
> -----------------------------------------------------------
> 
> 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 bug 733.
> 
> 
> Diffs
> -----
> 
>   client/WEB-INF/classes/resources/messages.properties d167a5c
>   client/tomcatconf/componentContext.xml.in 23566e2
>   client/tomcatconf/nonossComponentContext.xml.in 20e0c32
>   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 3aecaaa
>   ui/scripts/system.js 4d529ae
> 
> Diff: https://reviews.apache.org/r/9789/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 two guest networks 5. Create
> two VM instances and make sure each VM belongs to the right network.
> 6. Confirm VMs in the same network can ping each other and the virtual
> router.
> 
> 
> Thanks,
> 
> Kanzhe Jiang

Reply via email to