cinder: if I delete a volume type that is part of a volume we remove the type and list the volume with type=uuid

Bug #1214742 reported by Haim Ateya
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
In Progress
Undecided
Unassigned

Bug Description

Description of problem:

If I created a volume with a type and than remove the type, we remove the name of the type from the volume list but leave the UUID.

Version-Release number of selected component (if applicable):

openstack-cinder-2013.1.2-3.el6ost.noarch

How reproducible:

100%

Steps to Reproduce:
1. create a volume type
2. create a volume using the type you created
3. delete the type

Actual results:

the type is deleted but leaves the volumes with type which is UUID

Expected results:

if we remove it from all the associated objects

Additional info:

[root@opens-vdsb ~(keystone_admin)]# cinder list
+--------------------------------------+-----------+--------------+------+-------------+----------+-------------+
| ID | Status | Display Name | Size | Volume Type | Bootable | Attached to |
+--------------------------------------+-----------+--------------+------+-------------+----------+-------------+
| efd38e20-bc99-415c-aa0e-fc9efd22ca53 | available | TYPE | 10 | TYPE | false | |
+--------------------------------------+-----------+--------------+------+-------------+----------+-------------+

[root@opens-vdsb ~(keystone_admin)]# cinder list
+--------------------------------------+-----------+--------------+------+--------------------------------------+----------+-------------+
| ID | Status | Display Name | Size | Volume Type | Bootable | Attached to |
+--------------------------------------+-----------+--------------+------+--------------------------------------+----------+-------------+
| efd38e20-bc99-415c-aa0e-fc9efd22ca53 | available | TYPE | 10 | 220d424d-a28d-44a9-b043-6a027f8fb886 | false | |
+--------------------------------------+-----------+--------------+------+--------------------------------------+----------+-------------+

Changed in cinder:
assignee: nobody → Swapnil Kulkarni (swapnil.kulkarni)
Revision history for this message
Swapnil Kulkarni (coolsvap-deactivatedaccount) wrote :

Reproduced teh bug, working on the fix.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/43109

Changed in cinder:
status: New → In Progress
Revision history for this message
Swapnil Kulkarni (coolsvap-deactivatedaccount) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/43109

Changed in cinder:
status: In Progress → Fix Committed
Revision history for this message
Swapnil Kulkarni (coolsvap-deactivatedaccount) wrote :

Unit test results for verified code changes.

# cinder list
+--------------------------------------+-----------+--------------+------+-------------+----------+-------------+
| ID | Status | Display Name | Size | Volume Type | Bootable | Attached to |
+--------------------------------------+-----------+--------------+------+-------------+----------+-------------+
| 94706bfe-8141-46d2-940a-17c7fc256aef | available | inlvm3 | 1 | lvm | False | |
+--------------------------------------+-----------+--------------+------+-------------+----------+-------------+

# cinder type-list
+--------------------------------------+----------+
| ID | Name |
+--------------------------------------+----------+
| 06642a3d-263c-4361-bef8-ce74a8a6829a | LUKS |
| 7fa5af0b-f4fb-48ac-b1e4-93e30b944e7b | lvm |
| c6480559-8138-4102-89db-1875c2326e3a | dm-crypt |
+--------------------------------------+----------+

# cinder type-delete 7fa5af0b-f4fb-48ac-b1e4-93e30b944e7b

# cinder type-list
+--------------------------------------+----------+
| ID | Name |
+--------------------------------------+----------+
| 06642a3d-263c-4361-bef8-ce74a8a6829a | LUKS |
| c6480559-8138-4102-89db-1875c2326e3a | dm-crypt |
+--------------------------------------+----------+

# cinder list
+--------------------------------------+-----------+--------------+------+-------------+----------+-------------+
| ID | Status | Display Name | Size | Volume Type | Bootable | Attached to |
+--------------------------------------+-----------+--------------+------+-------------+----------+-------------+
| 94706bfe-8141-46d2-940a-17c7fc256aef | available | inlvm3 | 1 | None | False | |
+--------------------------------------+-----------+--------------+------+-------------+----------+-------------+

Changed in cinder:
status: Fix Committed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/43116

Revision history for this message
Swapnil Kulkarni (coolsvap-deactivatedaccount) wrote :

Had a conversation with john-griffith and there could be some ripple effects with deleting a volume-type (like outstanding volumes, quotas etc). I think it makes more sense to work on https://bugs.launchpad.net/cinder/+bug/1215329 where we can validate the volumes associated with that volume type and work accordingly.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.