Re: Cloudstack maven repo with sonatype nexus

2014-05-11 Thread Suresh Balineni
.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 sbalin...@juniper.netmailto:sbalin...@juniper.net Date: Friday, May 9, 2014 3:43 PM To: Sunil Bakhru sbak...@juniper.netmailto:sbak

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

2014-04-04 Thread Suresh Balineni
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 wanted to re-use the existing Internal LB element as is except required Nic resources should

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

2014-04-04 Thread Suresh Balineni
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 wanted to re-use the existing Internal LB element as is except required Nic resources should

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

2014-04-04 Thread Suresh Balineni
. Thanks, Suresh Balineni

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

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

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

2014-03-26 Thread Suresh Balineni
/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

2014-03-24 Thread Suresh Balineni
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. Suresh Balineni wrote: Hi Alena, 1) Why you've decided to introduce a new provider for the Internal

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

2014-03-23 Thread Suresh Balineni
--- On March 12, 2014, 6:19 p.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

2014-03-12 Thread Suresh Balineni
. 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

2014-03-11 Thread Suresh Balineni
-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

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

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

2014-03-04 Thread Suresh Balineni
/ 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

2014-03-03 Thread Suresh Balineni
://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 reply, visit: https

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

2014-03-01 Thread Suresh Balineni
--- Tested LB VM deployment. Traffic tests are done. Thanks, Suresh Balineni

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

2014-02-26 Thread Suresh Balineni
://reviews.apache.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

2014-02-26 Thread Suresh Balineni
--- Tested LB VM deployment. Traffic tests are done. Thanks, Suresh Balineni

Re: Event Interceptor fix - contrail plugin : Review Request

2014-02-24 Thread Suresh Balineni
Hi Darren, Can you please review and ship these changes (Master branch)? Without this, contrail implementation won't work. Thanks, Suresh From: Suresh Balineni sbalin...@junipernetworks.onmicrosoft.commailto:sbalin...@junipernetworks.onmicrosoft.com Date: Monday, February 10, 2014 12:10 PM

Re: Event Interceptor fix - contrail plugin : Review Request

2014-02-24 Thread Suresh Balineni
@cloudstack.apache.org, Suresh Balineni sbalin...@juniper.netmailto:sbalin...@juniper.net Cc: Sunil Bakhru sbak...@juniper.netmailto:sbak...@juniper.net, Sheng Yang sh...@yasker.orgmailto:sh...@yasker.org, Hugo h...@trippaers.nlmailto:h...@trippaers.nl, Animesh Chaturvedi animesh.chaturv

Re: Event Interceptor fix - contrail plugin : Review Request

2014-02-10 Thread Suresh Balineni
Hi Darren, Can you please review this ? Thanks, Suresh From: Suresh Balineni sbalin...@juniper.netmailto:sbalin...@juniper.net Date: Monday, February 3, 2014 3:24 PM Cc: Darren Shepherd darren.sheph...@citrix.commailto:darren.sheph...@citrix.com, Hugo h...@trippaers.nlmailto:h...@trippaers.nl

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

2014-02-03 Thread Suresh Balineni
/ Testing --- Verified locally. Thanks, Suresh Balineni

Re: Contrail issues in current master

2014-02-03 Thread Suresh Balineni
I fixed it and sent a review request. https://reviews.apache.org/r/17684/ - Suresh On 2/1/14 3:55 AM, Hugo Trippaers h...@trippaers.nl 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

Re: Review Request 17348: VPC support for contrail plugin

2014-01-30 Thread Suresh Balineni
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

2014-01-29 Thread Suresh Balineni
locally Thanks, Suresh Balineni

Review Request 17337: Contrail Event Interceptor fix

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

Review Request 17348: VPC support for contrail plugin

2014-01-24 Thread Suresh Balineni
://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 16973: Contrail Event Interceptor fix

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

Re: Review Request 16973: Contrail Event Interceptor fix

2014-01-21 Thread Suresh Balineni
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 sbalin...@juniper.netmailto:sbalin...@juniper.net wrote

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

2014-01-14 Thread Suresh Balineni
--- tested locally. Thanks, Suresh Balineni

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

2014-01-14 Thread Suresh Balineni
locally. Thanks, Suresh Balineni

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

2014-01-14 Thread Suresh Balineni
--- 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 16813: Contrail plugin fixes

2014-01-14 Thread Suresh Balineni
: 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 cloudstack

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

2014-01-11 Thread Suresh Balineni
/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

2014-01-10 Thread Suresh Balineni
/diff/ Testing --- verified locally. Thanks, Suresh Balineni

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

2014-01-10 Thread Suresh Balineni
--- 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: --- This is an automatically generated

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

2014-01-10 Thread Suresh Balineni
-68f42dd7d335__fixes.patch Thanks, Suresh Balineni

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

2014-01-10 Thread Suresh Balineni
/ 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

2014-01-10 Thread Suresh Balineni
To: Suresh Balineni sbalin...@service.juniper.netmailto:sbalin...@service.juniper.net, cloudstack dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org, Sheng Yang sh...@yasker.orgmailto:sh...@yasker.org Subject: Re: Review Request 16642: Contrail plguin fixes for CS 4.3

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

2014-01-07 Thread Suresh Balineni
/diff/ Testing --- verified locally. Thanks, Suresh Balineni

Review Request 16642: Contrail plguin fixes for CS 4.3

2014-01-04 Thread Suresh Balineni
. Thanks, Suresh Balineni

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

2013-12-30 Thread Suresh Balineni
://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

2013-12-30 Thread Suresh Balineni
. 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

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

2013-12-15 Thread Suresh Balineni
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

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

2013-12-15 Thread Suresh Balineni
Policy creation on Contrail router is verified. Thanks, Suresh Balineni

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

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

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

2013-12-15 Thread Suresh Balineni
-ee4a6ef2e0e7__vpc-fix.patch Thanks, Suresh Balineni

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

2013-12-15 Thread Suresh Balineni
/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 16274: Contrail VRouter Plugin - VPC Implementation, Bug fixes

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