cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sachchidanand Vaidya <vaidy...@juniper.net>
Subject Re: Review Request 16974: (ACS4.3) Juniper Contrail Plugin fixes
Date Thu, 16 Jan 2014 20:10:34 GMT
Hi Hugo,
    These fixes are blocker for contrail plugin. We missed this fix while submitting patches
to ACS.

Thanks,
Sachin

From: Hugo Trippaers <htrippaers@schubergphilis.com<mailto:htrippaers@schubergphilis.com>>
Reply-To: Hugo Trippaers <htrippaers@schubergphilis.com<mailto:htrippaers@schubergphilis.com>>
Date: Thursday, January 16, 2014 11:01 AM
To: Admin <vaidyasd@juniper.net<mailto:vaidyasd@juniper.net>>, Hugo Trippaers
<htrippaers@schubergphilis.com<mailto:htrippaers@schubergphilis.com>>, cloudstack
<dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>>
Subject: Re: Review Request 16974: (ACS4.3) Juniper Contrail Plugin fixes

This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16974/


Are these fixes release blockers or can they go into 4.4? The release 4.3 is effectively closed
by now.

Cheers,

Hugo


- Hugo Trippaers


On January 16th, 2014, 6:53 p.m. UTC, Sachchidanand Vaidya wrote:

Review request for cloudstack.
By Sachchidanand Vaidya.

Updated Jan. 16, 2014, 6:53 p.m.

Repository: cloudstack-git
Description

These are additional fixes to make Juniper Contrail plugin work in ACS4.3.
These changes were missed during last commit


Testing

Locally tested on centOS6.3 machine.


Diffs

  *   plugins/network-elements/juniper-contrail/pom.xml (22167fe)
  *   plugins/network-elements/juniper-contrail/resources/META-INF/cloudstack/contrail/module.properties
(PRE-CREATION)
  *   plugins/network-elements/juniper-contrail/resources/META-INF/cloudstack/contrail/spring-contrail-context.xml
(PRE-CREATION)
  *   plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailElementImpl.java
(18e8bde)
  *   plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailGuru.java
(6587534)
  *   plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ServiceManagerImpl.java
(82f2cc4)
  *   plugins/network-elements/juniper-contrail/test/org/apache/cloudstack/network/contrail/management/IntegrationTestConfiguration.java
(dec4a9e)
  *   plugins/network-elements/juniper-contrail/test/org/apache/cloudstack/network/contrail/management/MockAccountManager.java
(195e932)
  *   plugins/network-elements/juniper-contrail/test/resources/commonContext.xml (c577513)

View Diff<https://reviews.apache.org/r/16974/diff/>


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