[GitHub] cloudstack issue #1455: [CLOUDSTACK-9328]: Fix vlan issues from test suite t...

2016-06-09 Thread sanju1010
Github user sanju1010 commented on the issue: https://github.com/apache/cloudstack/pull/1455 Failures from CI are not related to changes in this file. This commit makes changes to test_privategw_acl.py --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1464: [CLOUDSTACK-9337]Enhance vcenter.py to create data c...

2016-06-09 Thread sanju1010
Github user sanju1010 commented on the issue: https://github.com/apache/cloudstack/pull/1464 @swill , BVT results are clean. Can we push this to master? This will not have any impact on acs master stablity. --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: Marvin test to verify that adding TCP ports 500,450...

2016-06-01 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1183 Test to add TCP Port Forwarding rule for specific ports(500,1701 and 4500) in VPN ... === TestName: test_08_add_TCP_PF_Rule_In_VPN | Status : SUCCESS === ok

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-05-30 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1455#issuecomment-222489855 This will help in stabilizing CI runs. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-05-30 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1455#issuecomment-222489678 @swill , can you consider marvin test related changes for merging into ACS master since they would not destabilize the master? --- If your project is set up

[GitHub] cloudstack pull request: [CLOUDSTACK-9207] Test to verify restarti...

2016-05-30 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1304#issuecomment-222488422 @swill , recent CI run on this PR shows 0 Failures. Can you please merge this? --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: Changing vlan to None since network offer...

2016-05-30 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1571 Changing vlan to None since network offering being used has Specify Vlan set to No Default isolated network offering with SourceNat service has Specify Vlan parameter set to No. When

[GitHub] cloudstack pull request: CLOUDSTACK-9380: fix NPE in listDomains A...

2016-05-26 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1550#issuecomment-221788191 I ran bunch of tests(related to network functionality) from the regression test suite(test/integration/component) on this PR branch and I don't see any test

[GitHub] cloudstack pull request: CLOUDSTACK-9388: Remove string conversion...

2016-05-25 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1561 CLOUDSTACK-9388: Remove string conversion in assertion statement Remove string convertion in Assertion statement, since the start port parameter in listFirewallAPI response is of type

[GitHub] cloudstack pull request: [CLOUDSTACK-9207] Test to verify restarti...

2016-05-03 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1304#issuecomment-216524259 @swill , added check to see if m has any match object and fail if not. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-04-28 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1455#issuecomment-215427658 @swill , it is all green now. Can you please merge this PR ? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: [CLOUDSTACK-9207] Test to verify restarti...

2016-04-27 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1304#issuecomment-215309683 Thanks @swill , I will fix the error and update the PR. Thanks for taking your time and running it on your CI. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: [CLOUDSTACK-9207] Test to verify restarti...

2016-04-22 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1304#issuecomment-213448269 I have made changes as per @pavanb018's comments and tested it on my environment. Following are the test results: Test restart network with cleanup

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

2016-04-21 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1308#issuecomment-212839936 @pavanb018 , those failures are not related to this PR. Can somebody please merge this PR? --- If your project is set up for it, you can reply to this email

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

2016-04-21 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1308#issuecomment-212839793 @pavanb018 , those failures are not related to this PR. Can somebody merge this PR? --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: Fixed: Error given when creating VPN user...

2016-04-21 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/826#issuecomment-212836461 Tested these changes manually in my setup. Changes works fine. LGTM!! --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-21 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1458#issuecomment-212820627 LGTM based on the code review!! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] cloudstack pull request: [CLOUDSTACK-9337]Enhance vcenter.py to cr...

2016-04-20 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1464#issuecomment-212756206 These changes will not have any impact on cloudstack code since it is purely to orchestrate the vcenter Server to create datacenter->cluster->hosts. Can we

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

2016-04-06 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1308#discussion_r58690197 --- Diff: test/integration/component/test_vpc.py --- @@ -2490,3 +2498,23 @@ def test_21_deploy_vm_with_gateway_ip(self): "F

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

2016-04-06 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1316#issuecomment-206313506 Squashed changed into single commit and addressed all the review comments and also posted test run results after making changes. Can somebody please merge

[GitHub] cloudstack pull request: [CLOUDSTACK-9218]Test to verify restart n...

2016-04-06 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1323#discussion_r58686721 --- Diff: test/integration/component/test_redundant_router_cleanups.py --- @@ -666,3 +667,156 @@ def test_network_gc(self

[GitHub] cloudstack pull request: New test to validate starting vm after ni...

2016-04-06 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1326#discussion_r58685208 --- Diff: test/integration/component/test_add_remove_network.py --- @@ -1021,6 +1021,103 @@ def test_29_remove_nic_CS22503(self

[GitHub] cloudstack pull request: [Bug-Id: CLOUDSTACK-9306]Replace testdata...

2016-04-06 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1436#issuecomment-206296119 @swill, did you get a chance to run CI on this PR? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-04-06 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1455#issuecomment-206295391 @swill , I have made changes based on the CI results. Following failure in the CI run is not related to the changes in this PR

[GitHub] cloudstack pull request: [CLOUDSTACK-9337]Enhance vcenter.py to cr...

2016-04-06 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1464#discussion_r58660462 --- Diff: tools/marvin/marvin/lib/vcenter.py --- @@ -183,7 +192,157 @@ def get_clusters(self, dc, clus=None): """

[GitHub] cloudstack pull request: [CLOUDSTACK-9337]Enhance vcenter.py to cr...

2016-04-05 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1464 [CLOUDSTACK-9337]Enhance vcenter.py to created data center in vcenter server automatically These changes have been made to support vmware deployments in CI. For CI to create cloudstack

[GitHub] cloudstack pull request: CLOUDSTACK-9289:Automation for feature de...

2016-04-05 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1417#issuecomment-205684992 LGTM based on the code review !! Please squash the commits into single commit. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-03-31 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1455#issuecomment-204248348 @alexandrelimassantana , cleanup does not clean the db connection. So it should work for other tests as well. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-03-28 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1455#issuecomment-202342204 Made changes as per Shwetaag's comments. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-03-28 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1455 [CLOUDSTACK-9328]: Fix vlan issues from test suite test_privategw_acl.py in BVT Please refer to CLOUDSTACK-9328 for the details. Test Results

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

2016-03-19 Thread sanju1010
Github user sanju1010 closed the pull request at: https://github.com/apache/cloudstack/pull/1156 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

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

2016-03-18 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1156#issuecomment-197701355 @swill, PR #1276 outdated this PR. So closing this. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack pull request: CLOUDSTACK-9304: Add nuagevsp userdata te...

2016-03-10 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1431#issuecomment-195224152 LGTM based on the code review. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] cloudstack pull request: [Bug-Id: CS-9306]Replace testdata with se...

2016-03-10 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1436 [Bug-Id: CS-9306]Replace testdata with services Replaced people.apache.org with home.apache.org in test_usage services class and test_data.py file. Test Create/Delete a ISO

[GitHub] cloudstack pull request: CLOUDSTACK-9304: Add nuagevsp userdata te...

2016-03-10 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1431#discussion_r55646437 --- Diff: test/integration/component/test_nuage_vsp.py --- @@ -1,334 +0,0 @@ -# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-9304: Add nuagevsp userdata te...

2016-03-09 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1431#discussion_r55635511 --- Diff: test/integration/plugins/nuagevsp/test_nuage_vpc_network.py --- @@ -0,0 +1,98 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-9304: Add nuagevsp userdata te...

2016-03-09 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1431#discussion_r55635451 --- Diff: test/integration/plugins/nuagevsp/nuageTestCase.py --- @@ -0,0 +1,830 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-9304: Add nuagevsp userdata te...

2016-03-09 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1431#discussion_r55635262 --- Diff: test/integration/component/test_nuage_vsp.py --- @@ -1,334 +0,0 @@ -# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-9304: Add nuagevsp userdata te...

2016-03-09 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1431#discussion_r55635106 --- Diff: test/integration/component/test_nuage_vsp.py --- @@ -1,334 +0,0 @@ -# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-9168: Testpath to check if wro...

2016-02-26 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1245#issuecomment-189233999 -1 This test is not passing on a basic zone. In basic zone we don't list the network cidr in list network response. So test is failing at line no: 163

[GitHub] cloudstack pull request: CLOUDSTACK-9289:Automation for feature de...

2016-02-25 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1417#discussion_r54070462 --- Diff: test/integration/component/test_deploy_vm_from_snapshot.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-9289:Automation for feature de...

2016-02-25 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1417#discussion_r54068802 --- Diff: test/integration/component/test_deploy_vm_from_snapshot.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-9026: Modifying testpath for a...

2016-02-18 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1031#issuecomment-185662747 LGTM based on test results!! Test migrate Volume (root and data disk) ... === TestName: test_01_migrate_root_and_data_disk_nonlive | Status : SUCCESS

[GitHub] cloudstack pull request: CLOUDSTACK-8717: Failed to start instance...

2016-02-16 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1416#issuecomment-185061120 LGTM based on PR #667 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] cloudstack pull request: CLOUDSTACK-8717: Failed to start instance...

2016-02-16 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1411#issuecomment-185057058 I have reverted this merge commit commit d159af93b4e6485b30424026e1a84ce85a3c7abf Author: sanjeev <sanj...@apache.org> Date: Wed Feb 17 11

[GitHub] cloudstack pull request: CLOUDSTACK-8717: Failed to start instance...

2016-02-11 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/667#issuecomment-182802497 LGTM !! Test Restore VM on VMWare ... === TestName: test_01_recover_VM | Status : SUCCESS === ok

[GitHub] cloudstack pull request: CLOUDSTACK-8728: Testcase to Verify if VR...

2016-02-10 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/684#issuecomment-182751801 LGTM !! Test router internal basic zone ... SKIP: Marvin configuration has no host credentials tocheck router services

[GitHub] cloudstack pull request: automated 9277 9276 9275 9274 9273 9179 9...

2016-02-09 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1401#discussion_r52417090 --- Diff: test/integration/component/maint/test_escalations_instances.py --- @@ -0,0 +1,270 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: automated 9277 9276 9275 9274 9273 9179 9...

2016-02-09 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1401#discussion_r52417314 --- Diff: test/integration/component/test_escalations_instances.py --- @@ -2068,6 +2068,94 @@ def test_13_vm_nics(self): "V

[GitHub] cloudstack pull request: automated 9277 9276 9275 9274 9273 9179 9...

2016-02-09 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1401#discussion_r52417597 --- Diff: test/integration/component/test_project_usage.py --- @@ -1819,3 +1844,183 @@ def test_01_vpn_usage(self

[GitHub] cloudstack pull request: automated 9277 9276 9275 9274 9273 9179 9...

2016-02-09 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1401#discussion_r52417529 --- Diff: test/integration/component/test_project_usage.py --- @@ -221,10 +219,35 @@ def test_01_vm_usage(self): #VM.Destroy

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

2016-02-04 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1011#discussion_r51857735 --- Diff: test/integration/component/test_coreos.py --- @@ -0,0 +1,291 @@ +# Licensed to the Apache Software Foundation (ASF) under one

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

2016-02-04 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1011#discussion_r51857784 --- Diff: tools/marvin/marvin/config/test_data.py --- @@ -784,6 +784,19 @@ "format": "OVA",

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

2016-02-03 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1218#discussion_r51693967 --- Diff: test/integration/component/maint/testpath_disable_enable_zone.py --- @@ -1691,3 +1696,159 @@ def test_01_disable_enable_host(self

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

2016-02-03 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1218#discussion_r51694011 --- Diff: test/integration/component/maint/testpath_disable_enable_zone.py --- @@ -1691,3 +1696,159 @@ def test_01_disable_enable_host(self

[GitHub] cloudstack pull request: CLOUDSTACK-8830 : VM snapshot creation fa...

2016-02-02 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1377#discussion_r51680970 --- Diff: test/integration/component/test_escalations_snapshots.py --- @@ -152,6 +156,34 @@ def __verify_values(self, expected_vals, actual_vals

[GitHub] cloudstack pull request: CLOUDSTACK-8830 : VM snapshot creation fa...

2016-02-02 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1377#discussion_r51681502 --- Diff: test/integration/component/test_escalations_snapshots.py --- @@ -639,3 +671,47 @@ def test_04_list_vm_snapshots_byid(self

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

2016-02-02 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1078#issuecomment-179054474 LGTM based on code review!! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] cloudstack pull request: CLOUDSTACK-9168: Testpath to check if wro...

2016-02-02 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1245#discussion_r51685979 --- Diff: test/integration/testpaths/testpath_netmask.py --- @@ -0,0 +1,152 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-9168: Testpath to check if wro...

2016-02-02 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1245#discussion_r51686104 --- Diff: test/integration/testpaths/testpath_netmask.py --- @@ -0,0 +1,152 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: New test to validate starting vm after ni...

2016-01-09 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1326 New test to validate starting vm after nic removal and attach Please refer bug CLOUDSTACK-9219 for more details. Test Results: == Test to verify vm start after NIC

[GitHub] cloudstack pull request: [CLOUDSTACK-9218]Test to verify restart n...

2016-01-08 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1323 [CLOUDSTACK-9218]Test to verify restart network after master VR destroyed Please refer CLOUDSTACK-9218 for more details Test Results: === Test restarting RvR

[GitHub] cloudstack pull request: CLOUDSTACK-9128: Testcase to verify physi...

2016-01-05 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1199#issuecomment-168940359 LGTM !! Test Results: === Check Snapshots size in database ... === TestName: test_02_snapshot_size_check | Status : SUCCESS ===  okÂ

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

2016-01-05 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1308#issuecomment-168938202 Test to create vpn customer gateway with hostname ... === TestName: test_22_vpn_customer_gw_with_hostname | Status : SUCCESS === ok

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

2016-01-05 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1308 Test to create vpn customer gateway with hostname While adding vpn customer gateway for site to site vpn connection, cs should also accept host name apart from gateway ip address. It should

[GitHub] cloudstack pull request: [CLOUDSTACK-9207] Test to verify restarti...

2016-01-04 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1304#issuecomment-168653437 Test restart network with cleanup ... === TestName: test_restart_network_with_cleanup | Status : SUCCESS === ok

[GitHub] cloudstack pull request: [CLOUDSTACK-9207] Test to verify restarti...

2016-01-04 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1304 [CLOUDSTACK-9207] Test to verify restarting vpc network with cleanup does not delete the secondary ip address from VR Please check jira https://issues.apache.org/jira/browse/CLOUDSTACK

[GitHub] cloudstack pull request: CLOUDSTACK-8895: Verify if storage on sto...

2015-12-29 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/869#issuecomment-167752950 @DaanHoogland / @remibergsma can you please merge this PR? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack pull request: CLOUDSTACK-8895: Verify if storage on sto...

2015-12-29 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/869#issuecomment-167752857 It is not type Daan. First test inside TestAttachDataDisk class tests some functionality on ZWPS and this one is for CWPS. Test Results : Attach Data

[GitHub] cloudstack pull request: CLOUDSTACK-8895: Verify if storage on sto...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/869#issuecomment-167551878 I did a code walk-through and looks good. LGTM !! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack pull request: CLOUDSTACK-8308:Adding automation testcas...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/384#discussion_r48475810 --- Diff: test/integration/testpaths/testpath_delta_snapshots.py --- @@ -0,0 +1,539 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-8308:Adding automation testcas...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/384#discussion_r48475772 --- Diff: test/integration/testpaths/testpath_delta_snapshots.py --- @@ -0,0 +1,539 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-8308:Adding automation testcas...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/384#discussion_r48475782 --- Diff: test/integration/testpaths/testpath_delta_snapshots.py --- @@ -0,0 +1,539 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-9091: Update testpath for para...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1130#issuecomment-167562545 This is one line change and looks good. I will get test results for this. LGTM!! --- If your project is set up for it, you can reply to this email and have

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

2015-12-28 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1041#issuecomment-167562620 I will post test results. LGTM !! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] cloudstack pull request: CLOUDSTACK-8308:Adding automation testcas...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/384#discussion_r48476636 --- Diff: test/integration/testpaths/testpath_delta_snapshots.py --- @@ -0,0 +1,539 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-8308:Adding automation testcas...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/384#discussion_r48476586 --- Diff: test/integration/testpaths/testpath_delta_snapshots.py --- @@ -0,0 +1,539 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: [4.7] FIX Site2SiteVPN on redundant VPC

2015-12-27 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1276#discussion_r48463512 --- Diff: test/integration/smoke/test_vpc_vpn.py --- @@ -494,9 +526,411 @@ def create_natrule(self, vpc, vm, public_port, private_port, public_ip

[GitHub] cloudstack pull request: [4.7] FIX Site2SiteVPN on redundant VPC

2015-12-27 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1276#discussion_r48463624 --- Diff: test/integration/smoke/test_vpc_vpn.py --- @@ -494,9 +526,411 @@ def create_natrule(self, vpc, vm, public_port, private_port, public_ip

[GitHub] cloudstack pull request: [4.7] FIX Site2SiteVPN on redundant VPC

2015-12-27 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1276#discussion_r48463596 --- Diff: test/integration/smoke/test_vpc_vpn.py --- @@ -494,9 +526,411 @@ def create_natrule(self, vpc, vm, public_port, private_port, public_ip

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

2015-12-07 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1156#issuecomment-162539384 Hi Remi, As I mentioned in my PR the second test failure is related to connectivity issues between the vms part of s2s vpn. It is not because of my

[GitHub] cloudstack pull request: Marvin test to verify that adding TCP por...

2015-12-07 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1183 Marvin test to verify that adding TCP ports 500,4500 and 1701 in vpn should not fail Please refer to JIRA ticket for more details https://issues.apache.org/jira/browse/CLOUDSTACK-9117

[GitHub] cloudstack pull request: Changes made to DeployDataCenter to suppo...

2015-12-04 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/809#issuecomment-161982308 Sure Daan I will add . On Dec 4, 2015 7:58 PM, "Daan Hoogland" <notificati...@github.com> wrote: > @sanju1010 <https://github.

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

2015-12-03 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1156#issuecomment-161599503 Test Remote Access VPN in VPC ... === TestName: test_vpc_remote_access_vpn | Status : SUCCESS === ok

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

2015-12-03 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1156 Fixed issues with test_vpc_vpn script There are few issues with this test file. Please refer to JIRA Ticket https://issues.apache.org/jira/browse/CLOUDSTACK-9102 for details on the issues

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

2015-11-29 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1122#discussion_r46107622 --- Diff: test/integration/component/test_tags.py --- @@ -2401,3 +2403,143 @@ def test_23_list_untagged_host_for_vm_migration(self

[GitHub] cloudstack pull request: Change references to people.apache.org to...

2015-11-25 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1123#issuecomment-159807931 Thanks Talluri for quickly acting on it. I checked the urls and all are reachable. LGTM !! --- If your project is set up for it, you can reply

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

2015-11-25 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1122 Add marvin test to verify if DomainAdmin is able to delete the tags on public ip address Please go through bug https://issues.apache.org/jira/browse/CLOUDSTACK-9084 for more details

[GitHub] cloudstack pull request: CLOUDSTACK-9016 Fail to create VM instanc...

2015-11-02 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1020#issuecomment-152963577 Test Results: test_21_deploy_vm_with_gateway_ip (integration.component.test_vpc.TestVPC) ... === TestName: test_21_deploy_vm_with_gateway_ip | Status

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

2015-10-26 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/902#issuecomment-151068985 @pvr9711 can you please review this PR? I have made changes as per the review comments and rebased it with master. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: CLOUDSTACK-8960: Remove Citrix Resources ...

2015-10-17 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/939 CLOUDSTACK-8960: Remove Citrix Resources from test_data.py Replace URLs related to templates and ISOs with the ones accessbile to everybody in the community. I have copied all

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-09-30 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/900 CLOUDSTACK-8924: Removed duplicate test from test_scale_vm.py Please go through CS-8924 for more details. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-09-30 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/900#issuecomment-144344567 root@localhost:~/cloudstack# nosetests --with-marvin --marvin-config=/root/cloudstack/setup/dev/local1.cfg --zone=Sandbox-simulator --hypervisor=simulator -a tags

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

2015-09-30 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/902 CLOUDSTACK-8924: Enable dynamic scaling to run test_scale_vm.py test on simulator Simulator setup uses the config file from following location: tools/marvin/marvin/config/setup.cfg

[GitHub] cloudstack pull request: Changes made to DeployDataCenter to suppo...

2015-09-29 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/809#issuecomment-144028463 @DaanHoogland , this test requires hardware. So I can't add this as part of travis yaml. --- If your project is set up for it, you can reply to this email

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

2015-09-22 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/871 CLOUDSTACK-8893: Fixing script as per the latest functionality Please check https://issues.apache.org/jira/browse/CLOUDSTACK-8893 for more details. You can merge this pull request into a Git

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

2015-09-22 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/871#issuecomment-142271544 @Desc: Test that Volume snapshot for root volume not allowed ... === TestName: test_01_test_vm_volume_snapshot | Status : SUCCESS === ok

[GitHub] cloudstack pull request: Tagging tests appropriately to pick them ...

2015-09-14 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/819 Tagging tests appropriately to pick them for running on basic zone Adding additional tag to the list of tags. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack pull request: CLOUDSTACK-8745 : verify usage after root...

2015-09-14 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/713#issuecomment-139975477 LGTM!! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] cloudstack pull request: Changes made to DeployDataCenter to suppo...

2015-09-11 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/809 Changes made to DeployDataCenter to support Baremetal Basic Zone Creation 1. Made changes to deployDataCenter.py to support zone creation with Baremetal as the Hyeprvisor 2. Added test

  1   2   >