Updated Branches: refs/heads/master b4662af0a -> e25cbd66d
CLOUDSTACK-3619: Cleanup of volume is not required as gc takes care. We don't need to delete the volume resource as part of cleanup as the storage gc is expected to cleanup orphaned volumes in deleted accounts/projects. Signed-off-by: Prasanna Santhanam <[email protected]> (cherry picked from commit 8bc3757e80d20575d1eeee530309ab61c02857d8) Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/c3a04b63 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/c3a04b63 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/c3a04b63 Branch: refs/heads/master Commit: c3a04b63a7ce3015ee430c1aac38696539def17c Parents: e90ef2b Author: Prasanna Santhanam <[email protected]> Authored: Thu Jul 18 14:59:35 2013 +0530 Committer: Prasanna Santhanam <[email protected]> Committed: Thu Jul 18 15:05:56 2013 +0530 ---------------------------------------------------------------------- test/integration/component/test_projects.py | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/c3a04b63/test/integration/component/test_projects.py ---------------------------------------------------------------------- diff --git a/test/integration/component/test_projects.py b/test/integration/component/test_projects.py index 68964e8..510542a 100644 --- a/test/integration/component/test_projects.py +++ b/test/integration/component/test_projects.py @@ -1203,10 +1203,9 @@ class TestProjectResources(cloudstackTestCase): raise Exception("Warning: Exception during cleanup : %s" % e) return - @attr(tags = ["advanced", "basic", "sg", "eip", "advancedns", "simulator"]) + @attr(tags = ["advanced", "basic", "sg", "eip", "advancedns", "simulator", "needle"]) def test_07_project_resources_account_delete(self): - """ Test Verify after an account is removed from the project, all his - resources stay with the project. + """ Test Verify after an account is removed from the project, all its resources stay with the project. """ # Validate the following # 1. Create a project. @@ -1261,13 +1260,13 @@ class TestProjectResources(cloudstackTestCase): ) # listProjectAccount to verify the user is added to project or not - accounts_reponse = Project.listAccounts( + accounts_response = Project.listAccounts( self.apiclient, projectid=project.id, account=self.user.name, ) self.assertEqual( - isinstance(accounts_reponse, list), + isinstance(accounts_response, list), True, "Check for a valid list accounts response" ) @@ -1277,7 +1276,7 @@ class TestProjectResources(cloudstackTestCase): 0, "Check list project response returns a valid project" ) - account = accounts_reponse[0] + account = accounts_response[0] self.assertEqual( account.role, @@ -1292,9 +1291,8 @@ class TestProjectResources(cloudstackTestCase): diskofferingid=self.disk_offering.id, projectid=project.id ) - self.cleanup.append(volume) - # Delete the project user + # Delete the project user ie the account self.user.delete(self.apiclient) volumes = Volume.list(self.apiclient, id=volume.id)
