cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David Nalley <da...@gnsa.us>
Subject Re: Review Request 16841: (ACS4.3): Add new physical network isolationMethod (L3VPN) for Juniper Contrail plugin.
Date Tue, 14 Jan 2014 21:55:47 GMT
Why are we adding functionality after 4.3 has closed?

--David

On Tue, Jan 14, 2014 at 4:53 PM, Sheng Yang <sheng@yasker.org> wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16841/#review31810
> -----------------------------------------------------------
>
> Ship it!
>
>
> Applied to 4.3 branch.
>
> Thanks.
>
>
> - Sheng Yang
>
>
> On Jan. 14, 2014, 1:54 a.m., Sachchidanand Vaidya wrote:
>>
>> -----------------------------------------------------------
>> This is an automatically generated e-mail. To reply, visit:
>> https://reviews.apache.org/r/16841/
>> -----------------------------------------------------------
>>
>> (Updated Jan. 14, 2014, 1:54 a.m.)
>>
>>
>> Review request for cloudstack.
>>
>>
>> Repository: cloudstack-git
>>
>>
>> Description
>> -------
>>
>> This diff is on ACS4.3 (Similar to Review Request 16258 for ACS-4.4).
>>
>>
>> Diffs
>> -----
>>
>>   api/src/com/cloud/network/PhysicalNetwork.java 55b18e6
>>   plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailGuru.java
6587534
>>   plugins/network-elements/juniper-contrail/test/org/apache/cloudstack/network/contrail/management/ManagementServerMock.java
bbff466
>>   ui/scripts/ui-custom/zoneWizard.js 28df193
>>
>> Diff: https://reviews.apache.org/r/16841/diff/
>>
>>
>> Testing
>> -------
>>
>> Unit testing done.
>>
>>
>> Thanks,
>>
>> Sachchidanand Vaidya
>>
>>
>

Mime
View raw message