From 3b3adbf71268fea526e91a94e7d64fe511daf840 Mon Sep 17 00:00:00 2001 From: Girish Shilamkar Date: Tue, 31 Dec 2013 11:34:16 +0530 Subject: [PATCH] CLOUDSTACK-5637: Detach volume before migrating --- test/integration/component/test_stopped_vm.py | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/test/integration/component/test_stopped_vm.py b/test/integration/component/test_stopped_vm.py index e700c46f04d..e8a90b70843 100644 --- a/test/integration/component/test_stopped_vm.py +++ b/test/integration/component/test_stopped_vm.py @@ -891,6 +891,8 @@ class TestDeployVM(cloudstackTestCase): "Running", "VM should be in Running state after deployment" ) + + self.debug("Stopping instance: %s" % self.virtual_machine.name) self.virtual_machine.stop(self.apiclient) self.debug("Instance is stopped!") @@ -939,6 +941,11 @@ class TestDeployVM(cloudstackTestCase): self.storage_id = spool.id self.storage_name = spool.name break + + self.debug("Detaching volume %s from vm %s" % (vol_response.id, self.virtual_machine.id)) + + self.virtual_machine.detach_volume(self.apiclient, vol_response) + self.debug("Migrating volume to storage pool: %s" % self.storage_name) Volume.migrate( self.apiclient, @@ -947,14 +954,14 @@ class TestDeployVM(cloudstackTestCase): ) volume = Volume.list( self.apiclient, - virtualmachineid=self.virtual_machine.id, - type='ROOT', - listall=True + virtualmachineid=self.virtual_machine.id, + type="ROOT", + listall=True, ) self.assertEqual( volume[0].storage, self.storage_name, - "Check volume migration response") + "Check volume migration response vol.storage %s self.storage_name %s" % (volume[0].storage, self.storage_name)) return