create backup from snapshot can't work as error in _create_temp_volume_from_snapshot

[Lisa Li]

Currently when creating backup from snapshot, it uses following way to create volume.

    def _create_temp_volume_from_snapshot(self, context, volume, snapshot):
        temp_volume = {
            'size': volume['size'],
            'display_name': 'backup-vol-%s' % volume['id'],
            'host': volume['host'],
            'user_id': context.user_id,
            'project_id': context.project_id,
            'status': 'creating',
        }

But in objects.Volume, the fields attach_status and availability_zone can't be null. As a result, it leads c-api/c-sch/c-bak can't work.

Please look into bug https://bugs.launchpad.net/cinder/+bug/1549669.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/285086

[Michal Dulko (michal-dulko-f)]

Reassigning back to Lisa, as I've just rebased a patch.

[Lisa Li (lisali)]

Set the bug as critical as the function 'create backup from snapshot' can't work.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/284600
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=231f856ece08ebc65ed5fcf201df10b1cec27d0b
Submitter: Jenkins
Branch: master

commit 231f856ece08ebc65ed5fcf201df10b1cec27d0b
Author: lisali <email address hidden>
Date: Thu Feb 25 08:59:42 2016 +0000

    Add necessary fields to volume creation

    This patch is to add the two fields in driver.py when creating
    volumes.

    Without the fix, it leads query related to the volume failed
    with exception.

    Change-Id: I0055b5bf402b0aa7b4ee78b0fb40bedea850af94
    Closes-Bug: #1549673
    Related-Bug: #1549669

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/285086
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=7a158ac4488ac86c742e15c714dda7dd76772366
Submitter: Jenkins
Branch: master

commit 7a158ac4488ac86c742e15c714dda7dd76772366
Author: LisaLi <email address hidden>
Date: Fri Feb 26 10:41:19 2016 +0800

    Make nullable of fields in db model and object match

    Currently there are some fields whose nullable in db
    model and object don't match. This may lead object query
    fails.

    For example, for volume, if attach_status is nullable in
    db schema, but is not nullable in object schema.
    When writting data to db using db interface, the attach_status
    can be null. But later when reading data from db to construct
    volume object, it raises exception. As a result, all the
    query related to the volume fails.

    This patch is to make all the fields in corresponding db
    model and object match.

    Change-Id: I84fb16f0a67f9a1f7a09e91bb8ebe6298f3f49c4
    Closes-Bug: #1549669
    Related-Bug: #1549673

[Thierry Carrez (ttx)]

This issue was fixed in the openstack/cinder 8.0.0.0rc1 release candidate.