]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Set bootable flag for volume cloned from image
authorxiaoxi_chen <xiaoxi.chen@intel.com>
Mon, 12 Aug 2013 11:25:59 +0000 (19:25 +0800)
committerGerrit Code Review <review@openstack.org>
Wed, 21 Aug 2013 10:35:35 +0000 (10:35 +0000)
commit162aa28824fb6caa6e3adaf7ed268e9fae178a73
treeb0da103f0659cec4c2d9d99c991680713b01cdc0
parentd2b48cd96db32668876c0d3e9518653f09d7cacb
Set bootable flag for volume cloned from image

In previous code we don't set the bootable flag for volume
cloned from image. This does not appear to break booting
from volume at present but the status displayed by the cinder
client is broken.

The bug is reported and fixed several months ago(bug #1185533)
but the author doesn't provide unit test with the patch. Now it
has been broken again by somebody else.The patch re-fixed this
bug together with unit tests.

Also add a function named _handle_bootable_volume_glance_meta,
it is a combination of enable_bootable_flag and glance_metadata
handling. There are 3 kinds of volume creation tasks may required
to copy/fetch the glance metadata. In previous code every kind of
task have its own handling code, this patch consolidate them together
to share some common code.

fixed bug #1211272
fixed bug #1185533

Change-Id: I86007e26efbebe58dc0c5995b6f14f68258144b5
cinder/tests/test_volume.py
cinder/volume/flows/create_volume.py