openstack/deb-tempest
Code Issues Proposed changes

Fix: WhiteBox server leak

Browse Source 
Deleteting the server explicity in the tearDownClass.

Fixing Bug 1180852

Change-Id: If94c82717832ef35473840b793fcfd6871bbf4bd
This commit is contained in:
Attila Fazekas
2013-06-09 17:21:15 +02:00
parent d021ea4b2c
commit cf94a14b26
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tempest/whitebox/test_images_whitebox.py
View File

@@ -20,6 +20,10 @@ from tempest.common.utils.data_utils import rand_name
from tempest import exceptions from tempest import exceptions
from tempest.whitebox import manager from tempest.whitebox import manager
#TODO(afazekas): The whitebox tests are using complex testclass/manager
# hierarchy, without a real need. It is difficult to maintain.
# They could share more code with scenario tests.
class ImagesWhiteboxTest(manager.ComputeWhiteboxTest, base.BaseComputeTest): class ImagesWhiteboxTest(manager.ComputeWhiteboxTest, base.BaseComputeTest):
_interface = 'json' _interface = 'json'
@@ -34,7 +38,8 @@ class ImagesWhiteboxTest(manager.ComputeWhiteboxTest, base.BaseComputeTest):
@classmethod @classmethod
def tearDownClass(cls): def tearDownClass(cls):
"""Delete images after a test is executed.""" """Delete images and server after a test is executed."""
cls.servers_client.delete_server(cls.shared_server['id'])
for image_id in cls.image_ids: for image_id in cls.image_ids:
cls.client.delete_image(image_id) cls.client.delete_image(image_id)
cls.image_ids.remove(image_id) cls.image_ids.remove(image_id)