Libvirt LVM storage backend doesn't initialise filesystems of ephemeral disks

Bug #1648109 reported by Matthew Booth
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Matthew Booth
Newton
In Progress
Low
Lee Yarwood

Bug Description

N.B. This is from code inspection only.

When creating an LVM-backed instance with an ephemeral disk, the ephemeral disk will not be initialised with the requested filesystem. This is because Image.cache() wraps the _create_ephemeral callback in fetch_func_sync, which will not call _create_ephemeral if the target already exists. Because the Lvm backend must create the disk first, this is never called.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Changed in nova:
assignee: nobody → Matthew Booth (mbooth-9)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/408152
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=de5bb0c3fe9e86166e81aa2465a8e7791f64ba39
Submitter: Jenkins
Branch: master

commit de5bb0c3fe9e86166e81aa2465a8e7791f64ba39
Author: Matthew Booth <email address hidden>
Date: Wed Dec 7 14:45:40 2016 +0000

    libvirt: Fix initialising of LVM ephemeral disks

    The LVM backend expects to write directly to the target disk rather
    than to the image cache when initialising an ephemeral disk. This is
    confounded by Image.cache(), which doesn't call the given callback
    (_create_ephemeral in this case), if the target already exists.

    Closes-Bug: #1648109
    Co-authored-by: Feodor Tersin <email address hidden>
    Change-Id: Ic9178d6c013670611e63d54a7669f35dc9770e91

Changed in nova:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/409708

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 15.0.0.0b2

This issue was fixed in the openstack/nova 15.0.0.0b2 development milestone.

Matt Riedemann (mriedem)
Changed in nova:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (stable/newton)

Change abandoned by Lee Yarwood (<email address hidden>) on branch: stable/newton
Review: https://review.openstack.org/409708

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.