Skip to content

Destroyvm also removes volumes #18

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
houthuis wants to merge 15 commits into from
Closed

Destroyvm also removes volumes #18

Changes from 2 commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
4a154b6
WIP volume validation, volume detach
Jul 9, 2018
880b38e
WIP delete volume code added. functionally complete, cleanup and refa…
Jul 9, 2018
eb929a0
WIP ui code
Jul 24, 2018
12e627b
Adding a test for DestroyVM with volumes
borisstoyanov Jul 25, 2018
e953db9
WIP front end complete
Jul 31, 2018
fbc3fdf
Merge branch 'destroyvm-also-removes-volumes' of https://github.com/s…
Jul 31, 2018
d2f3b35
code cleanup
Jul 31, 2018
bff19b3
Merge branch 'master' of https://github.com/shapeblue/cloudstack into…
Jul 31, 2018
3c7e0cc
improved wording
Jul 31, 2018
83ae447
changes following code review
Jul 31, 2018
6129eb6
changes following code review
Aug 1, 2018
ff5004e
Adding a test for DestroyVM with volumes
borisstoyanov Aug 2, 2018
eb02664
ui enhancement when no volumes are attached to a vm
Aug 3, 2018
9590da1
Merge branch 'destroyvm-also-removes-volumes' of https://github.com/s…
Aug 3, 2018
d996e77
added code to allow users to detach and delete volumes when destroyin…
Jul 9, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
115 changes: 114 additions & 1 deletion test/integration/smoke/test_vm_life_cycle.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,8 @@
Host,
Iso,
Router,
Configurations)
Configurations,
Volume)
from marvin.lib.common import (get_domain,
get_zone,
get_template,
Expand Down Expand Up @@ -786,6 +787,118 @@ def test_10_attachAndDetach_iso(self):
)
return

@attr(tags = ["devcloud", "advanced", "advancedns", "smoke", "basic", "sg"], required_hardware="false")
def test_11_destroy_vm_and_volumes(self):
"""Test destroy Virtual Machine and it's volumes
"""

# Validate the following
# 1. Deploys a VM and attaches disks to it
# 2. Destroys the VM with DataDisks option

small_virtual_machine = VirtualMachine.create(
self.apiclient,
self.services["small"],
accountid=self.account.name,
domainid=self.account.domainid,
serviceofferingid=self.small_offering.id,
mode=self.services["mode"]
)
vol1 = Volume.create(
self.apiclient,
self.services,
account=self.account.name,
domainid=self.account.domainid
)

vol2 = Volume.create(
self.apiclient,
self.services,
account=self.account.name,
domainid=self.account.domainid
)

small_virtual_machine.attach_volume(self.apiclient, vol1)
small_virtual_machine.attach_volume(self.apiclient, vol2)

cmd = destroyVirtualMachine.destroyVirtualMachineCmd()
cmd.id = small_virtual_machine.id
cmd.expunge = True
cmd.volumes = vol1.id, vol2.id

response = self.apiclient.destroyVirtualMachine(cmd)

self.debug("Destroy VM - ID: %s" % self.small_virtual_machine.id)
self.small_virtual_machine.delete(self.apiclient, expunge=False)

list_vm_response = VirtualMachine.list(
self.apiclient,
id=self.small_virtual_machine.id
)
self.assertEqual(
isinstance(list_vm_response, list),
True,
"Check list response returns a valid list"
)

self.assertNotEqual(
len(list_vm_response),
0,
"Check VM avaliable in List Virtual Machines"
)

self.assertEqual(
list_vm_response[0].state,
"Destroyed",
"Check virtual machine is in destroyed state"
)

list_vol = Volume.list(
self.apiclient,
id=vol1.id
)
self.assertEqual(
isinstance(list_vol, list),
True,
"Check list response returns a valid list"
)

self.assertNotEqual(
len(list_vol),
0,
"Check VM avaliable in List Virtual Machines"
)

self.assertEqual(
list_vol[0].state,
"Destroyed",
"Check virtual machine is in destroyed state"
)

list_vol = Volume.list(
self.apiclient,
id=vol2.id
)
self.assertEqual(
isinstance(list_vol, list),
True,
"Check list response returns a valid list"
)

self.assertNotEqual(
len(list_vol),
0,
"Check VM avaliable in List Virtual Machines"
)

self.assertEqual(
list_vol[0].state,
"Destroyed",
"Check virtual machine is in destroyed state"
)
return


class TestSecuredVmMigration(cloudstackTestCase):

@classmethod
Expand Down