Hi Hugo,
These fixes are blocker for contrail plugin. We missed this fix while
submitting patches to ACS.
Thanks,
Sachin
From: Hugo Trippaers
<[email protected]<mailto:[email protected]>>
Reply-To: Hugo Trippaers
<[email protected]<mailto:[email protected]>>
Date: Thursday, January 16, 2014 11:01 AM
To: Admin <[email protected]<mailto:[email protected]>>, Hugo Trippaers
<[email protected]<mailto:[email protected]>>,
cloudstack <[email protected]<mailto:[email protected]>>
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/>
