cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Suresh Balineni <sbalin...@service.juniper.net>
Subject Re: Review Request 16642: Contrail plguin fixes for CS 4.3
Date Sat, 11 Jan 2014 01:01:44 GMT
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 <sheng@yasker.org<mailto:sheng@yasker.org>>
Reply-To: Sheng Yang <sheng@yasker.org<mailto:sheng@yasker.org>>
Date: Friday, January 10, 2014 4:53 PM
To: Suresh Balineni <sbalineni@service.juniper.net<mailto:sbalineni@service.juniper.net>>,
cloudstack <dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>>, Sheng
Yang <sheng@yasker.org<mailto:sheng@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>


Mime
  • Unnamed multipart/mixed (inline, None, 0 bytes)
View raw message