]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
GlusterFS: Use correct base argument when deleting attached snaps
authorEric Harney <eharney@redhat.com>
Thu, 19 Dec 2013 00:11:44 +0000 (19:11 -0500)
committerGerrit Code Review <review@openstack.org>
Thu, 2 Jan 2014 15:07:00 +0000 (15:07 +0000)
commit186221779a92002ff9fa13c254710c0abb3803be
tree4a7ff35341aa3eda0efc97acd6d49c9acfa0dd80
parentf52cb1b1c44c20d034bbb72e2ae98dbadc8e2495
GlusterFS: Use correct base argument when deleting attached snaps

When deleting the most recent snapshot, the 'file_to_merge' field
which translates into the base= field for libvirt's blockRebase
call in Nova must be set depending on whether other snapshots exist.

If there are no other snapshots, base = None, which results in
libvirt clearing the qcow2 backing file pointer for the active
disk image.

If there are other snapshots, pass the parent of the file being
deleted as the new base file.  The snapshot info pointer for the
prior base file must also be updated in this case.

Closes-Bug: #1262880
Change-Id: If7bc8259b031d0406346caafb8f688e65a38dba6
cinder/tests/test_glusterfs.py
cinder/volume/drivers/glusterfs.py