----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10970/ -----------------------------------------------------------
(Updated May 30, 2013, 11:58 a.m.) Review request for cloudstack, Murali Reddy, Hugo Trippaers, and Chiradeep Vittal. Changes ------- applied comment on pvlan configuration and fixed some formatting Description ------- converting vlan id to uri to support a broader range of networks in for instance vpc gateway connections Diffs (updated) ----- 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 405cecd api/src/com/cloud/network/Networks.java 5aede05 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 5d278e9 api/src/com/cloud/network/vpc/VpcService.java 7a444c0 api/src/org/apache/cloudstack/api/command/admin/vpc/CreatePrivateGatewayCmd.java 22dfb9e api/src/org/apache/cloudstack/api/response/PrivateGatewayResponse.java c5c7df5 core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java 8b996d1 engine/schema/src/com/cloud/network/vpc/VpcGatewayVO.java 7df2dfd plugins/hypervisors/baremetal/src/com/cloud/baremetal/networkservice/BaremetaNetworkGuru.java 6d14e3f plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/BridgeVifDriver.java b897df2 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java f979cfe plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/OvsVifDriver.java eac3248 plugins/hypervisors/ovm/src/com/cloud/ovm/hypervisor/OvmResourceBase.java a626e31 plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java 0a15d4b plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java 7626d12 plugins/network-elements/cisco-vnmc/test/com/cloud/network/element/CiscoVnmcElementTest.java a16733b plugins/network-elements/f5/src/com/cloud/network/resource/F5BigIpResource.java 1733712 plugins/network-elements/juniper-srx/src/com/cloud/network/resource/JuniperSrxResource.java fd065d5 plugins/network-elements/netscaler/src/com/cloud/network/resource/NetscalerResource.java c0d4599 plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsTunnelManagerImpl.java b1ecaac server/src/com/cloud/api/ApiResponseHelper.java cf79ff8 server/src/com/cloud/configuration/ConfigurationManagerImpl.java 59e70cf server/src/com/cloud/network/ExternalFirewallDeviceManagerImpl.java 9d24e47 server/src/com/cloud/network/ExternalLoadBalancerDeviceManagerImpl.java 04f9682 server/src/com/cloud/network/ExternalLoadBalancerUsageManagerImpl.java 2c8031c server/src/com/cloud/network/NetworkManager.java 8627251 server/src/com/cloud/network/NetworkManagerImpl.java b92ef4b server/src/com/cloud/network/NetworkServiceImpl.java 9899284 server/src/com/cloud/network/guru/DirectPodBasedNetworkGuru.java cf27986 server/src/com/cloud/network/guru/PrivateNetworkGuru.java 2e266e7 server/src/com/cloud/network/guru/PublicNetworkGuru.java a83cdb3 server/src/com/cloud/network/router/VpcVirtualNetworkApplianceManagerImpl.java 9992b7c server/src/com/cloud/network/vpc/PrivateGatewayProfile.java d6480cd server/src/com/cloud/network/vpc/PrivateIpAddress.java 2f3cf53 server/src/com/cloud/network/vpc/VpcManagerImpl.java 380a95e server/test/com/cloud/network/CreatePrivateNetworkTest.java PRE-CREATION server/test/com/cloud/network/MockNetworkManagerImpl.java 7ab322b server/test/com/cloud/vpc/MockNetworkManagerImpl.java d46be7c server/test/com/cloud/vpc/MockVpcManagerImpl.java 921321f setup/db/db/schema-410to420.sql 196706f Diff: https://reviews.apache.org/r/10970/diff/ Testing ------- the NetworkTO is tested to accept uris with several initial states. createPrivateNetwork in NetworkServiceImpl is tested (to accept only vlan or lswitch based networks for now) test code that used to use 'vlan://#' now uses 'vlan:#' Thanks, daan Hoogland