Diff: https://reviews.apache.org/r/17307/diff/
Testing
---
Passes unit tests.
Thanks,
Sachchidanand Vaidya
---
All Juniper contrail plugin unit tests pass without any error.
Thanks,
Sachchidanand Vaidya
://reviews.apache.org/r/16523/diff/
Testing
---
All Juniper contrail plugin unit tests pass without any error.
Thanks,
Sachchidanand Vaidya
Hi,
Under "Networks" Tab in UI for Admin user, Cloudstack provides 2 options to
create networks
1) Add guest network 2) Add Isolated Network.
- What is the difference between these 2 networks.
Also, I created and Enabled a new "Network Offering". But when I create an
Isolated Ne
p, which vm's traffic go through
>>>switch
>>> directly.
>>> Isolated Network is a virtual network, all traffic go through VRouter.
>>> And if you assigned tag on Physical Network, if so you should set same
>>>tag
>>> on n/w offering.
>
/cloudstack/network/contrail/model/ServiceInstanceModel.java
e79053c
Diff: https://reviews.apache.org/r/19270/diff/
Testing
---
Unit tests passed and additional testing done locally.
Thanks,
Sachchidanand Vaidya
al testbed.
Thanks,
Sachchidanand Vaidya
Hi All,
I'm trying to run devCloud2 with CS Management server running master code.
Setup is: CS Mgmt Server: 192.168.56.1, XenHost ova:192.168.56.10
-> While trying to setup Basic networking using (devcloud.cfg), got "Cannot
create directory /opt/cloud/bin on XenServer hosts"
error during
quot; providers.
Diffs
-
tools/devcloud/devcloud-advanced_juniper-contrail.cfg PRE-CREATION
tools/marvin/marvin/deployDataCenter.py c4f6e1e
Diff: https://reviews.apache.org/r/19892/diff/
Testing
---
Unit tested with devCloud2 and ACS4.3 code base.
Thanks,
Sachchidanand Vaidya
diff/
Testing
---
Unit tested for null and empty "name" parameter.
Thanks,
Sachchidanand Vaidya
Hi,
Can someone pls review these changes.
Thanks & Regards,
Sachin
On 4/2/14 2:14 AM, "Sachchidanand Vaidya" wrote:
>
>---
>This is an automatically generated e-mail. To reply, visit:
>https://
Hi,
Can someone pls review these changes.
Thanks & Regards,
Sachin
From: Sachchidanand Vaidya mailto:vaidy...@juniper.net>>
Reply-To: Sachchidanand Vaidya
mailto:vaidy...@juniper.net>>
Date: Tuesday, April 1, 2014 3:05 PM
To: Sachchidanand Vaidya mailto:vaidy...@juniper.ne
Hi,
Can someone pls. review these changes.
Thanks & Regards,
Sachin
From: Sachchidanand Vaidya mailto:vaidy...@juniper.net>>
Reply-To: Sachchidanand Vaidya
mailto:vaidy...@juniper.net>>
Date: Thursday, March 27, 2014 4:38 PM
To: Sachchidanand Vaidya mailto:vaid
/r/19892/diff/
Testing
---
Unit tested with devCloud2 and ACS4.3 code base.
Thanks,
Sachchidanand Vaidya
a good idea from
usability/debugability point of view.
Thanks & Regards,
Sachin
- Sachchidanand
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19912/#review40393
------
an.hoogl...@gmail.com>>,
Sachchidanand Vaidya mailto:vaidy...@juniper.net>>,
cloudstack mailto:dev@cloudstack.apache.org>>
Subject: Re: Review Request 19912: [CLOUDSTACK-6322] Fixed Contrail: Params
validation is missing while launching a service instance thru cloudmonkey
This is an a
.
Thanks,
Sachchidanand Vaidya
rvin/deployDataCenter.py 31f663e
Diff: https://reviews.apache.org/r/19892/diff/
Testing
---
Unit tested with devCloud2 and ACS4.3 code base.
Thanks,
Sachchidanand Vaidya
Thanks Daan. Can it be committed to 4.4 as well.
Best Regards,
Sachin
From: daan Hoogland mailto:daan.hoogl...@gmail.com>>
Reply-To: daan Hoogland
mailto:daan.hoogl...@gmail.com>>
Date: Thursday, May 1, 2014 4:11 AM
To: daan Hoogland mailto:daan.hoogl...@gmail.com>>,
Sa
Testing
---
Unit tested with devCloud2 and ACS4.3 code base.
Thanks,
Sachchidanand Vaidya
Hi Daan,
I uploaded a diff
(ACS4.4-0004-Add-OpenContrail-support-to-devCloud-environment.patch) based on
ACS4.4 code base.
Thanks,
Sachin
From: Daan Hoogland mailto:daan.hoogl...@gmail.com>>
Date: Friday, May 2, 2014 11:03 AM
To: Sachchidanand Vaidya mailto:vaidy...@juniper.ne
f34eacc
plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/ServiceInstanceModel.java
e79053c
Diff: https://reviews.apache.org/r/19270/diff/
Testing
---
Unit tests passed and additional testing done locally.
Thanks,
Sachchidanand Vaidya
ts with vSRX network service provider. Also other unit tests
pass in local testbed.
Thanks,
Sachchidanand Vaidya
ly, visit:
https://reviews.apache.org/r/19270/#review42254
---
On May 7, 2014, 3:44 p.m., Sachchidanand Vaidya wrote:
>
> ---
> This is an automatically generated e-mail.
instance VM creation and destroy.
- Sachchidanand
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19270/#review42253
-------
On May 7, 2014, 3:44 p.m., Sachchidanand Vaidya wrote:
>
> ---
Hi,
I have an isolated network (192.168.3.x/24) being served by VirtualRouter,
where 10.84.59.131 is SourceNAT address
and eth0 is VN interface of VirtualRouter & eth2 is the public interface of
VirtualRouter.
When I look at the nat table entries on the VirtualRouter, it shows following :
9060f1
Diff: https://reviews.apache.org/r/19758/diff/
Testing
---
Performed unit tests with vSRX network service provider. Also other unit tests
pass in local testbed.
Thanks,
Sachchidanand Vaidya
rg/r/19758/diff/1/?file=539139#file539139line206>(Diff
revision 1)
206
@Override
missed formatting
[SACHIN] Will take care.
- Rajesh Battala
On May 3rd, 2014, 7:30 a.m. UTC, Sachchidanand Vaidya wrote:
Review request for cloudstack and Rajesh Battala.
By Sachchidanand Vaidya.
ck/network/contrail/management/ContrailVSrxElementImpl.java,
> > line 206
> > <https://reviews.apache.org/r/19758/diff/1/?file=539139#file539139line206>
> >
> > missed formatting
fixed.
- Sachchidanand
--
/ServiceManagerImpl.java
f34eacc
plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/ServiceInstanceModel.java
e79053c
Diff: https://reviews.apache.org/r/19270/diff/
Testing
---
Unit tests passed and additional testing done locally.
Thanks,
Sachchidanand Vaidya
Hi Rajesh,
Can you pls review the updated diff.
Thanks,
Sachin
On 5/7/14 3:44 PM, "Sachchidanand Vaidya" wrote:
>
>---
>This is an automatically generated e-mail. To reply, visit:
>https://revi
Hi,
I'm seeing the same issue with 4.4 code. After further debug, I see
that CS mgmt server is sending
following command to XenHost,
xensource.log: /opt/cloud/bin/ipassoc.sh -A -s -f -l
10.84.59.131/24 -c eth0 -g 10.84.59.254
VirtualRouter's public interface is eth2. Also as per dump of VIF l
ailto:dev@cloudstack.apache.org>>"
mailto:dev@cloudstack.apache.org>>, Sachchidanand
Vaidya mailto:vaidy...@juniper.net>>
Subject: Re: SNAT iptable entry on VirtualRouter
Hi Sachin,
The nicDevId() you see is coming from
prepareNetworkElementCommand(IpAssocCommand cmd)
work-elements/juniper-contrail/test/org/apache/cloudstack/network/contrail/management/VirtualNetworkModelTest.java
0938541
Diff: https://reviews.apache.org/r/15825/diff/
Testing
---
Juniper contrail Unit Tests ran successfully as part of package build.
Thanks,
Sachchidanand Vaidya
chidanand
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15825/#review29367
---
On Nov. 24, 2013, 6:59 p.m., Sachchidanand Vai
iper contrail Unit Tests ran successfully as part of package build.
Thanks,
Sachchidanand Vaidya
---
Changes pass unit tests and complete centos63 package build.
Thanks,
Sachchidanand Vaidya
g/r/15825/#review30945
>---
>
>Ship it!
>
>
>commit 7f9683af859224b4aa0cae037acda063a9d1536e
>Author: Sachchidanand Vaidya
>Date: Tue Dec 10 09:02:25 2013 -0800
>
>Added Virtual-Network and Virtual-Machine Model unit test.
>
>Signed-off
/apache/cloudstack/network/contrail/management/VMInterfaceModelTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/16523/diff/
Testing
---
All Juniper contrail plugin unit tests pass without any error.
Thanks,
Sachchidanand Vaidya
Hi,
Bug CS-5502 was fixed (12/20/13) and then reopened (12/26/13) again since
for
Vlan=UNTAGGED case, createVlanIpRange() API fails with "Vlan id is required
when add ip
range to the public network". Vlan ="untagged" can be a valid case for public
networks
(in advanced networking) as is t
ui/scripts/ui-custom/zoneWizard.js 28df193
Diff: https://reviews.apache.org/r/16258/diff/
Testing
---
Changes pass unit tests and complete centos63 package build.
Thanks,
Sachchidanand Vaidya
/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
/ManagementServerMock.java
806dd21
ui/scripts/ui-custom/zoneWizard.js 99aa3b9
Diff: https://reviews.apache.org/r/16258/diff/
Testing
---
Changes pass unit tests and complete centos63 package build.
Thanks,
Sachchidanand Vaidya
To reply, visit:
>https://reviews.apache.org/r/16258/#review31802
>---
>
>
>Looks good to me, but cannot applied to the latest MASTER branch(I
>suppose that's the target).
>
>- Sheng Yang
>
>
>
> 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 gene
g/r/16258/#review31812
>---
>
>Ship it!
>
>
>Applied to MASTER.
>
>Thanks.
>
>- Sheng Yang
>
>
>On Jan. 14, 2014, 9:44 p.m., Sachchidanand Vaidya wrote:
>>
>> ---
>> This is an
Diff: https://reviews.apache.org/r/16974/diff/
Testing
---
Locally tested on centOS6.3 machine.
Thanks,
Sachchidanand Vaidya
-mail. To reply, visit:
https://reviews.apache.org/r/16974/
Are these fixes release blockers or can they go into 4.4? The release 4.3 is
effectively closed by now.
Cheers,
Hugo
- Hugo Trippaers
On January 16th, 2014, 6:53 p.m. UTC, Sachchidanand Vaidya wrote:
Review request for cloudstack.
By Sachchid
48 matches
Mail list logo