----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13934/ -----------------------------------------------------------
Review request for cloudstack, Chiradeep Vittal, Hugo Trippaers, and Sheng Yang. Bugs: 4347 Repository: cloudstack-git Description ------- provisioning of a nicira based vpc router Diffs ----- api/src/com/cloud/agent/api/to/IpAddressTO.java 82c7d99 api/src/com/cloud/agent/api/to/NetworkTO.java 3edd4c0 api/src/com/cloud/network/NetworkService.java 87fecb0 api/src/com/cloud/network/Networks.java f8166c6 api/src/com/cloud/network/vpc/PrivateIp.java eb68433 api/src/com/cloud/network/vpc/StaticRouteProfile.java 54aa6e4 api/src/com/cloud/network/vpc/VpcGateway.java 9652b4b api/src/com/cloud/network/vpc/VpcService.java 7a444c0 api/src/org/apache/cloudstack/api/command/admin/vpc/CreatePrivateGatewayCmd.java 0bc905c api/src/org/apache/cloudstack/api/response/PrivateGatewayResponse.java c5c7df5 api/test/com/cloud/network/NetworksTest.java 07b55d2 core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java 1fb86e0 engine/schema/src/com/cloud/network/vpc/VpcGatewayVO.java 2c592cd engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java eb987ea plugins/hypervisors/baremetal/src/com/cloud/baremetal/networkservice/BaremetaNetworkGuru.java f5e0897 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/BridgeVifDriver.java c94856d plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java b56f3ca plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/OvsVifDriver.java 119f117 plugins/hypervisors/ovm/src/com/cloud/ovm/hypervisor/OvmResourceBase.java 8f21c13 plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java 8853b88 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java 35f30e4 plugins/network-elements/bigswitch-vns/src/com/cloud/network/guru/BigSwitchVnsGuestNetworkGuru.java 52303a4 plugins/network-elements/cisco-vnmc/src/com/cloud/network/element/CiscoVnmcElement.java a9f3256 plugins/network-elements/f5/src/com/cloud/network/resource/F5BigIpResource.java ffddd30 plugins/network-elements/juniper-srx/src/com/cloud/network/resource/JuniperSrxResource.java f9dd5e3 plugins/network-elements/netscaler/src/com/cloud/network/resource/NetscalerResource.java 58541c6 plugins/network-elements/nicira-nvp/src/com/cloud/network/element/NiciraNvpElement.java a5f1b92 plugins/network-elements/nicira-nvp/src/com/cloud/network/guru/NiciraNvpGuestNetworkGuru.java 9451441 plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsTunnelManagerImpl.java e8ff1a0 server/src/com/cloud/api/ApiResponseHelper.java 8d01274 server/src/com/cloud/configuration/ConfigurationManagerImpl.java 2fdc9f2 server/src/com/cloud/network/ExternalDeviceUsageManagerImpl.java 3db5111 server/src/com/cloud/network/ExternalFirewallDeviceManagerImpl.java ec48d13 server/src/com/cloud/network/ExternalLoadBalancerDeviceManagerImpl.java f594c8c server/src/com/cloud/network/NetworkManagerImpl.java 7feeb14 server/src/com/cloud/network/NetworkServiceImpl.java 7d0f9c7 server/src/com/cloud/network/guru/DirectPodBasedNetworkGuru.java 9ae636e server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java bed61ea server/src/com/cloud/network/guru/GuestNetworkGuru.java e980df9 server/src/com/cloud/network/guru/PrivateNetworkGuru.java 9f02175 server/src/com/cloud/network/router/VpcVirtualNetworkApplianceManagerImpl.java abe0746 server/src/com/cloud/network/vpc/PrivateGatewayProfile.java 74ce002 server/src/com/cloud/network/vpc/PrivateIpAddress.java 2f3cf53 server/src/com/cloud/network/vpc/VpcManagerImpl.java bce2d72 server/test/com/cloud/network/CreatePrivateNetworkTest.java dbb2b1f server/test/com/cloud/vpc/MockNetworkManagerImpl.java 7df4c9c server/test/com/cloud/vpc/MockVpcManagerImpl.java 7e40083 utils/src/com/cloud/utils/net/NetUtils.java 7a89103 Diff: https://reviews.apache.org/r/13934/diff/ Testing ------- This is a rebase/port of code that has been tested intensively. It is here for review only at this time. re-tests are needed. Thanks, daan Hoogland
