Lack of volume status checks when detaching volume in rebuild.

Bug #1489304 reported by Zhenyu Zheng on 2015-08-27
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Medium
Unassigned

Bug Description

Currently, when rebuilding an instance with volume attached, the Nova compute manager will directly call _detach_volume() which will skip the checks of volume status (volume_api.check_detach) and setting the volume to 'detaching' (volume_api.begin_detaching) at Cinder side. This is different with the normal volume detach process.

Besides, when rebuilding, we should only allow detaching the volume with in-use status, volume in status such as 'retyping' should not allowed.

Changed in nova:
assignee: nobody → Zhenyu Zheng (zhengzhenyu)

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

Changed in nova:
status: New → In Progress
Changed in nova:
importance: Undecided → Medium
description: updated
description: updated

Change abandoned by Michael Still (<email address hidden>) on branch: master
Review: https://review.openstack.org/217567
Reason: This patch has been sitting unchanged for more than 12 weeks. I am therefore going to abandon it to keep the nova review queue sane. Please feel free to restore the change if you're still working on it.

Maciej Szankin (mszankin) wrote :

This bug report has an assignee for a while now but there is no patch
for that. It looks like that the chance of getting a patch is low.
I'm going to remove the assignee to signal to others that they can take
over if they like.
If you want to work on this, please:
* add yourself as assignee AND
* set the status to "In Progress" AND
* provide a (WIP) patch within the next 2 weeks after that.
If you need assistance, reach out on the IRC channel #openstack-nova or
use the mailing list.

Changed in nova:
status: In Progress → New
assignee: Zhenyu Zheng (zhengzhenyu) → nobody
Changed in nova:
status: New → Confirmed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers