[Openstack-docs] [openstack/nova] DocImpact review request change Ida0b6cdcd14c43abe2ddabd169e53743bc28bc34
gerrit2 at review.openstack.org
gerrit2 at review.openstack.org
Wed Feb 6 22:11:14 UTC 2013
Hi, I'd like you to take a look at this patch for potential
DocImpact.
https://review.openstack.org/21374
Log:
commit a3f904a74386a25cf995449be1aa5a6b3ce62090
Author: Josh Durgin <josh.durgin at inktank.com>
Date: Wed Feb 6 02:03:10 2013 -0800
Add conversion between local and volume storage on boot
Cinder has the ability to convert from images to volumes, from volumes
to images, and snapshots to volumes. These are all useful ways to
create a new vm with the desired software.
Add source_type, dest_type, and image_id to the database (it already
has snapshot_id and volume_id for the existing block_device_mapping
handling). Use these existing fields to make backwards compatibility
at the virt driver level simpler.
source_type may be 'image', 'volume', 'snapshot', or 'blank'.
dest_type may be 'local' or 'volume'.
When the dest_type is different from the source_type and involves
volumes, use the cinder api to create a new image to use as the basis
for a local disk, or a new volume.
Wait for this to complete until the instance build timeout is reached,
or the image/volume being created enters a state that we cannot use it
in.
Note that this also enables using more than virtual disk created
from an image, even if they are all local. This could be restricted
to only one local disk based on an image so that there is less chance
of creating bugs with this new behavior.
This is based on ndipanov's prototype of a block_device_mapping_v2
addition to the api. It fixes a few typos in there temporarily,
and renames uuid to id in the api payload.
Handling of default device names is not addressed here, as other
patches already in progress do this. Once these land, I expect this
will need a bit of refactoring.
Still to come:
* unit tests
* handling size appropriately with the given instance_type
* better validation in the api
DocImpact
Blueprint: improve-boot-from-volume
Blueprint: flexible-block-device-handling
Change-Id: Ida0b6cdcd14c43abe2ddabd169e53743bc28bc34
Signed-off-by: Josh Durgin <josh.durgin at inktank.com>
More information about the Openstack-docs
mailing list