Re: Introducing Vishwas

2017-02-28 Thread Sanjeev N
telecom messaging domains. Excited to be part of this > >> community and happy to contribute. > > > > Welcome Vishwas! > > > > > > Nathan Johnson > > R Engineer > > Education Networks of America > -- Best Regards, Sanjeev N Chief Product Engineer@Accelerite

Re: Important Pending Items

2016-03-01 Thread Sanjeev N
Hi Steve, Currently travis runs all simulator tests(marvin tests which does not require any hardware, tagged with requierd_hardware=false) on every PR. If any test fails it updates the PR and marks it red. IMO apart from 2 LGTMs travis test pass is also mandatory for PR merge. Thanks, Sanjeev N

Re: [GitHub] cloudstack pull request: CLOUDSTACK-9066: Update testpath to delet...

2016-02-25 Thread Sanjeev N
Test Results: snapshot hardning ... === TestName: test_01_snapshot_hardning_kvm | Status : : SUCCESS === ok -- Ran 1 test in 1477.688s OK LGTM!! On Wed, Feb 17, 2016 at 3:02 PM, pritisarap12 wrote: >

Re: [GitHub] cloudstack pull request: CLOUDSTACK-9140: Testcase to verify if De...

2016-02-23 Thread Sanjeev N
LGTM !! On Wed, Feb 17, 2016 at 12:46 PM, pritisarap12 wrote: > Github user pritisarap12 commented on the pull request: > > https://github.com/apache/cloudstack/pull/1218#issuecomment-185067571 > > Testcase result after modifications: > > Dedicated cluster and

Re: VM state = Expunging

2016-02-22 Thread Sanjeev N
Expunging is the final state of the vm. We don't have expunged state. On Mon, Feb 22, 2016 at 12:51 PM, Daan Hoogland wrote: > I ran into that as well, a while ago. IMNSHO the VM record should be > deleted after the state expunging, so a state of expunged would never

Re: template available on http://dl.openvm.eu/cloudstack/macchinina/x86_64/

2016-02-18 Thread Sanjeev N
Hi Nux, Can you please convert this template to .ova format , so that tests can run on vmware. http://dl.openvm.eu/cloudstack/macchinina/x86_64/macchinina-vmware.vmdk.bz2 Thanks, Sanjeev On Mon, Jan 4, 2016 at 5:42 PM, Raja Pullela wrote: > Nux, I will provide you

Re: [GitHub] cloudstack pull request: Squashing two commits in to single commit

2016-02-15 Thread Sanjeev N
LGTM based on PR#667 On Mon, Feb 15, 2016 at 5:37 PM, DaanHoogland wrote: > Github user DaanHoogland commented on the pull request: > > https://github.com/apache/cloudstack/pull/1411#issuecomment-184182132 > > @sanjeevneelarapu If you LGTM this there are 2 LGTM here

Re: [GitHub] cloudstack pull request: CLOUDSTACK-9012 :automation of cores feat...

2016-02-09 Thread Sanjeev N
LGTM !! test1_coreos_VM_creation (integration.component.test_coreos.TestDeployVmWithCoreosTemplate) ... === TestName: test1_coreos_VM_creation | Status : SUCCESS === ok -- Ran 1 test in 754.205s OK On Thu, Feb 4, 2016 at 4:29

Re: Advice needed - switching IP for SS

2016-02-03 Thread Sanjeev N
Hi Daniel, It is possible to have the MAC-IP pair in the switch arp cache. You can use arping from your new NFS server to send gratious arp request, so that the switch can update the MAC address in its arp cache. Thanks, Sanjeev On Thu, Feb 4, 2016 at 1:47 PM, Daniel Mezentsev

Re: [ANNOUNCE] New PMC member: Boris Roman Schrijver

2016-01-19 Thread Sanjeev N
Congrats Boris !! On Wed, Jan 20, 2016 at 12:10 AM, ilya wrote: > Warm welcome Boris.. > > On 1/19/16 4:15 AM, Wilder Rodrigues wrote: > > The Project Management Committee (PMC) for Apache CloudStack is pleased > to > > announce that Boris Roman Schrijver has

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

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

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

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

Re: [GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-10 Thread Sanjeev N
@wilder, I am ok with it. You can take it over. One comment, is ostype "Other PV (64-bit)" valid for other hypervisors otherthan Xenserver? And vmware template format should be in .ova The one you have mentioned in the Services class won't work. On Thu, Dec 10, 2015 at 2:03 PM, wilderrodrigues

Re: [GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-10 Thread Sanjeev N
@Wilder, forgot to mention, either we should also add template for LXC or skip the test for LXC. On Thu, Dec 10, 2015 at 2:40 PM, Sanjeev N <sanj...@apache.org> wrote: > @wilder, I am ok with it. You can take it over. > One comment, is ostype "Other PV (64-bit)" valid

Re: [GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-08 Thread Sanjeev N
@terbolour, as per the previous logic in the test, it will register xenserver template for any hypervisor other than kvm and vm deployment fails for other hypervisors like vmware,hyper-v and lxc. So I was asking how we should handle this. On Tue, Dec 8, 2015 at 12:44 PM, terbolous

Re: [GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-08 Thread Sanjeev N
Thanks @michaelandersen, I will take those templates and make the test generic. On Tue, Dec 8, 2015 at 3:35 PM, michaelandersen wrote: > Github user michaelandersen commented on the pull request: > > https://github.com/apache/cloudstack/pull/1156#issuecomment-162837936

Re: [GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

2015-12-07 Thread Sanjeev N
@michaelandersen, if we want to use that template it only works for kvm. So either we have to skip the tests for other hypervisors or make the similar templates for all other hypervisors as well. On Tue, Dec 8, 2015 at 2:00 AM, michaelandersen wrote: > Github user

Re: [GitHub] cloudstack pull request: Add marvin test to verify if DomainAdmin ...

2015-12-02 Thread Sanjeev N
@daanhoogland I have updated the validation steps. On Wed, Dec 2, 2015 at 2:46 PM, DaanHoogland wrote: > Github user DaanHoogland commented on a diff in the pull request: > > https://github.com/apache/cloudstack/pull/1122#discussion_r46391214 > > --- Diff:

Re: marvin updates

2015-12-01 Thread Sanjeev N
Please let me know if there are any issues with Marvin? I can look into it. On Tue, Dec 1, 2015 at 5:58 PM, Daan Hoogland wrote: > thanks Abhi, > > On Tue, Dec 1, 2015 at 1:14 PM, Abhinandan Prateek < > abhinandan.prat...@shapeblue.com> wrote: > > > AFAIK, Santhosh and

Re: [4.6] VPN works?

2015-11-08 Thread Sanjeev N
Nux, When you say SNAT is it SourceNAT IP address or you have acquired IP address and trying VPN on it? If later is the case then it won't work. -Sanjeev On Fri, Nov 6, 2015 at 10:41 AM, Jayapal Reddy Uradi < jayapalreddy.ur...@citrix.com> wrote: > When I test last time remote access vpn and

Re: CloudStack-8933 changes regressed HyperV and VMware BVTs - 8 tests in test_ssvm.py

2015-11-03 Thread Sanjeev N
In case of VMWare and Hyper-v , linc local is on eth1. So the command in all the failed tests to verify link local IP should be modified. "cat /var/cache/cloud/cmdline | xargs | sed \"s/ /\\n/g\" | grep eth0ip= | sed \"s/\=/ /g\" | awk '{print $2}'",. It is using eth0ip. However, it should be

Re: test_vpc_vpn.py failing 2 BVTs on all Advnetworking config

2015-11-03 Thread Sanjeev N
Regards, > Remi > > > > > > On 02/11/15 13:29, "Sanjeev N" <sanj...@apache.org> wrote: > > >The reason for these two failures is VPC creation itself is failing. > Reason > >being "SSH access to VR on 3922 port using its management ip addre

Re: [GitHub] cloudstack pull request: CLOUDSTACK-9001: Modifying snapshot resul...

2015-11-02 Thread Sanjeev N
This is very small change related to list validation and it looks good. LGTM!! On Wed, Oct 28, 2015 at 11:28 AM, bhaisaab wrote: > Github user bhaisaab commented on the pull request: > > https://github.com/apache/cloudstack/pull/994#issuecomment-151733973 > >

Re: [GitHub] cloudstack pull request: CLOUDSTACK-9005: Modifying tearDown funct...

2015-11-02 Thread Sanjeev N
I looked at the changes Priti has done in this PR. It looks good to me. LGTM!! On Thu, Oct 29, 2015 at 11:00 AM, pritisarap12 wrote: > GitHub user pritisarap12 opened a pull request: > > https://github.com/apache/cloudstack/pull/1000 > > CLOUDSTACK-9005: Modifying

Re: test_vpc_vpn.py failing 2 BVTs on all Advnetworking config

2015-11-02 Thread Sanjeev N
The reason for these two failures is VPC creation itself is failing. Reason being "SSH access to VR on 3922 port using its management ip address" is failing. Hence failure in VR start. This looks like product bug to me. -Sanjeev On Fri, Oct 30, 2015 at 10:36 PM, Raja Pullela

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8924: Enable dynamic scaling t...

2015-10-25 Thread Sanjeev N
I will update the PR with required changes suggested by @pvr9711. On Sat, Oct 24, 2015 at 11:36 PM, remibergsma wrote: > Github user remibergsma commented on the pull request: > > https://github.com/apache/cloudstack/pull/902#issuecomment-150838537 > > @sanju1010

Re: [GitHub] cloudstack pull request: Add all tests in /test/integration/smoke ...

2015-10-14 Thread Sanjeev N
@kishankavala, Big no , because simulator regression suite takes almost 6 hours to complete. On Wed, Oct 14, 2015 at 11:24 AM, kishankavala wrote: > Github user kishankavala commented on the pull request: > >

Re: [GitHub] cloudstack pull request: Add all tests in /test/integration/smoke ...

2015-10-13 Thread Sanjeev N
Hi, travis/script.sh uses following nose command to pick the tests. nosetests --with-xunit --xunit-file=integration-test-results/$suite.xml --with-marvin --marvin-config=setup/dev/advanced.cfg test/integration/$suite.py -s -a tags=advanced,required_hardware=false --zone=Sandbox-simulator

Re: Blameless post mortem

2015-09-27 Thread Sanjeev N
I have a concern here. Some of us are actively involved in reviewing the PRs related to marvin tests(Enhancing existing tests/Adding new tests). If we have to test a PR it requires an environment to be created with actual resources and this is going to take lot of time. Some of the tests can run

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8893: Fixing script as per the...

2015-09-23 Thread Sanjeev N
@pavanb018 good catch !! It is not part of the code but will change. On Wed, Sep 23, 2015 at 10:52 AM, pavanb018 wrote: > Github user pavanb018 commented on the pull request: > > https://github.com/apache/cloudstack/pull/871#issuecomment-142497091 > > @Desc: Test

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8726 : Automation for Quickly ...

2015-09-23 Thread Sanjeev N
LGTM!! On Wed, Sep 23, 2015 at 2:15 PM, pavanb018 wrote: > Github user pavanb018 commented on the pull request: > > https://github.com/apache/cloudstack/pull/683#issuecomment-142531937 > > The result of operation is captured in the list volume validation > following

Re: Proposal to get to 100% passrate on BVTs

2015-09-21 Thread Sanjeev N
I would be happy to lend my hand in generalizing the BVT Infra. -Sanjeev On Mon, Sep 21, 2015 at 4:34 PM, Raja Pullela wrote: > Rohit, not sure about the efforts required to do this. Can certainly work > with you or someone to do this. Please let me know, > > best, >

Re: [GitHub] cloudstack pull request: verify that when createTemplate is ued to...

2015-08-26 Thread Sanjeev N
LGTM!! On Wed, Aug 26, 2015 at 3:01 PM, bhaisaab g...@git.apache.org wrote: Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/688#issuecomment-134918074 LGTM --- If your project is set up for it, you can reply to this email and have

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8723: Verify API call listUsa...

2015-08-13 Thread Sanjeev N
LGTM!! On Tue, Aug 11, 2015 at 4:56 PM, pritisarap12 g...@git.apache.org wrote: Github user pritisarap12 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/675#discussion_r36734678 --- Diff: test/integration/testpaths/testpath_usage.py --- @@

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8693-Adding-missing-code-in-te...

2015-08-10 Thread Sanjeev N
LGTM!! On Mon, Aug 10, 2015 at 11:34 AM, pritisarap12 g...@git.apache.org wrote: GitHub user pritisarap12 opened a pull request: https://github.com/apache/cloudstack/pull/668 CLOUDSTACK-8693-Adding-missing-code-in-testpath_same_vm_name.py_testpath -Adding cls.hypervisor =

Re: [Blocker] Default ip table rules on VR

2015-08-03 Thread Sanjeev N
iptables configuration with the old iptables-vpcrouter/iptables-router files. Cheers, Wilder On 31 Jul 2015, at 06:03, Sanjeev N sanj...@apache.org wrote: Thanks for working on it Wilder !! On Thu, Jul 30, 2015 at 6:05 PM, Wilder Rodrigues wrodrig...@schubergphilis.com wrote: Hi

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8686:Verify data disk attachme...

2015-08-03 Thread Sanjeev N
LGTM!! On Mon, Aug 3, 2015 at 11:46 AM, pritisarap12 g...@git.apache.org wrote: Github user pritisarap12 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/631#discussion_r36058447 --- Diff: test/integration/testpaths/testpath_attach_disk_zwps.py ---

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8689: Verify effect of changin...

2015-07-31 Thread Sanjeev N
LGTM On Fri, Jul 31, 2015 at 4:42 PM, nitt10prashant g...@git.apache.org wrote: Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/638#issuecomment-126656909 LGTM --- If your project is set up for it, you can reply to this email

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8689: Verify effect of changin...

2015-07-31 Thread Sanjeev N
How your logic works if there are no system vms on a host (ssvm, cpvm and VR)? I mean if the cluster has more than one host and system vms were deployed on another host in the cluster? On Fri, Jul 31, 2015 at 2:01 PM, pritisarap12 g...@git.apache.org wrote: Github user pritisarap12 commented on

Re: [Blocker] Default ip table rules on VR

2015-07-30 Thread Sanjeev N
:28 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: I changed it to critical. It is only a blocker if we agree on this list that it is. On Thu, Jul 30, 2015 at 6:44 AM, Sanjeev N sanj...@apache.org wrote: Hi, In latest ACS builds, the ip table rules in VR have ACCEPT

Re: [Blocker] Default ip table rules on VR

2015-07-30 Thread Sanjeev N
Thanks for working on it Wilder !! On Thu, Jul 30, 2015 at 6:05 PM, Wilder Rodrigues wrodrig...@schubergphilis.com wrote: Hi, We discussed that one yesterday and I already assigned the issue to myself on Jira. I will fix it. Cheers, WIlder On 30 Jul 2015, at 14:09, Sanjeev N sanj

[Blocker] Default ip table rules on VR

2015-07-29 Thread Sanjeev N
Hi, In latest ACS builds, the ip table rules in VR have ACCEPT as the default policy in INPUT and FORWARD chains, instead of DROP. Created a blocker bug for this issue https://issues.apache.org/jira/browse/CLOUDSTACK-8688 Can somebody please fix it? Thanks, Sanjeev

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8664: Verify if queryAsyncJobR...

2015-07-27 Thread Sanjeev N
LGTM!! On Mon, Jul 27, 2015 at 12:11 PM, pritisarap12 g...@git.apache.org wrote: Github user pritisarap12 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/619#discussion_r35508814 --- Diff: test/integration/testpaths/testpath_queryAsyncJobResult.py

[BlockerBug] VR creation fails in basic zone

2015-07-24 Thread Sanjeev N
Hi, In latest build with ACS master, VR creation fails in basic zone. Created a blocker bug for this. More details @ https://issues.apache.org/jira/browse/CLOUDSTACK-8668 Thanks, Sanjeev

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8634: Made changes to test_sec...

2015-07-22 Thread Sanjeev N
Can somebody please review this PR? On Tue, Jul 14, 2015 at 5:58 PM, sanju1010 g...@git.apache.org wrote: GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/586 CLOUDSTACK-8634: Made changes to test_security_group.py test suite to support EIP

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8632: Unable to revert snapsho...

2015-07-19 Thread Sanjeev N
LGTM On Thu, Jul 16, 2015 at 11:34 AM, wilderrodrigues g...@git.apache.org wrote: Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/590#issuecomment-121843137 LGTM --- If your project is set up for it, you can reply to this

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8636: Verify failure of creati...

2015-07-15 Thread Sanjeev N
LGTM On Wed, Jul 15, 2015 at 2:04 PM, pritisarap12 g...@git.apache.org wrote: Github user pritisarap12 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/592#discussion_r34656988 --- Diff: test/integration/testpaths/testpath_custom_disk_offering.py

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8637: Verify that VM with same...

2015-07-15 Thread Sanjeev N
LGTM On Wed, Jul 15, 2015 at 3:29 PM, pritisarap12 g...@git.apache.org wrote: GitHub user pritisarap12 opened a pull request: https://github.com/apache/cloudstack/pull/594 CLOUDSTACK-8637: Verify that VM with same Display name creation fails if vm.instancename.flag=true When

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8633: Changing file permission...

2015-07-15 Thread Sanjeev N
I was trying to bring up MS with xencluster. On Wed, Jul 15, 2015 at 4:10 PM, pvr9712 g...@git.apache.org wrote: Github user pvr9712 commented on the pull request: https://github.com/apache/cloudstack/pull/591#issuecomment-121571026 BTW, here is my email on the Dev list with the

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8556: Unable to delete attache...

2015-07-14 Thread Sanjeev N
You are right Wido. It is only a test case change to handle volume cleanup. On Tue, Jul 14, 2015 at 11:43 AM, wido g...@git.apache.org wrote: Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/481#issuecomment-121140868 The description of the

Re: [GitHub] cloudstack pull request: Newb

2015-07-14 Thread Sanjeev N
LGTM!! On Tue, Jul 14, 2015 at 11:47 AM, pavanb018 g...@git.apache.org wrote: GitHub user pavanb018 opened a pull request: https://github.com/apache/cloudstack/pull/581 Newb Test Create/Delete a LB rule and verify correct usage is recorded ... === TestName: test_01_lb_usage |

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8631 :changing if condition in...

2015-07-14 Thread Sanjeev N
LGTM!! On Tue, Jul 14, 2015 at 5:08 PM, DaanHoogland g...@git.apache.org wrote: Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/585#issuecomment-121211124 I agree with Sanju. please ammend to something like CLOUDSTACK-8631:

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8626 :[Automation]fixing test/...

2015-07-13 Thread Sanjeev N
LGTM On Mon, Jul 13, 2015 at 11:02 AM, nitt10prashant g...@git.apache.org wrote: GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/579 CLOUDSTACK-8626 :[Automation]fixing test/integration/component/test_p… …s_max_limits.py for lxc

Re: [GitHub] cloudstack pull request: Handling Basic Zone scenarios for startin...

2015-07-09 Thread Sanjeev N
LGTM!! I am taking this patch. On Thu, Jul 9, 2015 at 3:25 PM, pavanb018 g...@git.apache.org wrote: Github user pavanb018 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/569#discussion_r34239937 --- Diff:

Re: revert or not revert? (was: [GitHub] cloudstack pull request: CLOUDSTACK-8583 : fixing issue related to...)

2015-07-08 Thread Sanjeev N
I thought there is a limitation on LGTMs for integration tests? On Wed, Jul 8, 2015 at 5:11 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: guys and dolss, this was closed with only 1 LGTM! we agreed not to submit with less then two ok reviews on master. So next steps? revert? some extra

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8582: Skipping unsuitable test...

2015-07-02 Thread Sanjeev N
if required_hardware=false means these tests can run on Simulator. On Thu, Jul 2, 2015 at 3:06 PM, runseb g...@git.apache.org wrote: Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/532#issuecomment-117977659 @gauravaradhye if you answer my

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8577: fixing script testpath_d...

2015-06-23 Thread Sanjeev N
LGTM. I am taking this patch. On Mon, Jun 22, 2015 at 6:26 PM, nitt10prashant g...@git.apache.org wrote: GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/507 CLOUDSTACK-8577: fixing script testpath_disable_enable_zone disable enable

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8576: Skip tests for LXC

2015-06-22 Thread Sanjeev N
Please ignore my comment. I confirmed with Kishan and he said template creation from volume is not supported in LXC. On Mon, Jun 22, 2015 at 5:45 PM, sanju1010 g...@git.apache.org wrote: Github user sanju1010 commented on a diff in the pull request:

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8572:Unable to deploy VM as no...

2015-06-19 Thread Sanjeev N
Agree with Gaurav, We should look at the tests which are changing the storage pool status. We should make sure that those tests turn the status back to UP. -Sanjeev On Fri, Jun 19, 2015 at 1:05 PM, gauravaradhye g...@git.apache.org wrote: Github user gauravaradhye commented on a diff in the

Re: Swift storage

2015-05-29 Thread Sanjeev N
No issues with S3 provider. On Fri, May 29, 2015 at 12:48 PM, Andrija Panic andrija.pa...@gmail.com wrote: auth v1, old libraries, lot's of limitations (cant extract/download ISO/template from ACS, dont see if register/download of ISO/template from URL has started at all - no progress

Re: Swift storage

2015-05-29 Thread Sanjeev N
or if it must be only 1 nfs box(that will ve converted to staging store)? Thx On May 29, 2015 11:02 AM, Sanjeev N sanj...@apache.org wrote: No issues with S3 provider. On Fri, May 29, 2015 at 12:48 PM, Andrija Panic andrija.pa...@gmail.com wrote: auth v1, old libraries, lot's

Re: support for /31 Networks

2015-05-22 Thread Sanjeev N
Not sure when this check was added excatly :( On Fri, May 22, 2015 at 1:12 AM, Remi Bergsma rberg...@schubergphilis.com wrote: Exactly! Left my mail in outbox and missed your reply ;-) @Sanjeev, since you answered the original mail.. Do you know when/where this check was added? Or else I’ll

Re: FW: Support for /31 Networks on cloudstack 4.4.2

2015-05-14 Thread Sanjeev N
Hi Devender, IIRC some changes have been made in this area in recent releases. So first ip address in the cidr would not be treated as valid ip address for assignment since it is a network id. So /31 would not have sufficient ips to deploy vm. Thanks, Sanjeev On Thu, May 14, 2015 at 10:15 PM,

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 raj...@apache.org wrote: congratulations Gaurav.. ~Rajani On Tue, Apr 14, 2015 at 1:35 AM, Erik

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

2015-04-02 Thread Sanjeev N
Hosting On Wed, Apr 1, 2015 at 3:20 AM, Sanjeev N sanj...@apache.org 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 Wed, Apr 1, 2015 at 11:56 AM, Erik Weber terbol...@gmail.com wrote: Haven't

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 terbol...@gmail.com wrote: Haven't tried it myself, but i think you should be able to create the vpc and change nic on

Re: [DISCUSS] Stop using Review Board

2015-03-30 Thread Sanjeev N
+1 On Mon, Mar 30, 2015 at 3:58 PM, Raja Pullela raja.pull...@citrix.com 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 ?

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 f...@barred.org wrote: Congrats Rohit, awesome! On 27 Mar 2015, at 09:20, Remi Bergsma rberg...@schubergphilis.com

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

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

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

2015-02-15 Thread sanjeev n
and 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 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 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 gaurav.arad...@clogeny.com Date: Tue Jan 20 16:18:08 2015 +0530 CLOUDSTACK-8170: Skipping tests on HyperV which try to scale VM in running state because

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 gaurav.arad...@clogeny.com Date: Tue Jan 13 13:56:41 2015 +0530 CLOUDSTACK-8152: Adding delay before creating snapshot/template from root disk after writing

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 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 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 gaurav.arad...@clogeny.com Date: Wed Jan 21 22:31:24 2015 -0800 CLOUDSTACK-8174: Fixed cleanup issue in test_add_remove_network.py - sanjeev

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 ashut...@clogeny.com Date: Tue Jan 6 12:54:49 2015 +0530 CLOUDSTACK-8143: Corrected a test in BVT test_routers.py - sanjeev

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 ashut...@clogeny.com Date: Tue Jan 6 12:54:49 2015 +0530 CLOUDSTACK-8143: Corrected a test in BVT test_routers.py

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

2015-01-13 Thread sanjeev n
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 generated e-mail. To reply, visit: https

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

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
- sanjeev n On Sept. 26, 2014, 10:58 a.m., prashant mishra wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26080/ --- (Updated Sept

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

2014-09-23 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 sanj...@apache.org 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

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
25536 - sanjeev --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25536/#review53939 --- On Sept. 11, 2014, 10:31 a.m., sanjeev n wrote

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

2014-09-22 Thread sanjeev n
, sanjeev n

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

2014-09-18 Thread sanjeev n
is not 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

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

2014-09-16 Thread sanjeev n
: Chandan Purushothama chandan.purushoth...@citrix.com 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

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

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
--- Yes Thanks, sanjeev n

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

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

  1   2   >