Review Request 14307: CLOUDSTACK-702: New test for deploying vm in new cidr

2013-09-23 Thread sanjeev n
/test_multiple_ip_ranges.py PRE-CREATION Diff: https://reviews.apache.org/r/14307/diff/ Testing --- Yes. Thanks, sanjeev n

Review Request 14468: CLOUDSTACK-702: Added test for verifying dns service on alias IP

2013-10-03 Thread sanjeev n
/integration/component/maint/test_multiple_ip_ranges.py 782957c Diff: https://reviews.apache.org/r/14468/diff/ Testing --- Yes Thanks, sanjeev n

Re: Review Request 14335: CLOUDSTACK-4262: Fix TestVPCNetworkGc.test_01_wait_network_gc

2013-10-04 Thread sanjeev n
. However validation is missing in the code after listing NetworkAcl rules after gc. netacls = NetworkACL.list(self.apiclient, networkid=self.network_1.id) self.debug("List of NetACLS %s" % netacls) Add validation whether NetworkACL.list is returning empty or not. - sanjeev n On Sept. 25,

Re: Review Request 14248: CLOUDSTACK: 4537 - test_vpc_vm_life_cycle.py - Fixed vpc and shared network test cases

2013-10-04 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14248/#review26667 --- Ship it! Looks good to me. - sanjeev n On Oct. 3, 2013, 7:28

Re: Review Request 14335: CLOUDSTACK-4262: Fix TestVPCNetworkGc.test_01_wait_network_gc

2013-10-07 Thread sanjeev n
master. - sanjeev n On Oct. 7, 2013, 4:54 a.m., Girish Shilamkar wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Review Request 14531: CLOUDSTACK-702: Verify Userdata and Password service on alias ip on VR

2013-10-08 Thread sanjeev n
services on alias ip addresses on VR. Diffs - test/integration/component/maint/test_multiple_ip_ranges.py 68b5979 Diff: https://reviews.apache.org/r/14531/diff/ Testing --- yes Thanks, sanjeev n

Re: Review Request 14556: CLOUDSTACK-4766: Add timeout if vm does not reach running state

2013-10-09 Thread sanjeev n
issue? - sanjeev n On Oct. 9, 2013, 9:50 a.m., Girish Shilamkar wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 14556: CLOUDSTACK-4766: Add timeout if vm does not reach running state

2013-10-17 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14556/#review27113 --- Ship it! Patch is good to go. - sanjeev n On Oct. 9, 2013, 9:50

Review Request 14737: CLOUDSTACK-702: IP alias verification tests on VR

2013-10-18 Thread sanjeev n
and stop/start VR Diffs - test/integration/component/maint/test_multiple_ip_ranges.py 3f0035e Diff: https://reviews.apache.org/r/14737/diff/ Testing --- Yes Thanks, sanjeev n

Re: Review Request 14737: CLOUDSTACK-702: IP alias verification tests on VR

2013-10-21 Thread sanjeev n
destroying last vm in the cidr 3.Added tests to verify ip alias verification on VR after reboot and stop/start VR Diffs (updated) - test/integration/component/maint/test_multiple_ip_ranges.py 3f0035e Diff: https://reviews.apache.org/r/14737/diff/ Testing --- Yes Thanks, sanjeev n

Re: Review Request 14319: CLOUDSTACK 2238: Automation - Non Contiguous VLAN Ranges

2013-10-23 Thread sanjeev n
rk, read the vlan info and validate it. This applies to all the tests. test/integration/component/test_non_contiguous_vlan.py <https://reviews.apache.org/r/14319/#comment53232> No validation to check step3 mentioned in the dotted strings in the test. Please add code to validate step3.

Review Request 14924: Test to verify ip alias recreation after destroying and recreating VR

2013-10-25 Thread sanjeev n
://reviews.apache.org/r/14924/diff/ Testing --- Yes. Executed the test on a basic zone setup running with latest 4.2 build. Thanks, sanjeev n

Re: Review Request 14738: Testscript for DeployVM User data enhancement

2013-11-04 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14738/#review28087 --- Ship it! Looks good to me. - sanjeev n On Oct. 31, 2013, 1:11

Review Request: CLOUDSTACK-2543: createipAlias.sh/deleteipAlias.sh won't be copied to XenServer host

2013-05-17 Thread sanjeev n
scripts/vm/hypervisor/xenserver/xenserver60/patch 26205f2 Diff: https://reviews.apache.org/r/11221/diff/ Testing --- Yes. Tested with xenserver6.1 on master branch. Thanks, sanjeev n

Re: Review Request: CLOUDSTACK-2543: createipAlias.sh/deleteipAlias.sh won't be copied to XenServer host

2013-05-19 Thread sanjeev n
/11221/diff/ Testing --- Yes. Tested with xenserver6.1 on master branch. Thanks, sanjeev n

Re: Review Request: CLOUDSTACK-2543: createipAlias.sh/deleteipAlias.sh won't be copied to XenServer host

2013-05-20 Thread sanjeev n
isor/xenserver/xenserver60/patch 26205f2 Diff: https://reviews.apache.org/r/11221/diff/ Testing --- Yes. Tested with xenserver6.1 on master branch. Thanks, sanjeev n

Review Request: Test Snapshot Services

2013-06-16 Thread sanjeev n
che.org/r/11904/diff/ Testing --- Yes Thanks, sanjeev n

Re: Review Request: Test Snapshot Services

2013-06-18 Thread sanjeev n
dingly. - sanjeev --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11904/#review22054 --- On June 17, 2013, 6:37 a.

Review Request: Test Volumes Services

2013-06-20 Thread sanjeev n
Diff: https://reviews.apache.org/r/12021/diff/ Testing --- Thanks, sanjeev n

Review Request 12229: Fixing script issue

2013-07-02 Thread sanjeev n
/ Testing --- Yes, Tested on master branch Thanks, sanjeev n

Review Request 12899: CLOUDSTACK-702: Tests for Multiple IP Ranges

2013-07-24 Thread sanjeev n
: https://reviews.apache.org/r/12899/diff/ Testing --- Yes. Tests on ACS 4.2 branch Thanks, sanjeev n

Review Request 13964: CLOUDSTACK-4203: Adding a test for migrating volumes of stopped vms to test_stopped_vm.py

2013-09-04 Thread sanjeev n
. Bugs: CLOUDSTACK-4203 Repository: cloudstack-git Description --- Adding test for migrating volumes of stopped vm. Diffs - test/integration/component/test_stopped_vm.py 41eeb46 Diff: https://reviews.apache.org/r/13964/diff/ Testing --- Yes Thanks, sanjeev n

Re: Review Request 13964: CLOUDSTACK-4203: Adding a test for migrating volumes of stopped vms to test_stopped_vm.py

2013-09-04 Thread sanjeev n
/r/13964/diff/ Testing --- Yes Thanks, sanjeev n

Review Request 13991: Made misc changes in test_multiple_ip_ranges.py

2013-09-05 Thread sanjeev n
/test_multiple_ip_ranges.py 18409c5 Diff: https://reviews.apache.org/r/13991/diff/ Testing --- Yes. Thanks, sanjeev n

Re: Review Request: Two test cases from snapshots imporevement feature

2013-04-08 Thread sanjeev n
the tests in advanced zone setup. Thanks, sanjeev n

Review Request: Multiple IP Ranges Support in CloudStack

2013-04-25 Thread sanjeev n
--- Adding two test cases(Adding guest CIDR in same and diff subnet) for "Multiple IP Ranges" support in CloudStack. Diffs - test/integration/component/test_multiple_ip_ranges.py PRE-CREATION Diff: https://reviews.apache.org/r/10792/diff/ Testing --- Thanks, sanjeev n

Re: Review Request: Multiple IP Ranges Support in CloudStack

2013-04-26 Thread sanjeev n
-- test/integration/component/test_multiple_ip_ranges.py PRE-CREATION Diff: https://reviews.apache.org/r/10792/diff/ Testing --- Thanks, sanjeev n

Re: Review Request: Multiple IP Ranges Support in CloudStack

2013-04-26 Thread sanjeev n
-- test/integration/component/test_multiple_ip_ranges.py PRE-CREATION Diff: https://reviews.apache.org/r/10792/diff/ Testing --- Thanks, sanjeev n

Re: Review Request: Multiple IP Ranges Support in CloudStack

2013-04-28 Thread sanjeev n
-- test/integration/component/test_multiple_ip_ranges.py PRE-CREATION Diff: https://reviews.apache.org/r/10792/diff/ Testing --- Thanks, sanjeev n

Review Request: CLOUDSTACK-702: Tests for Multiple IP Ranges

2013-04-30 Thread sanjeev n
--- Tested with fixed IP addresses since code has not been implemented(To make sure no syntax errors). Thanks, sanjeev n

Re: Review Request: CLOUDSTACK-702: Tests for Multiple IP Ranges

2013-05-02 Thread sanjeev n
implemented(To make sure no syntax errors). Thanks, sanjeev n

Review Request: CLOUDSTACK-702: Tests for Multiple IP Ranges support

2013-05-06 Thread sanjeev n
/ Testing --- Yes Thanks, sanjeev n

Review Request: CLOUDSTACK-702: Tests for Multiple IP Ranges

2013-05-09 Thread sanjeev n
://reviews.apache.org/r/11026/diff/ Testing --- yes Thanks, sanjeev n

Re: Review Request 19399: CLOUDSTACK-6257: Resolving issue in template creation from user VM volume, added method to check the state of VM before template creation

2014-05-05 Thread sanjeev n
/lib/base.py:22 error: tools/marvin/marvin/lib/base.py: patch does not apply - sanjeev n On April 28, 2014, 10:41 a.m., Ashutosh Kelkar wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 19399: CLOUDSTACK-6257: Resolving issue in template creation from user VM volume, added method to check the state of VM before template creation

2014-05-05 Thread sanjeev n
> On May 5, 2014, 5:53 a.m., ASF Subversion and Git Services wrote: > > Commit d81c5c44f112df8c0e4388e9d88300b24eefe3d0 in cloudstack's branch > > refs/heads/4.4-forward from Girish Shilamkar > > [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=d81c5c4 ] > > > > Revert "CLOUDSTACK-6

Re: Review Request 20891: CLOUDSTACK-6536: Code cleanup - removing unnecessary waits after VM stop operation

2014-05-05 Thread sanjeev n
> On May 5, 2014, 10:20 a.m., Santhosh Edukulla wrote: > > Ship It! I tried to apply the patch on master, however it failed. Please rebase it with master. Following are the erros: [root@cent62-Sanjeev acs-repo]# git apply /root/0002-CLOUDSTACK-6536-Code-cleanup-removing-unnecessary-wa.patch e

Re: Review Request 21070: Fixed few exception cases.

2014-05-05 Thread sanjeev n
. - sanjeev n On May 5, 2014, 6:49 a.m., Santhosh Edukulla wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 21070: Fixed few exception cases.

2014-05-05 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21070/#review42157 --- Ship it! Ship It! - sanjeev n On May 5, 2014, 6:49 a.m

Re: Review Request 21070: Fixed few exception cases.

2014-05-05 Thread sanjeev n
> On May 5, 2014, 4:06 p.m., sanjeev n wrote: > > Ship It! Applied patch to 4.4-automation branch commit ef286da8ccc005fee58c4be5d508c5d7ddfc2a6c Author: sanjeevneelarapu Date: Tue May 6 02:57:08 2014 +0530 Fixed few exception cases

Re: Introducing Vishwas

2017-02-28 Thread Sanjeev N
to contribute. > > > > Welcome Vishwas! > > > > > > Nathan Johnson > > R&D Engineer > > Education Networks of America > -- Best Regards, Sanjeev N Chief Product Engineer@Accelerite

Review Request 23009: Fix for test_portable_ip.py script issues

2014-06-26 Thread sanjeev n
/23009/diff/ Testing --- Yes Thanks, sanjeev n

Review Request 23449: Fix for test_portable_ip.py script issues

2014-07-14 Thread sanjeev n
, sanjeev n

Review Request 23609: Signature changes to delete method in VirtualMachine class in base.py

2014-07-17 Thread sanjeev n
additional parameters. Diffs - tools/marvin/marvin/lib/base.py e9d5bb4 Diff: https://reviews.apache.org/r/23609/diff/ Testing --- yes Thanks, sanjeev n

Review Request 23610: New script for verifying multiple shared networks with overlapped cidrs in different vlans

2014-07-17 Thread sanjeev n
.Create network3 with cidr1 and vlan2- pass Diffs - test/integration/component/test_shared_networks.py f99bfbf Diff: https://reviews.apache.org/r/23610/diff/ Testing --- yes Thanks, sanjeev n

Review Request 23614: New test for deploying vm with static ip address

2014-07-17 Thread sanjeev n
/test_escalations_instances.py 4900ff6 Diff: https://reviews.apache.org/r/23614/diff/ Testing --- yes Thanks, sanjeev n

Re: Review Request 23614: New test for deploying vm with static ip address

2014-07-22 Thread sanjeev n
deployment Diffs - test/integration/component/test_escalations_instances.py 4900ff6 Diff: https://reviews.apache.org/r/23614/diff/ Testing --- yes Thanks, sanjeev n

Re: Review Request 22805: CLOUDSTACK-1466:Adding automation tests for Secondary Storage Limits

2014-07-22 Thread sanjeev n
tps://reviews.apache.org/r/22805/#comment84906> Step6 is missing. Please add test for step6 - sanjeev n On July 22, 2014, 8:39 a.m., Ashutosh Kelkar wrote: > > --- > This is an automatically generated e-mail. To reply,

Review Request 23838: CLOUDSTACK-5986: Test scipt to verify the fix in isolated network for non-vpc networks

2014-07-23 Thread sanjeev n
79fd3b1 Diff: https://reviews.apache.org/r/23838/diff/ Testing --- Yes Thanks, sanjeev n

Review Request 23928: Test script to verify CS-5986 fix for vpc networks

2014-07-25 Thread sanjeev n
- test/integration/component/test_vpc_vms_deployment.py 1c1f93d Diff: https://reviews.apache.org/r/23928/diff/ Testing --- yes Thanks, sanjeev n

Re: Review Request 23976: CLOUDSTACK-7190: Fix Hyper-V issue when attempting SSH to System VMs

2014-07-28 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23976/#review48837 --- Ship it! Ship It! - sanjeev n On July 28, 2014, 10:34 a.m

Re: Review Request 23610: New script for verifying multiple shared networks with overlapped cidrs in different vlans

2014-08-10 Thread sanjeev n
--pass 2.Create network2 with cidr1 and vlan1--fails 3.Create network3 with cidr1 and vlan2- pass Diffs - test/integration/component/test_shared_networks.py f99bfbf Diff: https://reviews.apache.org/r/23610/diff/ Testing --- yes Thanks, sanjeev n

Review Request 24589: Test script to verify fix for CS-5332

2014-08-11 Thread sanjeev n
and use the above offering for VR 3.Implement network and deploy vm in it Diffs - test/integration/component/test_network_offering.py f2251ff8 Diff: https://reviews.apache.org/r/24589/diff/ Testing --- Yes Thanks, sanjeev n

Re: Review Request 23452: CLOUDSTACK-4840: Automation tests - LB for secondary IP

2014-08-12 Thread sanjeev n
tps://reviews.apache.org/r/23452/#comment87988> Add Assertion to check whether assigning lb rule to secondary ip address is success or not. Applicable in all tests - sanjeev n On Aug. 12, 2014, 10:05 a.m., Ashutosh Kelkar wrote: > > --

Re: Review Request 23982: CLOUDSTACK-7192: Skip tests on Hyper-V which don't apply

2014-08-12 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23982/#review50307 --- Ship it! Ship It! - sanjeev n On Aug. 6, 2014, 2:41 p.m., John

Review Request 24600: Test script to verify if there is an error in editing network without network domain specified

2014-08-12 Thread sanjeev n
network and verify it does not error out Diffs - test/integration/component/test_network_offering.py 2c9a057 Diff: https://reviews.apache.org/r/24600/diff/ Testing --- yes Thanks, sanjeev n

Re: Review Request 24055: CLOUDSTACK-2251: Automation test cases for feature - Dedicated guest VLAN ranges per tenant

2014-08-17 Thread sanjeev n
e to handle the error in releasing vlan range. - sanjeev n On July 29, 2014, 2:20 p.m., Ashutosh Kelkar wrote: > > --- > This is an automatically generated e-mail. To reply, visit: >

Re: Review Request 23819: CLOUDSTACK-2251: Automation tests for dedicated public IP addreses per tenant feature

2014-08-18 Thread sanjeev n
ment88742> Add code to verify the ip range release is success or not. - sanjeev n On July 22, 2014, 5:52 p.m., Girish Shilamkar wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://re

Re: Review Request 23610: New script for verifying multiple shared networks with overlapped cidrs in different vlans

2014-08-20 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23610/#review51157 --- Ship it! Ship It! - sanjeev n On Aug. 11, 2014, 5:38 a.m

Re: Review Request 23452: CLOUDSTACK-4840: Automation tests - LB for secondary IP

2014-08-24 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23452/#review51361 --- Ship it! - sanjeev n On Aug. 22, 2014, 6:24 a.m., Ashutosh

Re: Review Request 23452: CLOUDSTACK-4840: Automation tests - LB for secondary IP

2014-08-24 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23452/#review51364 --- Ship it! Ship It! - sanjeev n On Aug. 22, 2014, 6:24 a.m

Review Request 25017: Disabling VPC tests on Hyper-v

2014-08-24 Thread sanjeev n
://reviews.apache.org/r/25017/diff/ Testing --- Yes Thanks, sanjeev n

Re: Review Request 25017: Disabling VPC tests on Hyper-v

2014-08-26 Thread sanjeev n
/integration/smoke/test_vpc_vpn.py 9e8f97f Diff: https://reviews.apache.org/r/25017/diff/ Testing --- Yes Thanks, sanjeev n

Review Request 25097: Test script to verify vm deployment with two networks in SG enabled advanced zone

2014-08-27 Thread sanjeev n
-- Ran 1 test in 357.750s OK Thanks, sanjeev n

Re: Review Request 24055: CLOUDSTACK-2251: Automation test cases for feature - Dedicated guest VLAN ranges per tenant

2014-08-28 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24055/#review51756 --- Ship it! Ship It! - sanjeev n On Aug. 22, 2014, 10:14 a.m

Re: Review Request 22805: CLOUDSTACK-1466:Adding automation tests for Secondary Storage Limits

2014-08-28 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22805/#review51758 --- Ship it! Ship It! - sanjeev n On Aug. 27, 2014, 8:31 a.m

Review Request 25293: Test script to verify vm deployment with two networks in SG enabled advanced zone

2014-09-03 Thread sanjeev n
-- Ran 1 test in 343.198s OK Thanks, sanjeev n

Review Request 25536: Adding new test case to verify the fix for issue "Exception when attaching data disk to Rhel vm on vSphere"

2014-09-11 Thread sanjeev n
Testing --- Yes Thanks, sanjeev n

Review Request 25580: Adding test case to verify fix for issue "Create volume from custom disk offering does not work as expected"

2014-09-12 Thread sanjeev n
as expected ... === TestName: test_13_volume_custom_disk_size | Status : SUCCESS === ok -- Ran 1 test in 303.508s OK Thanks, sanjeev n

Re: Review Request 25681: Fixed the test_templates.py script bug CLOUDSTACK-7554

2014-09-15 Thread sanjeev n
: Chandan Purushothama Date: Mon Sep 15 21:12:26 2014 -0700 CLOUDSTACK-7554 : Fixed the script - /component/test_templates.py - User Account now has permissions to the Template created by Admin - sanjeev n On Sept. 16, 2014, 4:21 a.m., Chandan Purushothama wrote

Review Request 25771: Adding new test case to verify the fix provided in bug CLOUDSTACK-6172

2014-09-18 Thread sanjeev n
retaining same uuid when migrating from one storage to another. ... === TestName: test_01_migrateVolume | Status : SUCCESS === ok -- Ran 1 test in 343.250s OK Thanks, sanjeev n

Review Request 25889: Add more wait time for ssh execute command in case of Hyper-v

2014-09-22 Thread sanjeev n
, sanjeev n

Re: Review Request 25771: Adding new test case to verify the fix provided in bug CLOUDSTACK-6172

2014-09-22 Thread sanjeev n
in 343.250s OK Thanks, sanjeev n

Re: Review Request 25580: Adding test case to verify fix for issue "Create volume from custom disk offering does not work as expected"

2014-09-23 Thread sanjeev n
: sanjeev Date: Tue Sep 23 14:15:39 2014 +0530 Creating custom disk does not work as expected - sanjeev n On Sept. 12, 2014, 1:32 p.m., sanjeev n wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 25536: Adding new test case to verify the fix for issue "Exception when attaching data disk to Rhel vm on vSphere"

2014-09-23 Thread sanjeev n
ments provided in RR 25536 - sanjeev --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25536/#review53939 --- On Sept.

Re: Review Request 25293: Test script to verify vm deployment with two networks in SG enabled advanced zone

2014-09-26 Thread sanjeev n
... === TestName: test_34_DeployVM_in_SecondSGNetwork | Status : SUCCESS === ok -- Ran 1 test in 343.198s OK Thanks, sanjeev n

Re: Review Request 26080: Automation for volume life cycle testPath

2014-09-26 Thread sanjeev n
comment94908> move this line after assertion - sanjeev n On Sept. 26, 2014, 10:58 a.m., prashant mishra wrote: > > --- > This is an automatically generated e-mail. To

Review Request 26488: New Test to verify the fix for issue "The ISO / Template is automatically deleted after download URL expires."

2014-10-09 Thread sanjeev n
-- Ran 1 test in 651.143s OK Thanks, sanjeev n

Review Request 30611: Before throwing exception for iso download try one more time

2015-02-04 Thread sanjeev n
: cloudstack-git Description --- Before throwing exception for iso download try one more time Diffs - tools/marvin/marvin/lib/base.py e38c394 Diff: https://reviews.apache.org/r/30611/diff/ Testing --- Yes Thanks, sanjeev n

Re: Review Request 30611: Before throwing exception for iso download try one more time

2015-02-15 Thread sanjeev n
SrikanteswaraRao Talluri. Repository: cloudstack-git Description --- Before throwing exception for iso download try one more time Diffs - tools/marvin/marvin/lib/base.py e38c394 Diff: https://reviews.apache.org/r/30611/diff/ Testing --- Yes Thanks, sanjeev n

Review Request 31189: skip password assignment from services

2015-02-19 Thread sanjeev n
: cloudstack-git Description --- If self.password is available skip password assignement from test data Diffs - tools/marvin/marvin/lib/base.py 7443373 Diff: https://reviews.apache.org/r/31189/diff/ Testing --- yes Thanks, sanjeev n

Review Request 31476: If password is defined in vm response then skip password assignement

2015-02-25 Thread sanjeev n
: cloudstack-git Description --- If password is defined in vm response then skip password assignement Diffs - tools/marvin/marvin/lib/base.py 7443373 Diff: https://reviews.apache.org/r/31476/diff/ Testing --- Yes Thanks, sanjeev n

Re: [ANNOUNCE] Rohit Yadav as new PMC member of CloudStack

2015-03-27 Thread Sanjeev N
Congrats Rohit!! On Fri, Mar 27, 2015 at 2:06 PM, Jayapal Reddy Uradi < jayapalreddy.ur...@citrix.com> wrote: > Congrats Bhaisaab :) > > On 27-Mar-2015, at 2:01 PM, Funs Kessen wrote: > > > Congrats Rohit, awesome! > > > >> On 27 Mar 2015, at 09:20, Remi Bergsma > wrote: > >> > >> Congratz Rohi

Re: [DISCUSS] Stop using Review Board

2015-03-30 Thread Sanjeev N
+1 On Mon, Mar 30, 2015 at 3:58 PM, Raja Pullela wrote: > +1 for GitHub... > > Some of us are already using GH and any new review requests on RB, > reviewers should ask for the owners to post them to GH. > Otherwise, there is no easy way to stop people from using RB ? > > -Original Message--

Re: Migrating to VPC/Site-to-Site VPN

2015-04-01 Thread Sanjeev N
As Erik mentioned, we can add nic from VPC network to existing vms so the vms will be in both isolated(non-vpc) and vpc networks. On Wed, Apr 1, 2015 at 11:56 AM, Erik Weber wrote: > Haven't tried it myself, but i think you should be able to create the vpc > and change nic on VMs. > > If you wan

Re: Migrating to VPC/Site-to-Site VPN

2015-04-01 Thread Sanjeev N
gt; Thank You, > > Logan Barfield > Tranquil Hosting > > > On Wed, Apr 1, 2015 at 3:20 AM, Sanjeev N wrote: > > As Erik mentioned, we can add nic from VPC network to existing vms so the > > vms will be in both isolated(non-vpc) and vpc networks. > > > > On

Re: [ANNOUNCE] New committer: Gaurav Nandkumar Aradhye

2015-04-13 Thread Sanjeev N
Congrats Gaurav!! On Tue, Apr 14, 2015 at 9:34 AM, Jayapal Reddy Uradi < jayapalreddy.ur...@citrix.com> wrote: > Congrats Gaurav! > > -Jayapal > > On 14-Apr-2015, at 9:10 AM, Rajani Karuturi > wrote: > > > congratulations Gaurav.. > > > > ~Rajani > > > > On Tue, Apr 14, 2015 at 1:35 AM, Erik We

Re: Review Request 29843: CLOUDSTACK-8149: Code-refactor - test_VirtualRouter_alerts.py

2015-01-13 Thread sanjeev n
gt; Read the router.alerts.check.interval value from Global settings and add 10mins instead of hard coding the value in time.sleep() - sanjeev n On Jan. 13, 2015, 7:04 a.m., Ashutosh Kelkar wrote: > > --- > This is an automatically generat

Re: Review Request 29620: CLOUDSTACK-8143: Corrected a test in BVT test_routers.py

2015-01-13 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29620/#review67838 --- Ship it! Ship It! - sanjeev n On Jan. 6, 2015, 7:37 a.m

Re: Review Request 29620: CLOUDSTACK-8143: Corrected a test in BVT test_routers.py

2015-01-13 Thread sanjeev n
> On Jan. 13, 2015, 11:10 a.m., sanjeev n wrote: > > Ship It! commit 9f55411295c9abc843a49f226691a6b720fb0c07 Author: Ashutosh K Date: Tue Jan 6 12:54:49 2015 +0530 CLOUDSTACK-8143: Corrected a test in BVT test_routers.py

Re: Review Request 29620: CLOUDSTACK-8143: Corrected a test in BVT test_routers.py

2015-01-13 Thread sanjeev n
> On Jan. 13, 2015, 11:10 a.m., sanjeev n wrote: > > Ship It! > > sanjeev n wrote: > commit 9f55411295c9abc843a49f226691a6b720fb0c07 > Author: Ashutosh K > Date: Tue Jan 6 12:54:49 2015 +0530 > > CLOUDSTACK-8143: Corrected a test in BVT

Re: Review Request 30157: CLOUDSTACK-8174: Fixed network delete issue in test_add_remove_network.py

2015-01-22 Thread sanjeev n
> On Jan. 22, 2015, 8:49 a.m., sanjeev n wrote: > > Ship It! commit 8894fc897b9e6d20efda6f18f7e9178fa63e87a6 Author: Gaurav Aradhye Date: Wed Jan 21 22:31:24 2015 -0800 CLOUDSTACK-8174: Fixed cleanup issue in test_add_remove_network.py

Re: Review Request 30157: CLOUDSTACK-8174: Fixed network delete issue in test_add_remove_network.py

2015-01-22 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30157/#review69147 --- Ship it! Ship It! - sanjeev n On Jan. 22, 2015, 7:03 a.m

Re: Review Request 30067: CLOUDSTACK-8170: Skipping tests on HyperV which try to scale VM in running state because feature not supported

2015-01-22 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30067/#review69162 --- Ship it! Ship It! - sanjeev n On Jan. 22, 2015, 5:29 a.m

Re: Review Request 30067: CLOUDSTACK-8170: Skipping tests on HyperV which try to scale VM in running state because feature not supported

2015-01-22 Thread sanjeev n
> On Jan. 22, 2015, 11:09 a.m., sanjeev n wrote: > > Ship It! commit d37c1589bea8a02c9dd8c17254b96d9fb01d70b3 Author: Gaurav Aradhye Date: Tue Jan 20 16:18:08 2015 +0530 CLOUDSTACK-8170: Skipping tests on HyperV which try to scale VM in running state because feature not

Re: Review Request 29845: CLOUDSTACK-8152: Adding delay before creating snapshot/template from root disk after writing data to disk - test_vm_passwordenabled.py

2015-01-22 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29845/#review69164 --- Ship it! Ship It! - sanjeev n On Jan. 22, 2015, 5:29 a.m

Re: Review Request 29845: CLOUDSTACK-8152: Adding delay before creating snapshot/template from root disk after writing data to disk - test_vm_passwordenabled.py

2015-01-22 Thread sanjeev n
> On Jan. 22, 2015, 11:22 a.m., sanjeev n wrote: > > Ship It! commit 2db5ee3c0a5d40cdc4d5de60b2f8f3afca32e57f Author: Gaurav Aradhye Date: Tue Jan 13 13:56:41 2015 +0530 CLOUDSTACK-8152: Adding delay before creating snapshot/template from root disk after writing dat

Re: [GitHub] cloudstack pull request: CLOUDSTACK-9121: Adding VmSnapshot valida...

2015-12-30 Thread Sanjeev N
LGTM!! Test revert snapshot on XenServer ... === TestName: test_01_check_revert_snapshot | Status : SUCCESS === ok -- Ran 1 test in 132.503s OK On Wed, Dec 23, 2015 at 5:44 PM, pvr9711 wrote: > Github user pvr9711 comme

Re: [GitHub] cloudstack pull request: CLOUDSTACK-9041: Modifying template creat...

2016-01-05 Thread Sanjeev N
This PR is related to changes in base.py So ran one test which invokes the changes to make sure the changes works fine. Check Snapshots size in database ... === TestName: test_02_snapshot_size_check | Status : SUCCESS === ok -- R

Re: [GitHub] cloudstack pull request: Test to create vpn customer gateway with ...

2016-01-05 Thread Sanjeev N
Makes sense. I will add code to list vpn customer gateways and will check if the response contains the customer gateway or not. On Tue, Jan 5, 2016 at 3:55 PM, DaanHoogland wrote: > Github user DaanHoogland commented on a diff in the pull request: > > https://github.com/apache/cloudstack/pul

Re: [GitHub] cloudstack pull request: [CLOUDSTACK-9215]Test to verify vm deploy...

2016-01-08 Thread Sanjeev N
Will add some validations On Fri, Jan 8, 2016 at 7:51 PM, DaanHoogland wrote: > Github user DaanHoogland commented on the pull request: > > https://github.com/apache/cloudstack/pull/1316#issuecomment-170016433 > > test succeeds as is : > ``` > [root@daans-dev-machinina ~]# ssh cs

  1   2   >