Hi Sheng, Its my mistake. I just updated review with new file fixes.patch.
Please find the attached one. This is same. Thanks, Suresh From: Sheng Yang <sh...@yasker.org<mailto:sh...@yasker.org>> Reply-To: Sheng Yang <sh...@yasker.org<mailto:sh...@yasker.org>> Date: Friday, January 10, 2014 4:53 PM To: Suresh Balineni <sbalin...@service.juniper.net<mailto:sbalin...@service.juniper.net>>, cloudstack <dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>>, Sheng Yang <sh...@yasker.org<mailto:sh...@yasker.org>> Subject: Re: Review Request 16642: Contrail plguin fixes for CS 4.3 This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16642/ The attached file is also the same file like others. Please check your uploaded file. - Sheng Yang On January 11th, 2014, 12:51 a.m. UTC, Suresh Balineni wrote: Review request for cloudstack. By Suresh Balineni. Updated Jan. 11, 2014, 12:51 a.m. Repository: cloudstack-git Description - Contrail plugin bug fixes for CS 4.3. - Code cleanup. - Most of the fixes are in plugin code. Testing verified locally. Diffs * api/src/com/cloud/network/Network.java (0d916ab) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailElementImpl.java (dfbe7d3) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailGuru.java (ca53f41) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailManager.java (b96b00c) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailManagerImpl.java (4eaafca) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ModelDatabase.java (f705f07) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ServerDBSyncImpl.java (7a77040) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ServerEventHandlerImpl.java (ec37440) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ServiceManagerImpl.java (47fc99e) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/NetworkPolicyModel.java (PRE-CREATION) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/ServiceInstanceModel.java (20b80be) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/VirtualMachineModel.java (ec20a94) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/VirtualNetworkModel.java (b4968ac) * plugins/network-elements/juniper-contrail/test/org/apache/cloudstack/network/contrail/management/ManagementServerMock.java (86750ca) * plugins/network-elements/juniper-contrail/test/org/apache/cloudstack/network/contrail/management/NetworkProviderTest.java (40cd1ea) View Diff<https://reviews.apache.org/r/16642/diff/> File Attachments * Use thief patch file <https://reviews.apache.org/media/uploaded/files/2014/01/11/ed6deb38-08b3-4ee3-bf44-68f42dd7d335__fixes.patch>
