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

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

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

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

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

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

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

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

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

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

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

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

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

2015-08-18 Thread ksowmya
Github user ksowmya commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/691#discussion_r37381189 --- Diff: test/integration/component/test_persistent_networks.py --- @@ -1459,6 +1459,135 @@ def test_delete_account(self): return

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

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

[GitHub] cloudstack pull request: CLOUDSTACK-8738: Added the two methods fo...

2015-08-17 Thread ksowmya
Github user ksowmya closed the pull request at: https://github.com/apache/cloudstack/pull/702 --- 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 feature

[GitHub] cloudstack pull request: Disablestorage pep8

2015-08-17 Thread ksowmya
GitHub user ksowmya opened a pull request: https://github.com/apache/cloudstack/pull/703 Disablestorage pep8 Fixed pep8 issues in script, most of them for E501 (long lines) with max-line-length set to default Test Result: Test Steps: ... === TestName

[GitHub] cloudstack pull request: Disablestorage pep8

2015-08-17 Thread ksowmya
Github user ksowmya commented on the pull request: https://github.com/apache/cloudstack/pull/703#issuecomment-131794674 @remibergsma , Thanks! I've updated this one to include both commits. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: CLOUDSTACK-8738: Added the two methods fo...

2015-08-17 Thread ksowmya
GitHub user ksowmya opened a pull request: https://github.com/apache/cloudstack/pull/702 CLOUDSTACK-8738: Added the two methods for enable and cancel maintena… Added the two methods for enable and cancel maintenance used in testpath_disablestoragepool Test results

[GitHub] cloudstack pull request: CLOUDSTACK-8738: Added the two methods fo...

2015-08-17 Thread ksowmya
Github user ksowmya commented on the pull request: https://github.com/apache/cloudstack/pull/702#issuecomment-131794425 Closing since this is part of https://github.com/apache/cloudstack/pull/703/commits --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: Move tests to maint/ folder and added few...

2015-06-19 Thread ksowmya
GitHub user ksowmya opened a pull request: https://github.com/apache/cloudstack/pull/483 Move tests to maint/ folder and added few error checking Moved disable_storagepool to maint/ folder Added better error checking if zone wide or local storages are not present You can merge

[GitHub] cloudstack pull request: Move tests to maint/ folder and added few...

2015-06-19 Thread ksowmya
Github user ksowmya closed the pull request at: https://github.com/apache/cloudstack/pull/483 --- 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 feature

[GitHub] cloudstack pull request: Move tests to maint/ folder and added few...

2015-06-19 Thread ksowmya
Github user ksowmya commented on the pull request: https://github.com/apache/cloudstack/pull/483#issuecomment-113470122 This closes #483 --- 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