Re: Cloudstack maven repo with sonatype nexus

PSHOT/juniper-contrail-api-1.0-20131001.003401-3.pom.sha1 .m2/repository/net/juniper/contrail/juniper-contrail-api/1.0-SNAPSHOT/juniper-contrail-api-1.0-20131001.003401-3.jar - Suresh From: Suresh Balineni mailto:sbalin...@juniper.net>> Date: Friday, May 9, 2014 3:43 PM To: Sunil Bakhru mailto:sbak...@j

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

. Thanks, Suresh Balineni

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

Service LB > > with capability LbSchemas="internal" to your contrail provider > > implementation. > > > > The current fix though implies that the separate Internal LB vm will be > > started? Please elaborate. > > Suresh Balineni wrote: > I w

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

Service LB > > with capability LbSchemas="internal" to your contrail provider > > implementation. > > > > The current fix though implies that the separate Internal LB vm will be > > started? Please elaborate. > > Suresh Balineni wrote: > I w

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

------ On March 27, 2014, 12:42 a.m., Suresh Balineni wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18552/ > ---

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

org/r/18552/diff/ Testing --- Tested LB VM deployment. Traffic tests are done. Thanks, Suresh Balineni

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

; > that matter. Adding a provider is much more than just adding the class for > > the element. Its also adding the API logic for handling all the operations. > > Look at api package, commands/internallb folder. Each element is gotta have > > its own set of APIs. > >

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

t, based on the network offering chosen the new Contrail VPC Element can get invoked. - Suresh ------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18552/#review37581 -

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

. Thanks, Suresh Balineni

Review Request 19062: Per contrail implementation, CS ingress policy mapping: rule cidr should be mapped to src net, dst net is 'local' i.e same as 'guest network' on which rule is applied

ork-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/NetworkPolicyModel.java 3092cad Diff: https://reviews.apache.org/r/19062/diff/ Testing --- verified locally. Thanks, Suresh Balineni

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

/ Testing --- Tested LB VM deployment. Traffic tests are done. Thanks, Suresh Balineni

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

/ Testing --- Tested LB VM deployment. Traffic tests are done. Thanks, Suresh Balineni

Re: Review Request 17506: mvn clean - test logs- contrail -cleanps up only target, dist directories; testclient.log file should be generated inside target directory

ly, visit: https://reviews.apache.org/r/17506/#review35951 --- On Jan. 29, 2014, 6:56 p.m., Suresh Balineni wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

--- Tested LB VM deployment. Traffic tests are done. Thanks, Suresh Balineni

Re: Review Request 18552: Internal LB support for Juniper contrail VPC implementation

--- Tested LB VM deployment. Traffic tests are done. Thanks, Suresh Balineni

Review Request 18552: Internal LB support for Juniper contrail VPC implementation

://reviews.apache.org/r/18552/diff/ Testing --- Tested LB VM deployment. Traffic tests are done. Thanks, Suresh Balineni

Re: Event Interceptor fix - contrail plugin : Review Request

Thanks Darren. It has been checked in. Thanks, Suresh From: Darren Shepherd mailto:darren.sheph...@citrix.com>> Date: Monday, February 24, 2014 11:53 AM To: "dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>" mailto:dev@cloudstack.apache.org>>, Su

Re: Event Interceptor fix - contrail plugin : Review Request

Hi Darren, Can you please review and ship these changes (Master branch)? Without this, contrail implementation won't work. Thanks, Suresh From: Suresh Balineni mailto:sbalin...@junipernetworks.onmicrosoft.com>> Date: Monday, February 10, 2014 12:10 PM To: "dev@cloud

Re: Event Interceptor fix - contrail plugin : Review Request

Hi Darren, Can you please review this ? Thanks, Suresh From: Suresh Balineni mailto:sbalin...@juniper.net>> Date: Monday, February 3, 2014 3:24 PM Cc: Darren Shepherd mailto:darren.sheph...@citrix.com>>, Hugo mailto:h...@trippaers.nl>>, Animesh Chaturvedi mailto:animesh.cha

Re: Contrail issues in current master

I fixed it and sent a review request. https://reviews.apache.org/r/17684/ - Suresh On 2/1/14 3:55 AM, "Hugo Trippaers" wrote: >Hey Contrail friends, > >I¹m noticing some issues with the contrail plugin in current master. > >When you startup CloudStack master with a fresh database it will die w

Review Request 17684: Do not initialize contrail plugin component functionality like db sync, network offerring creation when contrail is not installed

/ Testing --- Verified locally. Thanks, Suresh Balineni

Re: Review Request 17348: VPC support for contrail plugin

server/src/com/cloud/configuration/ConfigurationManagerImpl.java 2003716 server/src/com/cloud/network/vpc/VpcManagerImpl.java 51e476f Diff: https://reviews.apache.org/r/17348/diff/ Testing --- This has been verified in 4.3 and porting the same in master branch Thanks, Suresh

Review Request 17506: mvn clean - test logs- contrail -cleanps up only target, dist directories; testclient.log file should be generated inside target directory

locally Thanks, Suresh Balineni

Re: [ACS 43] juniper-contrail RAT failure in logs

Hi Animesh, I fixed this issue. This is the review request url: https://reviews.apache.org/r/17506/ Regards, Suresh From: Animesh Chaturvedi mailto:animesh.chaturv...@citrix.com>> Subject: [ACS 43] juniper-contrail RAT failure in logs Date: January 28, 2014 at 4:40:43 PM PST To: "dev@cloudsta

Review Request 17348: VPC support for contrail plugin

://reviews.apache.org/r/17348/diff/ Testing --- This has been verified in 4.3 and porting the same in master branch Thanks, Suresh Balineni

Review Request 17337: Contrail Event Interceptor fix

c/org/apache/cloudstack/network/contrail/management/EventUtils.java 29b58fb Diff: https://reviews.apache.org/r/17337/diff/ Testing --- Verified locally Thanks, Suresh Balineni

Re: Review Request 16973: Contrail Event Interceptor fix

Hi All, This is contrail event interceptor fix for plugin to receive cloudstack events. This is must required for contrail plugin to work. Please review and shit this in 4.3. Thanks On Jan 21, 2014, at 11:56 AM, "Suresh Balineni" mailto:sbalin...@juniper.net>> wrote: This is

Review Request 16973: Contrail Event Interceptor fix

c/org/apache/cloudstack/network/contrail/management/EventUtils.java dd18ca6 Diff: https://reviews.apache.org/r/16973/diff/ Testing --- Thanks, Suresh Balineni

Re: Review Request 16813: Contrail plugin fixes

/contrail/management/NetworkProviderTest.java 6996a28 Diff: https://reviews.apache.org/r/16813/diff/ Testing --- compiled and verified locally Thanks, Suresh Balineni

Re: Review Request 16813: Contrail plugin fixes

ote: This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16813/ I didn't see the updated patch. Both attached file and diff are old with transactions. - Sheng Yang On January 14th, 2014, 10:56 p.m. UTC, Suresh Balineni wrote: Review request for

Review Request 16876: db sync fix : make sure that db sync task gets executed once: master branch fix

--- Making sure that db sync task gets executed once, a master branch patch Diffs - Diff: https://reviews.apache.org/r/16876/diff/ Testing --- verified locally Thanks, Suresh Balineni

Re: Review Request 16805: Contrail plugin fixes - db update, policy port end fix

locally. Thanks, Suresh Balineni

Re: Review Request 16813: Contrail plugin fixes

master branch - with fix for sheng review comment https://reviews.apache.org/media/uploaded/files/2014/01/14/26c95dad-b064-4e66-8128-08d786858cc6__master.patch Thanks, Suresh Balineni

Re: Review Request 16864: Cloudstack contrail db sync task must be executed at least once even contrail api server is up after cs management server is up

ttps://reviews.apache.org/r/16864/diff/ Testing --- tested locally. Thanks, Suresh Balineni

Review Request 16864: Cloudstack contrail db sync task must be executed at least once even contrail api server is up after cs management server is up

ting --- tested locally. Thanks, Suresh Balineni

Re: Review Request 16805: Contrail plugin fixes - db update, policy port end fix

Hi Sheng, This is a reminder. Can you please ship this? Thanks, Suresh On Jan 11, 2014, at 4:13 PM, "Suresh Balineni" mailto:sbalin...@service.juniper.net>> wrote: Hi Sheng, This is another 4.3 review request. Code changes are minimal. Can you please review and the updat

Re: Review Request 16813: Contrail plugin fixes

Hi Sheng, Please review and ship this master version patch of contrail plugin fixes : https://reviews.apache.org/r/16813/ This patch contains fixes of 4.3 review requests : https://reviews.apache.org/r/16642/ https://reviews.apache.org/r/16805/ Thanks, Suresh From: Suresh Balineni

Review Request 16813: Contrail plugin fixes

Thanks, Suresh Balineni

Review Request 16805: Contrail plugin fixes - db update, policy port end fix

/NetworkPolicyModel.java d5a8722 Diff: https://reviews.apache.org/r/16805/diff/ Testing --- verified locally. Thanks, Suresh Balineni

Re: Review Request 16642: Contrail plguin fixes for CS 4.3

Hi Sheng, Its my mistake. I just updated review with new file fixes.patch. Please find the attached one. This is same. Thanks, Suresh From: Sheng Yang mailto:sh...@yasker.org>> Reply-To: Sheng Yang mailto:sh...@yasker.org>> Date: Friday, January 10, 2014 4:53 PM To: Sur

Re: Review Request 16642: Contrail plguin fixes for CS 4.3

-- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16642/#review31568 --- On Jan. 11, 2014, 12:59 a.m., Suresh Balineni wrote: > > -

Re: Review Request 16642: Contrail plguin fixes for CS 4.3

/ Testing --- verified locally. File Attachments (updated) Good file https://reviews.apache.org/media/uploaded/files/2014/01/11/9ce72fe1-cb37-42c7-838b-395229b3e082__fixes.patch Thanks, Suresh Balineni

Re: Review Request 16642: Contrail plguin fixes for CS 4.3

-4ee3-bf44-68f42dd7d335__fixes.patch Thanks, Suresh Balineni

Re: Review Request 16642: Contrail plguin fixes for CS 4.3

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16642/#review31556 --- On Jan. 11, 2014, 12:44 a.m., Suresh Balineni wrote: > > ---

Re: Review Request 16642: Contrail plguin fixes for CS 4.3

/diff/ Testing --- verified locally. Thanks, Suresh Balineni

Re: Review Request 16642: Contrail plguin fixes for CS 4.3

/diff/ Testing --- verified locally. Thanks, Suresh Balineni

Review Request 16642: Contrail plguin fixes for CS 4.3

. Thanks, Suresh Balineni

Re: Review Request 16274: Contrail VRouter Plugin - VPC Implementation, Bug fixes

. File Attachments Contrail VPC Implementation https://reviews.apache.org/media/uploaded/files/2013/12/15/206339e2-7695-48b9-8549-f9090d5e24c7__juniper-contrail-vpc-implementation.patch Thanks, Suresh Balineni

Re: Review Request 16274: Contrail VRouter Plugin - VPC Implementation, Bug fixes

://reviews.apache.org/media/uploaded/files/2013/12/15/206339e2-7695-48b9-8549-f9090d5e24c7__juniper-contrail-vpc-implementation.patch Thanks, Suresh Balineni

Re: Review Request 16274: Contrail VRouter Plugin - VPC Implementation, Bug fixes

://reviews.apache.org/media/uploaded/files/2013/12/15/206339e2-7695-48b9-8549-f9090d5e24c7__juniper-contrail-vpc-implementation.patch Thanks, Suresh Balineni

Re: Review Request 16276: Contrail Plugin Code fixes - Virtual Network name should have fully qualified name for policy creation.

/16276/diff/ Testing --- Network Policy creation on Contrail router is verified. File Attachments (updated) Contrail plugin fixes https://reviews.apache.org/media/uploaded/files/2013/12/15/4d041bf8-d5c8-42e0-8a51-e0a9a2cb2f58__vn-fqn.patch Thanks, Suresh Balineni

Re: Review Request 16279: Contrail plugin fixes for db updates, adding JuniperContrailVPCRouter as a SDN Provider.

cc1-b232-ee4a6ef2e0e7__vpc-fix.patch Thanks, Suresh Balineni

Review Request 16279: Contrail plugin fixes for db updates, adding JuniperContrailVPCRouter as a SDN Provider.

vider in VpcManagerImpl. Diffs - Diff: https://reviews.apache.org/r/16279/diff/ Testing --- Tests are done. Verified. Thanks, Suresh Balineni

Review Request 16276: Contrail Plugin Code fixes - Virtual Network name should have fully qualified name for policy creation.

Policy creation on Contrail router is verified. Thanks, Suresh Balineni

Review Request 16274: Contrail VRouter Plugin - VPC Implementation, Bug fixes

spring context xml file. Diffs - Diff: https://reviews.apache.org/r/16274/diff/ Testing --- - Contrail plugin code is completely tested. - VPC tests are verified. Thanks, Suresh Balineni