From fc6280f84eddc1271dd7b80b7a8a613d11a9c36f Mon Sep 17 00:00:00 2001 From: zhufl Date: Tue, 26 Sep 2017 16:12:44 +0800 Subject: [PATCH] Remove redundant volume check in nova_volume_detach In nova_volume_detach, show_volume and assertEqual are redundant because in wait_for_volume_resource_status this check is already done. This is to remove them for code clean. Change-Id: If5a25914bd4be039e0a8ddad661732ef1f3a28ee --- tempest/scenario/manager.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/tempest/scenario/manager.py b/tempest/scenario/manager.py index 284322238c..ef2dd3adce 100644 --- a/tempest/scenario/manager.py +++ b/tempest/scenario/manager.py @@ -506,9 +506,6 @@ class ScenarioTest(tempest.test.BaseTestCase): waiters.wait_for_volume_resource_status(self.volumes_client, volume['id'], 'available') - volume = self.volumes_client.show_volume(volume['id'])['volume'] - self.assertEqual('available', volume['status']) - def rebuild_server(self, server_id, image=None, preserve_ephemeral=False, wait=True, rebuild_kwargs=None):