[GitHub] cloudstack issue #1573: CLOUDSTACK-9396:fixing issue related to script test_...

2017-03-02 Thread nitt10prashant
Github user nitt10prashant commented on the issue: https://github.com/apache/cloudstack/pull/1573 not needed --- 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

[GitHub] cloudstack pull request #1573: CLOUDSTACK-9396:fixing issue related to scrip...

2017-03-02 Thread nitt10prashant
Github user nitt10prashant closed the pull request at: https://github.com/apache/cloudstack/pull/1573 --- 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 issue #1563: CLOUDSTACK-9389:[automation]updating test_routers_ne...

2017-03-02 Thread nitt10prashant
Github user nitt10prashant commented on the issue: https://github.com/apache/cloudstack/pull/1563 Not needed --- 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

[GitHub] cloudstack pull request #1563: CLOUDSTACK-9389:[automation]updating test_rou...

2017-03-02 Thread nitt10prashant
Github user nitt10prashant closed the pull request at: https://github.com/apache/cloudstack/pull/1563 --- 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 issue #1563: CLOUDSTACK-9389:[automation]updating test_routers_ne...

2016-06-09 Thread nitt10prashant
Github user nitt10prashant commented on the issue: https://github.com/apache/cloudstack/pull/1563 @swill just curious to know Which guest OS you are using ? Since Cent OS template is default in ACS i modified test_data for cent OS. Regx should be consider ? ->I Ag

[GitHub] cloudstack issue #1563: CLOUDSTACK-9389:[automation]updating test_routers_ne...

2016-06-09 Thread nitt10prashant
Github user nitt10prashant commented on the issue: https://github.com/apache/cloudstack/pull/1563 -->I think it is important that the test data does not need to be changed for our automated(ish) CI to show the tests are passing @swill i think test-data meant to be changed dep

[GitHub] cloudstack pull request: CLOUDSTACK-9389:[automation]updating test_routers_n...

2016-05-31 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1563 @swill Thinking about 3(.*?) received , i have placed check_string in test_data.py , which can be modified depend on guest OS type . I followed cent os ping pattern since most

[GitHub] cloudstack pull request: CLOUDSTACK-9389:[automation]updating test_routers_n...

2016-05-31 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1563 @swill My MS is running on Rhel6.3 , in script we are doing ssh to guest vm and then trying to ping 8.8.8.8 so ping is dependent on guest template OS , in my case i am using

[GitHub] cloudstack pull request: CLOUDSTACK-9396:fixing issue related to s...

2016-05-31 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/1573 CLOUDSTACK-9396:fixing issue related to script test_dhcphosts fixing following 1-issue related to check_String:changed check string to "3 received" and use test_data

[GitHub] cloudstack pull request: CLOUDSTACK-9389:[automation]updating test...

2016-05-31 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1563#issuecomment-222635435 @koushik-das as you have asked done following 1- moved check_string to test_data.py 2-provided test result can you review changes

[GitHub] cloudstack pull request: CLOUDSTACK-9389:[automation]updating test...

2016-05-30 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1563#issuecomment-222439377 @swill i don't see any issue in my setup , please checktest result [results.txt](https://github.com/apache/cloudstack/files/289139/results.txt

[GitHub] cloudstack pull request: CLOUDSTACK-9389:[automation]updating test...

2016-05-30 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1563#issuecomment-222438208 test result Prashants-MacBook-Pro:test_routers_network_ops_TWUWR8 prashant$ cat results.txt Test redundant router internals

[GitHub] cloudstack pull request: CLOUDSTACK-9389:changing check_string fro...

2016-05-26 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/1563 CLOUDSTACK-9389:changing check_string from 3 packets received to 3 re… test cases were failing due to invalid check_string so changing check_string from "3 packets rec

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

2016-05-19 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1417#issuecomment-220521199 @swill i do not see any comment on parent ticket CLOUDSTACK-8676 , i have asked Sateesh to update the ticket , will update the PR as soon as i hear from

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

2016-05-03 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1417#issuecomment-216513459 @swill thanks for your comment and time , it seems the code is not in the master for which i have written the test script , it will take another 2 weeks

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-27 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1500#issuecomment-215308516 LGTM for test code 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

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

2016-04-26 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1417#issuecomment-214660883 @swill seedTemplateFromVmSnapshot is new API got introduced with this feature , if marvin installation is up to date it should have this API @bvbharat can

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

2016-04-22 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1417#issuecomment-213424673 @swill thanks for comment , i just did a force push --- 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-8745 : verify usage after root...

2016-04-22 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/713#issuecomment-213278268 @bhaisaab rebased and merged into single commit ,@swill @koushik-das can you please look into this --- If your project is set up for it, you can reply

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

2016-04-22 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1417#issuecomment-213277092 @cloudsadhu @pavanb018 @GabrielBrascher @sanju1010 can you please test this PR --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-21 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1500#discussion_r60533148 --- Diff: test/integration/component/test_volumes.py --- @@ -603,7 +603,144 @@ def test_01_volume_attach_detach(self): "

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-21 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1500#discussion_r60533118 --- Diff: test/integration/component/test_volumes.py --- @@ -603,7 +603,144 @@ def test_01_volume_attach_detach(self): "

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-21 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1500#discussion_r60532285 --- Diff: test/integration/component/test_volumes.py --- @@ -603,7 +603,144 @@ def test_01_volume_attach_detach(self): "

[GitHub] cloudstack pull request: Fixing an issue in Marvin around creating...

2016-04-21 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1501#issuecomment-212767179 LGTM based on code review @mike-tutkowski can you provide test results --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-21 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1500#discussion_r60531417 --- Diff: test/integration/component/test_volumes.py --- @@ -603,7 +603,144 @@ def test_01_volume_attach_detach(self): "

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-21 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1500#discussion_r60531171 --- Diff: test/integration/component/test_volumes.py --- @@ -603,7 +603,144 @@ def test_01_volume_attach_detach(self): "

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

2016-03-19 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1417#issuecomment-198314365 @GabrielBrascher @sanju1010 i have removed redundant code --- If your project is set up for it, you can reply to this email and have your reply appear

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

2016-03-06 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1417#discussion_r55160105 --- 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-8745 : verify usage after root...

2016-03-01 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/713#issuecomment-191098254 sure --- 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

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

2016-02-24 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1377#issuecomment-188173843 this is redundant case , and checking time to create vmsnapshot is not required for this bug. I have talked to @sarathkouk , he understood the issue

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

2016-02-18 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1031#discussion_r53300709 --- Diff: test/integration/testpaths/testpath_storage_migration.py --- @@ -248,6 +248,11 @@ def setUpClass(cls): DomainName

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

2016-02-16 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/1417 CLOUDSTACK-9289:Automation for feature deploy vm from vmsnapshot Automation for deploy vm from snapshot . 1-Test to create VM from vm snapshots without data disk 2-Deploy vm from

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

2016-02-11 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/667#issuecomment-182784773 LGTM ..based on code review and result . --- 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-9128: Testcase to verify physi...

2016-01-18 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1199#issuecomment-172463466 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

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

2016-01-18 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1190#issuecomment-172466590 LGTM Test revert snapshot on XenServer ... === TestName:test_01_check_revert_snapshot | Status SUCCESS === ok --- If your project is set up

[GitHub] cloudstack pull request: CLOUDSTACK-8731-checking usage event for ...

2015-12-15 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1048#issuecomment-164736785 can you please share test result ? --- 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-8731-checking usage event for ...

2015-12-15 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1048#issuecomment-164740710 LGTM based on code review and test case result --- 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-9087:adding projectid paramete...

2015-12-04 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1153#issuecomment-161931625 @pdube you are right projectid is not required , i did check API doc but got confused with UI and took it as doc bug. when createVMSnapshot is being

[GitHub] cloudstack pull request: CLOUDSTACK-9087:adding projectid paramete...

2015-12-04 Thread nitt10prashant
Github user nitt10prashant closed the pull request at: https://github.com/apache/cloudstack/pull/1153 --- 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: CLOUDSTACK-9087:adding projectid paramete...

2015-12-02 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/1153 CLOUDSTACK-9087:adding projectid parameter to create method of class VmSnapshot Adding projectid parameter to class VmSnapshot of base.py . You can merge this pull request into a Git

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

2015-11-03 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1000#issuecomment-153287076 LGTM Based on code understanding One line change if attribute is there then proceed to cleanup if hasattr(self, "data_volume_cr

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

2015-11-03 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/994#issuecomment-153285936 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

[GitHub] cloudstack pull request: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-27 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/960#issuecomment-151720250 based on discussion closing the PR Thanks @remibergsma @runseb @miguelaferreira @pvr9711 for great discussion . --- If your project is set up

[GitHub] cloudstack pull request: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-27 Thread nitt10prashant
Github user nitt10prashant closed the pull request at: https://github.com/apache/cloudstack/pull/960 --- 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: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-23 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/960#issuecomment-150501158 @runseb "I personally fail to see the benefit of skipped tests versus a failed test" If i may , skip is a type of failure wher

[GitHub] cloudstack pull request: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-22 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/960#issuecomment-150161922 @miguelaferreira i was going though mail thread and found this "I think the test should fail if the config is not right, or otherwise be exc

[GitHub] cloudstack pull request: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-21 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/960 CLOUDSTACK-8975:skip test case if niciraNvp settings are not available in config file root/cloudstack/test/integration/smoke/test_nicira_controller.py is failing if niciraNvp settings

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

2015-09-24 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/871#issuecomment-142820449 @remibergsma Next time sure I will add some details .for this PR i have updated my comments . --- If your project is set up for it, you can reply

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

2015-09-22 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/871#issuecomment-142493294 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

[GitHub] cloudstack pull request: [Automation]Volume migration between pool...

2015-09-22 Thread nitt10prashant
Github user nitt10prashant closed the pull request at: https://github.com/apache/cloudstack/pull/742 --- 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: CLOUDSTACK-8756:Incorrect guest os mappin...

2015-09-11 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/728#issuecomment-139530956 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

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

2015-08-26 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/688#issuecomment-134972409 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

[GitHub] cloudstack pull request: [Automation]Volume migration between pool...

2015-08-25 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/742 [Automation]Volume migration between pools times out in ACS, but but the migration completes on Xenserve Ticket :CLOUDSTACK-8771 description: Volume migration between

[GitHub] cloudstack pull request: Customer escalations

2015-08-21 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/724#discussion_r37625731 --- Diff: test/integration/component/test_escalations_routers.py --- @@ -0,0 +1,189 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-8716: Verify creation of snaps...

2015-08-19 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/665#issuecomment-132583048 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

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

2015-08-19 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/713#issuecomment-132476722 test result of enough storage is not available to perform test put storage in maintenance mode and start ha vm and check usage ... SKIP: sufficient

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

2015-08-18 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/713 CLOUDSTACK-8745 : verify usage after root disk migration put storage in maintenance mode and start ha vm and check usage ... === TestName: test_ha_with_storage_maintenance | Status

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

2015-08-18 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/713#discussion_r37379426 --- Diff: test/integration/component/maint/test_ha_pool_maintenance.py --- @@ -0,0 +1,229 @@ +#!/usr/bin/env python +# Licensed

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

2015-08-18 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/713#discussion_r37379373 --- Diff: test/integration/component/maint/test_ha_pool_maintenance.py --- @@ -0,0 +1,229 @@ +#!/usr/bin/env python +# Licensed

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

2015-08-18 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/713#discussion_r37379748 --- Diff: test/integration/component/maint/test_ha_pool_maintenance.py --- @@ -0,0 +1,229 @@ +#!/usr/bin/env python +# Licensed

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

2015-08-18 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/713#discussion_r37380762 --- Diff: test/integration/component/maint/test_ha_pool_maintenance.py --- @@ -0,0 +1,229 @@ +#!/usr/bin/env python +# Licensed

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

2015-08-18 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/713#discussion_r37380740 --- Diff: test/integration/component/maint/test_ha_pool_maintenance.py --- @@ -0,0 +1,229 @@ +#!/usr/bin/env python +# Licensed

[GitHub] cloudstack pull request: test case automated for list template pag...

2015-08-13 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/646#issuecomment-130588174 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

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

2015-08-13 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/683#issuecomment-130587552 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

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

2015-08-12 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/683#discussion_r36940905 --- Diff: test/integration/component/test_simultaneous_volume_attach.py --- @@ -0,0 +1,256 @@ +# Licensed to the Apache Software Foundation

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

2015-08-12 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/683#discussion_r36845173 --- Diff: test/integration/component/test_simultaneous_volume_attach.py --- @@ -0,0 +1,264 @@ +# Licensed to the Apache Software Foundation

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

2015-08-12 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/683#discussion_r36845123 --- Diff: test/integration/component/test_simultaneous_volume_attach.py --- @@ -0,0 +1,264 @@ +# Licensed to the Apache Software Foundation

[GitHub] cloudstack pull request: CLOUDSTACK-8716: Verify creation of snaps...

2015-08-11 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/665#discussion_r36735392 --- Diff: test/integration/testpaths/testpath_multiple_snapshot.py --- @@ -0,0 +1,279 @@ +# Licensed to the Apache Software Foundation (ASF

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

2015-08-11 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/675#discussion_r36731289 --- Diff: test/integration/testpaths/testpath_usage.py --- @@ -2879,6 +2882,60 @@ def test_07_positive_tests_usage(self

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

2015-08-11 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/667#issuecomment-129836794 can we get test result ? --- 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-8723: Verify API call listUsa...

2015-08-11 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/675#discussion_r36732367 --- Diff: test/integration/testpaths/testpath_usage.py --- @@ -2879,6 +2882,60 @@ def test_07_positive_tests_usage(self

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

2015-08-11 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/667#discussion_r36733719 --- Diff: test/integration/testpaths/testpath_restore_vm.py --- @@ -0,0 +1,201 @@ +# Licensed to the Apache Software Foundation (ASF) under one

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

2015-08-11 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/667#issuecomment-129837984 can we get 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-8716: Verify creation of snaps...

2015-08-11 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/665#discussion_r36734361 --- Diff: test/integration/testpaths/testpath_multiple_snapshot.py --- @@ -0,0 +1,279 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-8716: Verify creation of snaps...

2015-08-11 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/665#issuecomment-129846827 can you update 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-8716: Verify creation of snaps...

2015-08-11 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/665#discussion_r36734207 --- Diff: test/integration/testpaths/testpath_multiple_snapshot.py --- @@ -0,0 +1,279 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-8720: Handle corner case in re...

2015-08-10 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/671#issuecomment-129401139 Looks Good --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8379:add support to marvin to ...

2015-08-05 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/158#issuecomment-127891937 enabled = enabled if enabled else 'true' will set enable to true if enable is false or None which is unexpected, ideally enable should be false

[GitHub] cloudstack pull request: Removed medium dictionary from test_dat...

2015-08-03 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/644#issuecomment-127186217 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

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

2015-07-31 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/638#issuecomment-126589627 can you post 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-8689: Verify effect of changin...

2015-07-31 Thread nitt10prashant
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 and have your reply appear on GitHub as well. If your project does not have

[GitHub] cloudstack pull request: CLOUDSTACK-8659: Verify presentation of v...

2015-07-22 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/613#discussion_r35197771 --- Diff: test/integration/testpaths/testpath_uuid_event.py --- @@ -0,0 +1,198 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-8639:fixing calculation mistak...

2015-07-15 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/595 CLOUDSTACK-8639:fixing calculation mistakes in component/test_ss_doma… test result = Test secondary storage limit of domain and its sub-domains ... === TestName

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

2015-07-14 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/585 CLOUDSTACK-8631 :changing if condition in component/test_ss_max_limit… test result Test Try to deploy VM with admin account where account has not used ... === TestName

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

2015-07-14 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/585#issuecomment-121210527 its there in ticket On Tue, Jul 14, 2015 at 5:03 PM, sanju1010 notificati...@github.com wrote: Can you please specify the reason

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

2015-07-12 Thread nitt10prashant
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 hypervisor test results Test Try to deploy VM with admin

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

2015-07-09 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/569#issuecomment-119913350 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

[GitHub] cloudstack pull request: CLOUDSTACK-8620 :skip test cases if rbd s...

2015-07-09 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/570 CLOUDSTACK-8620 :skip test cases if rbd storage is not available in l… skipping tests based on storage for lxc hype and fixing pep8 issues test result = Test

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

2015-07-09 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/569#discussion_r34238492 --- Diff: test/integration/component/test_blocker_bugs.py --- @@ -663,7 +663,14 @@ def test_01_restart_network_cleanup(self): #all

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

2015-07-09 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/569#issuecomment-119887574 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

[GitHub] cloudstack pull request: Privtmpl

2015-07-08 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/566#discussion_r34134668 --- Diff: test/integration/component/test_project_template.py --- @@ -0,0 +1,172 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8583 : fixing issue related to...

2015-07-08 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/567 CLOUDSTACK-8583 : fixing issue related to script test/integration/com… issue --- Few testcases are failing because of unsupported storage type .Deploy vm with data disk

[GitHub] cloudstack pull request: Privtmpl

2015-07-08 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/566#discussion_r34134916 --- Diff: test/integration/component/test_project_template.py --- @@ -0,0 +1,172 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: Privtmpl

2015-07-08 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/566#discussion_r34134700 --- Diff: test/integration/component/test_project_template.py --- @@ -0,0 +1,172 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: Privtmpl

2015-07-08 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/566#discussion_r34134758 --- Diff: test/integration/component/test_user_taglisting.py --- @@ -0,0 +1,129 @@ +# Licensed to the Apache Software Foundation (ASF) under

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

2015-07-07 Thread nitt10prashant
GitHub user nitt10prashant reopened a pull request: https://github.com/apache/cloudstack/pull/507 CLOUDSTACK-8577: fixing script testpath_disable_enable_zone disable enable cluster ... === TestName: test_01_disable_enable_cluster | Status : SUCCESS === ok disable enable

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

2015-07-07 Thread nitt10prashant
Github user nitt10prashant closed the pull request at: https://github.com/apache/cloudstack/pull/507 --- 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: CLOUDSTACK-8615 :skip all tests in test_p...

2015-07-07 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/565#issuecomment-119188008 Test assign Vm to different account ... SKIP: unsupported storage type Test assign Vm to different account ... SKIP: unsupported storage type Stop

[GitHub] cloudstack pull request: CLOUDSTACK-8615 :skip all tests in test_p...

2015-07-07 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/565 CLOUDSTACK-8615 :skip all tests in test_ps_limits.py for lxc issue --- All test cases are failing due to unsupported storage type solution --- skip

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

2015-07-06 Thread nitt10prashant
Github user nitt10prashant closed the pull request at: https://github.com/apache/cloudstack/pull/507 --- 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

  1   2   >