[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