Juniper Contrail Plugin

2016-07-20 Thread Venkata Yedugundla
Hi all, I see juniper contrail plugin publishing events to message bus. It is causing a deadlock in a specific scenario. May I know the reason for the same. The following is the code piece I am referring to and the corresponding event is not subscribed to far. public void interceptComplete

Re: Review Request 19270: Added service-instance delete command to Juniper Contrail Plugin and related fixes

2015-04-06 Thread Sebastien Goasguen
sh Battala. > > > Repository: cloudstack-git > > > Description > --- > > This patch has following changes: > - Added service-instance delete command to Juniper Contrail Plugin. > - Fixes to plugin code to handle instance-delete operation properly. > -

Re: Review Request 19270: Added service-instance delete command to Juniper Contrail Plugin and related fixes

2014-06-09 Thread Rajesh Battala
pository: cloudstack-git > > > Description > --- > > This patch has following changes: > - Added service-instance delete command to Juniper Contrail Plugin. > - Fixes to plugin code to handle instance-delete operation properly. > - Fix related to service-instance

Re: Review Request 19270: Added service-instance delete command to Juniper Contrail Plugin and related fixes

2014-05-20 Thread Sachchidanand Vaidya
te and delete. > >Thanks, >Sachin > > >Repository: cloudstack-git > > >Description >--- > >This patch has following changes: > - Added service-instance delete command to Juniper Contrail Plugin. > - Fixes to plugin code to handle instance-delete operati

Re: Review Request 19270: Added service-instance delete command to Juniper Contrail Plugin and related fixes

2014-05-16 Thread Sachchidanand Vaidya
Battala. Changes --- Fixed licenser header and created new EventTypes for serviceInstance VM create and delete. Thanks, Sachin Repository: cloudstack-git Description --- This patch has following changes: - Added service-instance delete command to Juniper Contrail Plugin. - Fixes to

Re: Review Request 19270: Added service-instance delete command to Juniper Contrail Plugin and related fixes

2014-05-14 Thread Sachchidanand Vaidya
dstack and Rajesh Battala. > > > Repository: cloudstack-git > > > Description > --- > > This patch has following changes: > - Added service-instance delete command to Juniper Contrail Plugin. > - Fixes to plugin code to handle instance-delete

Re: Review Request 19270: Added service-instance delete command to Juniper Contrail Plugin and related fixes

2014-05-13 Thread Sachchidanand Vaidya
t; --- > > This patch has following changes: > - Added service-instance delete command to Juniper Contrail Plugin. > - Fixes to plugin code to handle instance-delete operation properly. > - Fix related to service-instance persistence and re-creation during Mgmt >

Re: Review Request 19270: Added service-instance delete command to Juniper Contrail Plugin and related fixes

2014-05-05 Thread Rajesh Battala
t; Review request for cloudstack and Rajesh Battala. > > > Repository: cloudstack-git > > > Description > --- > > This patch has following changes: > - Added service-instance delete command to Juniper Contrail Plugin. > - Fixes to plugin code to handle ins

Re: Review Request 19270: Added service-instance delete command to Juniper Contrail Plugin and related fixes

2014-05-05 Thread Rajesh Battala
(Updated May 3, 2014, 7:29 a.m.) > > > Review request for cloudstack and Rajesh Battala. > > > Repository: cloudstack-git > > > Description > --- > > This patch has following changes: > - Added service-instance delete command to Juniper Co

Re: Review Request 19270: Added service-instance delete command to Juniper Contrail Plugin and related fixes

2014-05-03 Thread Sachchidanand Vaidya
Battala. Changes --- . Repository: cloudstack-git Description --- This patch has following changes: - Added service-instance delete command to Juniper Contrail Plugin. - Fixes to plugin code to handle instance-delete operation properly. - Fix related to service-instance persistence

Review Request 19270: Added service-instance delete command to Juniper Contrail Plugin and related fixes

2014-03-16 Thread Sachchidanand Vaidya
--- This patch has following changes: - Added service-instance delete command to Juniper Contrail Plugin. - Fixes to plugin code to handle instance-delete operation properly. - Fix related to service-instance persistence and re-creation during Mgmt server reboot. Diffs - client

Re: Review Request 17307: Juniper Contrail Plugin fixes

2014-01-28 Thread Hugo Trippaers
: Sachchidanand Vaidya Date: Wed Jan 8 00:26:46 2014 -0800 These are additional fixes to make Juniper Contrail plugin work - Hugo Trippaers On Jan. 24, 2014, 8:06 a.m., Sachchidanand Vaidya wrote: > > --- > This is an auto

Review Request 17307: Juniper Contrail Plugin fixes

2014-01-24 Thread Sachchidanand Vaidya
--- These are additional fixes to make Juniper Contrail plugin work on ACS master Diffs - plugins/network-elements/juniper-contrail/pom.xml ae6961e plugins/network-elements/juniper-contrail/resources/META-INF/cloudstack/contrail/module.properties PRE-CREATION plugins/network

Re: Review Request 16974: (ACS4.3) Juniper Contrail Plugin fixes

2014-01-16 Thread Sachchidanand Vaidya
11:01 AM To: Admin mailto:vaidy...@juniper.net>>, Hugo Trippaers mailto:htrippa...@schubergphilis.com>>, cloudstack mailto:dev@cloudstack.apache.org>> Subject: Re: Review Request 16974: (ACS4.3) Juniper Contrail Plugin fixes This is an automatically generated e

Re: Review Request 16974: (ACS4.3) Juniper Contrail Plugin fixes

2014-01-16 Thread David Nalley
ated Jan. 16, 2014, 6:53 p.m.) >> >> >> Review request for cloudstack. >> >> >> Repository: cloudstack-git >> >> >> Description >> --- >> >> These are additional fixes to make Juniper Contrail plugin work in ACS4.3. >> These chang

Re: Review Request 16974: (ACS4.3) Juniper Contrail Plugin fixes

2014-01-16 Thread Hugo Trippaers
.org/r/16974/ > --- > > (Updated Jan. 16, 2014, 6:53 p.m.) > > > Review request for cloudstack. > > > Repository: cloudstack-git > > > Description > --- > > These are additional fixes to mak

Review Request 16974: (ACS4.3) Juniper Contrail Plugin fixes

2014-01-16 Thread Sachchidanand Vaidya
--- These are additional fixes to make Juniper Contrail plugin work in ACS4.3. These changes were missed during last commit Diffs - plugins/network-elements/juniper-contrail/pom.xml 22167fe plugins/network-elements/juniper-contrail/resources/META-INF/cloudstack/contrail/module.properties

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

2014-01-14 Thread Sachchidanand Vaidya
Thanks Sheng. Best Regards, Sachin On 1/14/14 1:56 PM, "Sheng Yang" wrote: > >--- >This is an automatically generated e-mail. To reply, visit: >https://reviews.apache.org/r/16258/#review31812 >--

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 16258: 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/16258/#review31812 --- Ship it! Applied to MASTER. Thanks. - Sheng Yang On Jan. 14, 20

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

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

2014-01-14 Thread Sachchidanand Vaidya
Hi Sheng, I have updated the review with changes based off latest on ACS master. Thanks & Best Regards, Sachin On 1/14/14 1:25 PM, "Sheng Yang" wrote: > >--- >This is an automatically generated e-mail. To reply, visit: >https://reviews

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

2014-01-14 Thread Sachchidanand Vaidya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16258/ --- (Updated Jan. 14, 2014, 9:44 p.m.) Review request for cloudstack. Changes ---

Re: Review Request 16258: 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/16258/#review31802 --- Looks good to me, but cannot applied to the latest MASTER branch(I s

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 ---

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

2014-01-12 Thread Sachchidanand Vaidya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16258/ --- (Updated Jan. 13, 2014, 5:46 a.m.) Review request for cloudstack. Changes ---

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

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