Cinder

ceph rbd flatten clone image

Bug #1374256 reported by ling-yun
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Cinder
Won't Fix
Wishlist
Unassigned

Bug Description

Suppose cinder uses ceph rbd driver as volume back end, glance uses ceph rbd driver as storage back end and cinder volume back end and glance storage back end are in the same ceph storage cluster.
If cinder creates a volume from image, it could create a clone volume from image, and the relationship is image-->clone snapshot-->clone image.
If we want a higher performance rbd image than the clone image, we should flatten the clone image to meet the application requirements.

So it is necessary to add an config option rbd_flatten_volume_from_image to decide whether flatten rbd clone image or not. If flatten rbd clone image, cinder should also delete the clone_snap.

Tags: ceph
ling-yun (zengyunling)
Changed in cinder:
assignee: nobody → ling-yun (zengyunling)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (master)

Change abandoned by Mike Perez (<email address hidden>) on branch: master
Review: https://review.openstack.org/124669

Revision history for this message
Sean McGinnis (sean-mcginnis) wrote :

Automatically unassigning due to inactivity.

Changed in cinder:
assignee: ling-yun (zengyunling) → nobody
status: In Progress → Triaged
Revision history for this message
Matt Riedemann (mriedem) wrote :

This was fixed way back in havana:

https://review.openstack.org/#/c/32490/

Changed in cinder:
status: Triaged → Fix Released
Revision history for this message
Matt Riedemann (mriedem) wrote :

This was not fixed in havana, this is not the same use case as rbd_flatten_volume_from_snapshot. Marked as invalid though since it looks like it's been abandoned. It's a feature request (blueprint) anyway.

Changed in cinder:
status: Fix Released → Invalid
importance: Undecided → Wishlist
tags: added: ceph
Maximiliano Osorio (mosorio-0)
Changed in cinder:
assignee: nobody → Maximiliano Osorio (mosorio-0)
status: Invalid → Opinion
status: Opinion → In Progress
Revision history for this message
Jon Bernard (jbernard) wrote :

If we want a higher performance rbd image than the clone image, then don't configure glance and cinder to make clones possible (e. g. show_direct_url=False) and you'll have a full copy which is what flattening achieves. An extra configuration option is not necessary and so this bug is probably invalid.

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

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

Changed in cinder:
assignee: Maximiliano Osorio (mosorio-0) → maximiliano osorio (maxiosorio)
Revision history for this message
Maximiliano Osorio (mosorio-0) wrote :

If you’re using glance and cinder with rbd and show_direct_url=False, cinder has to download the image (you can see it at [1]). That isn't efficient.
Using show_direct_url=true and rbd_flatten_volume_from_image=True, cinder can directly fetch the image from rbd using the url.
[1] http://paste.openstack.org/show/492690/ - image_utils.TemporaryImages.fetch

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/300286

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (stable/liberty)

Change abandoned by Eric Harney (<email address hidden>) on branch: stable/liberty
Review: https://review.openstack.org/300286
Reason: It's too late to do this for liberty at this point.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (master)

Change abandoned by Sean McGinnis (<email address hidden>) on branch: master
Review: https://review.openstack.org/300211
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Sean McGinnis (sean-mcginnis)
Changed in cinder:
status: In Progress → Won't Fix
Revision history for this message
Dr. Jens Harbott (j-harbott) wrote :

There is a new spec being proposed here https://review.openstack.org/400100 , so maybe someone can change the status back to Confirmed?

Changed in cinder:
assignee: maximiliano osorio (maxiosorio) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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