[GitHub] cloudstack pull request: CLOUDSTACK-8299: Adding test case to test...

2015-03-04 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/95 CLOUDSTACK-8299: Adding test case to test ingress rule for security group for specific IP set Adding test case to test ingress rules for specific IP sets (Not 0.0.0.0/0, e.g. 10.2.3.4/32

[GitHub] cloudstack pull request: CLOUDSTACK-8299: Adding additional test c...

2015-03-09 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/102 CLOUDSTACK-8299: Adding additional test case to test ingress rules with two security groups together This is in addition to pull request at https://github.com/apache/cloudstack/pull/95

[GitHub] cloudstack pull request: CLOUDSTACK-8311: Adding test case to conf...

2015-03-10 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/107 CLOUDSTACK-8311: Adding test case to confirm Volume Snapshot for root disk is not allowed when VM snapshot is present for a VM CLOUDSTACK-8311: Adding test case to confirm Volume Snapshot

[GitHub] cloudstack pull request: CLOUDSTACK-8314: Add test case to validat...

2015-03-11 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/110 CLOUDSTACK-8314: Add test case to validate VM.DESTROY event is logged when VM deployment fails VM is created with huge service offering so that VM creation fails. In this case

[GitHub] cloudstack pull request: CLOUDSTACK-8299: Adding additional test c...

2015-03-11 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/102#issuecomment-78430462 No, this is not intended for 4.5. I have not tested on 4.5. Regards, Gaurav On Tue, Mar 10, 2015 at 10:22 AM, Rohit Yadav wrote

[GitHub] cloudstack pull request: CLOUDSTACK-8323: Adding test case to veri...

2015-03-12 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/116 CLOUDSTACK-8323: Adding test case to verify NIC Adapter types for nic in VM when NIC Adapter is specified in template The test case will verify that when a template is registered with

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-12 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26301214 --- Diff: test/integration/smoke/test_vm_snapshots.py --- @@ -16,14 +16,16 @@ # under the License. --- End diff -- This file seems

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-12 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26301332 --- Diff: test/integration/testpaths/testpath_volume_snapshot.py --- @@ -0,0 +1,745 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-12 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26301374 --- Diff: test/integration/testpaths/testpath_volume_snapshot.py --- @@ -0,0 +1,745 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-12 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26301476 --- Diff: test/integration/testpaths/testpath_volume_snapshot.py --- @@ -0,0 +1,745 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-12 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26301316 --- Diff: test/integration/testpaths/testpath_volume_snapshot.py --- @@ -0,0 +1,745 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-12 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26301742 --- Diff: test/integration/testpaths/testpath_volume_snapshot.py --- @@ -0,0 +1,745 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-12 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26301554 --- Diff: test/integration/testpaths/testpath_volume_snapshot.py --- @@ -0,0 +1,745 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-12 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26301600 --- Diff: test/integration/testpaths/testpath_volume_snapshot.py --- @@ -0,0 +1,745 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-19 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/117#issuecomment-83436486 Hi Rohit, I am yet to review updated pull request. Regards, Gaurav On Thu, Mar 19, 2015 at 2:22 PM, Rohit Yadav wrote

[GitHub] cloudstack pull request: CLOUDSTACK-8336: Adding test for shared n...

2015-03-19 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/120 CLOUDSTACK-8336: Adding test for shared network restart, with and without cleanup Network restart test case was missing in the repo for shared network. So adding a basic test case that

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-19 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26748249 --- Diff: tools/marvin/marvin/lib/common.py --- @@ -1395,3 +1399,199 @@ def isNetworkDeleted(apiclient, networkid, timeout=600

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-19 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26748285 --- Diff: tools/marvin/marvin/lib/common.py --- @@ -1395,3 +1399,199 @@ def isNetworkDeleted(apiclient, networkid, timeout=600

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-19 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26748307 --- Diff: tools/marvin/marvin/lib/common.py --- @@ -1395,3 +1399,199 @@ def isNetworkDeleted(apiclient, networkid, timeout=600

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-19 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/117#issuecomment-83533242 Also please add the test case execution results. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack pull request: CLOUDSTACK-8330: Tagging test cases which...

2015-03-19 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/121 CLOUDSTACK-8330: Tagging test cases which should be run with vmware dvs configuration Tagging test case with "dvs" tag which should be run with Vmware DVS configuration. You

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-20 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26833696 --- Diff: tools/marvin/marvin/lib/base.py --- @@ -1086,7 +1092,7 @@ def create(cls, apiclient, services, volumeid=None, @classmethod

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-20 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26833707 --- Diff: tools/marvin/marvin/lib/base.py --- @@ -1143,9 +1149,6 @@ def register(cls, apiclient, services, zoneid=None, elif "proj

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-20 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26833788 --- Diff: tools/marvin/marvin/lib/common.py --- @@ -1395,3 +1396,218 @@ def isNetworkDeleted(apiclient, networkid, timeout=600

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-03-20 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r26833819 --- Diff: tools/marvin/marvin/lib/common.py --- @@ -1395,3 +1396,218 @@ def isNetworkDeleted(apiclient, networkid, timeout=600

[GitHub] cloudstack pull request: CLOUDSTACK-8340: Adding test case for sto...

2015-03-23 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/122 CLOUDSTACK-8340: Adding test case for stop/start operation for VPC router Adding basic start/stop VPC router test case as this scenario was absent in repository. Steps: 1

[GitHub] cloudstack pull request: CLOUDSTACK-8278: Usage test path - Correc...

2015-03-23 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/127 CLOUDSTACK-8278: Usage test path - Correct code flow in case the usage job configuration is not to be set through test case The flag "setUsageConfigurationThroughTestCase"

[GitHub] cloudstack pull request: CLOUDSTACK-8251: Adding automation test c...

2015-03-24 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/128 CLOUDSTACK-8251: Adding automation test cases for dedicated public IP ranges Test cases for dedicated public IP ranges feature. Tested on KVM setup. Although this test cases won&#

[GitHub] cloudstack pull request: CLOUDSTACK-8344: Fixed SSH failures in te...

2015-03-24 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/130 CLOUDSTACK-8344: Fixed SSH failures in test_haproxy.py test suite The test case in test suite test_haproxy.py are failing while attempting SSH due to Authentication error. The test

[GitHub] cloudstack pull request: CLOUDSTACK-8330: Adding dvs tag to test c...

2015-03-31 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/136 CLOUDSTACK-8330: Adding dvs tag to test cases which can be run on DVS configuration on vmware In continuation with Pull Request: https://github.com/apache/cloudstack/pull/121 These

[GitHub] cloudstack pull request: CLOUDSTACK-8356: Mentioning netaddr modul...

2015-04-01 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/138 CLOUDSTACK-8356: Mentioning netaddr module in setup.py in marvin as it is used across test cases The netaddr module is used in various test cases but it is not mentioned in setup.py file

[GitHub] cloudstack pull request: CLOUDSTACK-8356: Mentioning netaddr modul...

2015-04-01 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/138#issuecomment-88421366 Rebased with latest master. --- 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-8356: Mentioning netaddr modul...

2015-04-01 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/138#issuecomment-88421582 Done. Regards, Gaurav On Wed, Apr 1, 2015 at 3:20 PM, Srikanteswararao Talluri < notificati...@github.com> wrote: > L

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-04-02 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r27650336 --- Diff: test/integration/testpaths/testpath_storage_migration.py --- @@ -479,6 +283,19 @@ def setUpClass(cls

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-04-02 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r27650373 --- Diff: test/integration/testpaths/testpath_storage_migration.py --- @@ -536,8 +353,10 @@ def test_01_migrate_root_and_data_disk_nonlive(self

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-04-02 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r27650440 --- Diff: test/integration/testpaths/testpath_storage_migration.py --- @@ -659,11 +478,11 @@ def test_01_migrate_root_and_data_disk_nonlive(self

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-04-02 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r27650484 --- Diff: test/integration/testpaths/testpath_storage_migration.py --- @@ -765,7 +597,7 @@ def test_01_migrate_root_and_data_disk_nonlive(self

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-04-02 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r27650534 --- Diff: test/integration/testpaths/testpath_storage_migration.py --- @@ -1771,17 +1666,17 @@ def

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-04-02 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r27650755 --- Diff: test/integration/testpaths/testpath_volume_recurring_snap.py --- @@ -0,0 +1,1015 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-04-02 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r27650780 --- Diff: test/integration/testpaths/testpath_volume_recurring_snap.py --- @@ -0,0 +1,1015 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-04-02 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/117#discussion_r27651210 --- Diff: test/integration/testpaths/testpath_volume_recurring_snap.py --- @@ -0,0 +1,1015 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding automation test c...

2015-04-02 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/117#issuecomment-0495 Will review remaining changes tomorrow. A quick question, running the test suite should not leave anything behind in the setup. Is this taken care of? --- If

[GitHub] cloudstack pull request: CLOUDSTACK-8361: Adding functionality in ...

2015-04-03 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/144 CLOUDSTACK-8361: Adding functionality in marvin to check the port groups in VCenter corresponding to configuration of traffics in physical networks Adding common function in marvin which

[GitHub] cloudstack pull request: CLOUDSTACK-8356: Mentioning netaddr modul...

2015-04-04 Thread gauravaradhye
Github user gauravaradhye closed the pull request at: https://github.com/apache/cloudstack/pull/138 --- 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, or if the

[GitHub] cloudstack pull request: CLOUDSTACK-8356: Mentioning netaddr modul...

2015-04-04 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/138#issuecomment-89627323 Thanks Rohit. Closed now. On Apr 4, 2015, at 2:20 PM, Rohit Yadav wrote: > Looks like already merged c05e37b, but the merger did not add the &q

[GitHub] cloudstack pull request: CLOUDSTACK-8375: Marvin - Code Improvemen...

2015-04-11 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/155 CLOUDSTACK-8375: Marvin - Code Improvement - Verifying VCenter port groups corresponding to traffic types in a zone 1. Reducing complexity of function "verifyVCenterPortGroups"

[GitHub] cloudstack pull request: CLOUDSTACK-8377: Marvin - base library - ...

2015-04-11 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/156 CLOUDSTACK-8377: Marvin - base library - Fix list method for PhysicalNetwork class Removed map() from list method for PhysicalNetwork class. map() function works only when the second

[GitHub] cloudstack pull request: CLOUDSTACK-8380: Adding script to testSet...

2015-04-13 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/159 CLOUDSTACK-8380: Adding script to testSetupSuccess.py to check VCenter port groups are created for storage, management and public traffic as soon as zone is deployed This test will be

[GitHub] cloudstack pull request: CLOUDSTACK-8375: Marvin - Code Improvemen...

2015-04-13 Thread gauravaradhye
Github user gauravaradhye closed the pull request at: https://github.com/apache/cloudstack/pull/155 --- 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, or if the

[GitHub] cloudstack pull request: CLOUDSTACK-8374: Adding tests for guest t...

2015-04-14 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/166 CLOUDSTACK-8374: Adding tests for guest traffic port group verification - Shared and Isolated networks For Isolated network, network and VM is already created in setUpClass, so only

[GitHub] cloudstack pull request: CLOUDSTACK-8374: Adding tests for guest t...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye closed the pull request at: https://github.com/apache/cloudstack/pull/166 --- 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, or if the

[GitHub] cloudstack pull request: CLOUDSTACK-8374: Adding tests for guest t...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/166#issuecomment-92815608 This is committed. Closing. --- 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-8377: Marvin - base library - ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/156#issuecomment-92825267 You are right Sanjeev. For all the other classes it is done this way, without using map, but sometimes we need to use the object methods and that are not

[GitHub] cloudstack pull request: CLOUDSTACK-8377: Marvin - base library - ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye closed the pull request at: https://github.com/apache/cloudstack/pull/156 --- 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, or if the

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395596 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + --- End diff -- Please

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395604 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395630 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395658 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395789 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395840 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395900 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase

[GitHub] cloudstack pull request: Issue:Domian admin can view templates in ...

2015-04-14 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/169#discussion_r28395963 --- Diff: test/integration/component/test_escalation_listTemplatesDomainAdmin1.py --- @@ -0,0 +1,94 @@ + +from marvin.cloudstackTestCase

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

2015-04-15 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/170#issuecomment-93288143 Please include test results. --- 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-test-ca...

2015-04-15 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28406940 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -0,0 +1,355 @@ +# Licensed to the Apache Software Foundation (ASF) under

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

2015-04-15 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28407016 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -0,0 +1,355 @@ +# Licensed to the Apache Software Foundation (ASF) under

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

2015-04-15 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28407125 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -0,0 +1,355 @@ +# Licensed to the Apache Software Foundation (ASF) under

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

2015-04-15 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28407152 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -0,0 +1,355 @@ +# Licensed to the Apache Software Foundation (ASF) under

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

2015-04-15 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28407187 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -0,0 +1,355 @@ +# Licensed to the Apache Software Foundation (ASF) under

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

2015-04-15 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28407255 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -0,0 +1,355 @@ +# Licensed to the Apache Software Foundation (ASF) under

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

2015-04-15 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28407300 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -0,0 +1,355 @@ +# Licensed to the Apache Software Foundation (ASF) under

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

2015-04-15 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28407289 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -0,0 +1,355 @@ +# Licensed to the Apache Software Foundation (ASF) under

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

2015-04-15 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28407426 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -0,0 +1,355 @@ +# Licensed to the Apache Software Foundation (ASF) under

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

2015-04-15 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28407493 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -0,0 +1,355 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8388: test_accounts.py - Fixin...

2015-04-15 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/171 CLOUDSTACK-8388: test_accounts.py - Fixing pep8 issues Fixed all pep8 issues. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] cloudstack pull request: CLOUDSTACK-8388: test_VirtualRouter_alert...

2015-04-15 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/173 CLOUDSTACK-8388: test_VirtualRouter_alerts.py - Fix wait time and pep8 issues The wait period should be only 60 seconds more than "router.alerts.check.interval", it was 6

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

2015-04-16 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28493161 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -0,0 +1,341 @@ +# Licensed to the Apache Software Foundation (ASF) under

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

2015-04-16 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28493150 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -0,0 +1,341 @@ +# Licensed to the Apache Software Foundation (ASF) under

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

2015-04-16 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28493223 --- Diff: test/integration/testpaths/testpath_snapshot_limits.py --- @@ -0,0 +1,341 @@ +# Licensed to the Apache Software Foundation (ASF) under

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

2015-04-16 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/170#discussion_r28493238 --- Diff: tools/marvin/marvin/codes.py --- @@ -132,3 +132,9 @@ Switch Type ''' VMWARED

[GitHub] cloudstack pull request: CLOUDSTACK-8390: Skipping VPC tests on Hy...

2015-04-17 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/179 CLOUDSTACK-8390: Skipping VPC tests on Hyperv VPC network is not supported on hyper-v hypervisor. Skip relevant test cases accordingly. You can merge this pull request into a Git

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

2015-04-22 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/181#discussion_r28939755 --- Diff: test/integration/testpaths/testpath_volume_cuncurrent_snapshots.py --- @@ -0,0 +1,827 @@ +# Licensed to the Apache Software

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

2015-04-22 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/181#discussion_r28939823 --- Diff: test/integration/testpaths/testpath_volume_cuncurrent_snapshots.py --- @@ -0,0 +1,827 @@ +# Licensed to the Apache Software

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

2015-04-22 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/181#discussion_r28940101 --- Diff: test/integration/testpaths/testpath_volume_cuncurrent_snapshots.py --- @@ -0,0 +1,827 @@ +# Licensed to the Apache Software

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

2015-04-22 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/181#discussion_r28940210 --- Diff: test/integration/testpaths/testpath_volume_cuncurrent_snapshots.py --- @@ -0,0 +1,827 @@ +# Licensed to the Apache Software

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

2015-04-22 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/181#discussion_r28940257 --- Diff: test/integration/testpaths/testpath_volume_cuncurrent_snapshots.py --- @@ -0,0 +1,827 @@ +# Licensed to the Apache Software

[GitHub] cloudstack pull request: To Verfiy that list templates by domain a...

2015-04-23 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/189#discussion_r29025564 --- Diff: test/integration/component/test_escalation_listTemplateDomainAdmin.py --- @@ -0,0 +1,157 @@ + +# Licensed to the Apache Software

[GitHub] cloudstack pull request: To Verfiy that list templates by domain a...

2015-04-23 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/189#discussion_r29025553 --- Diff: test/integration/component/test_escalation_listTemplateDomainAdmin.py --- @@ -0,0 +1,157 @@ + +# Licensed to the Apache Software

[GitHub] cloudstack pull request: CLOUDSTACK-8394: Use custom decorator to ...

2015-04-24 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/190 CLOUDSTACK-8394: Use custom decorator to skip test case When the tests are skipped through setUpClass, the statement "raise unittest.SkipTest()" is used. This is seen as except

[GitHub] cloudstack pull request: CLOUDSTACK-8304: Modify tags accordingly ...

2015-04-26 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/193 CLOUDSTACK-8304: Modify tags accordingly for tests which can't be run on simulator The test cases try to download ISO, should not be run on simulator. required_hardware tag val

[GitHub] cloudstack pull request: CLOUDSTACK-8394: Skip test cases through ...

2015-04-28 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/203 CLOUDSTACK-8394: Skip test cases through setUp() instead of setUpClass() When the tests are skipped through setUpClass, the statement "raise unittest.SkipTest()" is used. This

[GitHub] cloudstack pull request: CLOUDSTACK-8394: Skipping download ISO te...

2015-04-29 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/207 CLOUDSTACK-8394: Skipping download ISO tests for simulator Download ISO tests should not be run on simulator You can merge this pull request into a Git repository by running: $ git

[GitHub] cloudstack pull request: CLOUDSTACK-8394: Skipping snapshots test ...

2015-04-29 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/212 CLOUDSTACK-8394: Skipping snapshots test cases for HyperV and LXC Skipping these tests as they are not supported on given hypervisors. You can merge this pull request into a Git

[GitHub] cloudstack pull request: Added recently announced committers

2015-05-04 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/209#discussion_r29569460 --- Diff: who.html --- @@ -240,12 +240,20 @@ Committers ** Devdeep Singh ** (devdeep) +** Wilder Rodrigues ** (ekho

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

2015-05-04 Thread gauravaradhye
Github user gauravaradhye commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/181#discussion_r29571592 --- Diff: test/integration/testpaths/testpath_volume_cuncurrent_snapshots.py --- @@ -0,0 +1,849 @@ +# Licensed to the Apache Software

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

2015-05-04 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/181#issuecomment-98685482 Looks Good. Merging. --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8394: Skip test cases through ...

2015-05-06 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/231 CLOUDSTACK-8394: Skip test cases through setUp() instead of setUpClass() Doing the changes for regression test cases. This has already been done for BVTs. Reason: When raise

[GitHub] cloudstack pull request: CLOUDSTACK-8445: Keep only dvs tag for te...

2015-05-06 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/232 CLOUDSTACK-8445: Keep only dvs tag for test case which tests the VCenter port groups The test cases is tested only for DVS. This will prevent it to run on normal configuration. You can

[GitHub] cloudstack pull request: CLOUDSTACK-8449: Include zoneid parameter...

2015-05-06 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/234 CLOUDSTACK-8449: Include zoneid parameter in base library for updateConfiuration method Some test cases pass zoneid for Configurations.update method but it is absent in base library

[GitHub] cloudstack pull request: CLOUDSTACK-8450: Fix method calls in test...

2015-05-07 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/235 CLOUDSTACK-8450: Fix method calls in test_escalations_hosts.py Number of arguments in method definition and method call are mismatched in this test suite. Did not test this. Only

[GitHub] cloudstack pull request: CLOUDSTACK-8453: Fix DB result check in t...

2015-05-08 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/239 CLOUDSTACK-8453: Fix DB result check in test_VirtualRouter_alerts.py After stopping the apache service, the alerts in database is matched currently with the string "Monitoring Servi

  1   2   3   >