----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9789/#review18512 -----------------------------------------------------------
Ship it! Branch: refs/heads/master Commit: ee3fd1843533528eb3872951730fee6e5802c348 Parents: 690c312 Author: Kanzhe Jiang <[email protected]> Authored: Fri Mar 29 18:48:50 2013 +0530 Committer: Murali Reddy <[email protected]> Committed: Fri Mar 29 18:54:48 2013 +0530 - Murali Reddy On March 29, 2013, 6:38 a.m., Kanzhe Jiang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9789/ > ----------------------------------------------------------- > > (Updated March 29, 2013, 6:38 a.m.) > > > Review request for cloudstack, Murali Reddy and Chiradeep Vittal. > > > 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 > >
