Sheng, Dave, Chiradeep and Hugo, Can you please review this? In my experience the time that this patch will expire is rather short, I have been resolving conflicts on this a lot. If now, given the 4.2 release is inconvenient, I would like to set a window for submitting this in which I will rebase it a couple of times so as to make sure it works in the end.
thanks, Daan On Fri, Aug 23, 2013 at 10:48 AM, daan Hoogland <daan.hoogl...@gmail.com>wrote: > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13771/ > Review request for cloudstack, Chiradeep Vittal, Dave Cahill, Hugo > Trippaers, and Sheng Yang. > By daan Hoogland. > *Bugs: * CLOUDSTACK-4346 > *Repository: * cloudstack-git > Description > > After global search and replace all calls to retrieve ids for networks from > URIs using getHost() should be gone. Creating URI should now all use > appropriate calls as well so maitaining the way uris are built can now be > done centrally. > > Testing > > tested with old style uris in regular networks and vpc based networks as well > as in nicira based networks > test build in nonoss but not all code has probably been touched yet. or at > least I am unsure of that. > > Diffs > > - > plugins/hypervisors/baremetal/src/com/cloud/baremetal/networkservice/BaremetaNetworkGuru.java > (07ee12d) > - > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/BridgeVifDriver.java > (195cf40) > - > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java > (a156ae6) > - > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/OvsVifDriver.java > (7038d7e) > - plugins/hypervisors/ovm/src/com/cloud/ovm/hypervisor/OvmResourceBase.java > (59ba001) > - > plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java > (5ab2216) > - > plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java > (ecdec1e) > - > plugins/network-elements/bigswitch-vns/src/com/cloud/network/element/BigSwitchVnsElement.java > (54623e9) > - > plugins/network-elements/cisco-vnmc/src/com/cloud/network/element/CiscoVnmcElement.java > (3ae6a08) > - > plugins/network-elements/f5/src/com/cloud/network/resource/F5BigIpResource.java > (1733712) > - > plugins/network-elements/juniper-srx/src/com/cloud/network/resource/JuniperSrxResource.java > (3d3d797) > - > plugins/network-elements/nicira-nvp/src/com/cloud/network/element/NiciraNvpElement.java > (c7d0884) > - > plugins/network-elements/nicira-nvp/src/com/cloud/network/guru/NiciraNvpGuestNetworkGuru.java > (ff238ed) > - > plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsTunnelManagerImpl.java > (36a807f) > - server/src/com/cloud/api/ApiResponseHelper.java (c771431) > - server/src/com/cloud/configuration/ConfigurationManagerImpl.java > (57dc0b3) > - server/src/com/cloud/network/ExternalDeviceUsageManagerImpl.java > (e91dcfa) > - server/src/com/cloud/network/ExternalFirewallDeviceManagerImpl.java > (a934024) > - server/src/com/cloud/network/ExternalLoadBalancerDeviceManagerImpl.java > (c14d5c7) > - server/src/com/cloud/network/NetworkManagerImpl.java (00103e3) > - server/src/com/cloud/network/guru/DirectPodBasedNetworkGuru.java > (5b87d54) > - server/src/com/cloud/network/guru/ExternalGuestNetworkGuru.java > (00598dd) > - server/src/com/cloud/network/guru/GuestNetworkGuru.java (b0da42f) > - server/src/com/cloud/network/guru/PrivateNetworkGuru.java (6521cf4) > - server/src/com/cloud/network/guru/PublicNetworkGuru.java (d109468) > - > server/src/com/cloud/network/router/VpcVirtualNetworkApplianceManagerImpl.java > (ee0d058) > - utils/src/com/cloud/utils/net/NetUtils.java (05b485b) > > View Diff <https://reviews.apache.org/r/13771/diff/> >