cinder-manage volume delete doesn't support volume ids in uuid format
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Cinder |
Fix Released
|
High
|
stephen mulcahy |
Bug Description
cinder-manage volume delete uses param2id() function to convert user-specified volume id to internal id. param2id() in trunk currently looks like
def param2id(
"""Helper function to convert various id types to internal id.
args: [object_id], e.g. 'vol-0000000a' or 'volume-0000000a' or '10'
"""
if '-' in object_id:
# FIXME(ja): mapping occurs in nova?
pass
else:
return int(object_id)
which looks like it'll never return an id for an object_id with a dash in it (which includes all uuids).
Propose updating param2id to return the object_id unmodified if it looks like a uuid (using uuidutils.
Changed in cinder: | |
assignee: | nobody → stephen mulcahy (stephen-mulcahy) |
Changed in cinder: | |
milestone: | none → grizzly-rc2 |
importance: | Undecided → High |
Changed in cinder: | |
milestone: | grizzly-rc2 → 2013.1 |
Fix proposed to branch: master /review. openstack. org/25033
Review: https:/