----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13934/#review25884 -----------------------------------------------------------
This looks reasonable to me. For future reviews though, could you avoid the high ratio of whitespace changes? This one is about 99% whitespace changes, which makes it extremely hard to see what has actually changed. - Dave Cahill On Sept. 2, 2013, 2:53 p.m., daan Hoogland wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13934/ > ----------------------------------------------------------- > > (Updated Sept. 2, 2013, 2:53 p.m.) > > > 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 > >