Re: Review Request 16841: (ACS4.3): Add new physical network isolationMethod (L3VPN) for Juniper Contrail plugin.

2014-01-14 Thread Sachchidanand Vaidya
dev@cloudstack.apache.org>" mailto:dev@cloudstack.apache.org>>, Admin mailto:vaidy...@juniper.net>> Subject: Re: Review Request 16841: (ACS4.3): Add new physical network isolationMethod (L3VPN) for Juniper Contrail plugin. I don't think it's a feature after ch

Re: Review Request 16841: (ACS4.3): Add new physical network isolationMethod (L3VPN) for Juniper Contrail plugin.

2014-01-14 Thread Sheng Yang
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 wrote: > Why are we adding functionality after 4.3 has closed? > > --David > > On Tue, Jan 14, 2014 at 4:53

Re: Review Request 16841: (ACS4.3): Add new physical network isolationMethod (L3VPN) for Juniper Contrail plugin.

2014-01-14 Thread David Nalley
Why are we adding functionality after 4.3 has closed? --David On Tue, Jan 14, 2014 at 4:53 PM, Sheng Yang wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16841/#review31810 > -

Re: Review Request 16841: (ACS4.3): Add new physical network isolationMethod (L3VPN) for Juniper Contrail plugin.

2014-01-14 Thread Sheng Yang
--- 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. 1

Review Request 16841: (ACS4.3): Add new physical network isolationMethod (L3VPN) for Juniper Contrail plugin.

2014-01-13 Thread Sachchidanand Vaidya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16841/ --- Review request for cloudstack. Repository: cloudstack-git Description ---