> On Jan. 11, 2014, 12:53 a.m., Sheng Yang wrote: > > The attached file is also the same file like others. > > > > Please check your uploaded file.
Hi Sheng, Its my mistake. Please use the just updated file. - Suresh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16642/#review31568 ----------------------------------------------------------- On Jan. 11, 2014, 12:59 a.m., Suresh Balineni wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16642/ > ----------------------------------------------------------- > > (Updated Jan. 11, 2014, 12:59 a.m.) > > > Review request for cloudstack. > > > Repository: cloudstack-git > > > Description > ------- > > - Contrail plugin bug fixes for CS 4.3. > - Code cleanup. > - Most of the fixes are in plugin code. > > > 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 > > Diff: https://reviews.apache.org/r/16642/diff/ > > > Testing > ------- > > verified locally. > > > File Attachments > ---------------- > > Good file > > https://reviews.apache.org/media/uploaded/files/2014/01/11/9ce72fe1-cb37-42c7-838b-395229b3e082__fixes.patch > > > Thanks, > > Suresh Balineni > >
