cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sheng Yang <sh...@yasker.org>
Subject Re: Review Request 16841: (ACS4.3): Add new physical network isolationMethod (L3VPN) for Juniper Contrail plugin.
Date Tue, 14 Jan 2014 21:57:12 GMT
I don't think it's a feature after checking the patch.

It's basically they called it "L3VPN" instead of "GRE" in the patch I think.

--Sheng


On Tue, Jan 14, 2014 at 1:55 PM, David Nalley <david@gnsa.us> wrote:

> 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
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message