Bug 1298605 - [RFE] Live Merge: Merge adjacent snapshots concurrently
Summary: [RFE] Live Merge: Merge adjacent snapshots concurrently
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: vdsm
Classification: oVirt
Component: RFEs
Version: 4.17.19
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
: ---
Assignee: Ala Hino
QA Contact: Raz Tamir
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-01-14 14:32 UTC by Adam Litke
Modified: 2022-06-30 07:51 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-06-11 13:16:03 UTC
oVirt Team: Storage
Embargoed:
sbonazzo: ovirt-4.2-


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1287522 0 unspecified CLOSED [RFE] Add a way to delete multiple snapshots using select box 2021-02-22 00:41:40 UTC
Red Hat Issue Tracker RHV-46744 0 None None None 2022-06-30 07:51:58 UTC

Internal Links: 1287522

Description Adam Litke 2016-01-14 14:32:28 UTC
The qemu and libvirt live merge APIs allow the contents of multiple adjacent snapshots to be merged with a single operation.  If engine knows that multiple snapshots are going to be deleted it should submit them all in a single 'merge' call so the merging can be done more efficiently by qemu.

Take the example chain: A-B-C (A is the base and C is the active layer)

This chain has two snapshots (preserved in volumes A and B).  Today if we wanted to reduce this chain to a single volume we would use two merge calls:

merge(base=A, top=B) and merge(base=A, top=C)
(If we removed the oldest snapshot first)

-or-

merge(base=B, top=C) and merge(base=A, top=B)
(if we removed the newest snapshot first)

These two operations could be consolidated into a single merge(base=A, top=C) which would merge the data from B and C into A.

All of the live merge flows need to be evaluated to determine how they will work when multiple volumes are being merged simultaneously.

Comment 1 Yaniv Kaul 2016-01-17 11:02:12 UTC
Can you quantify the improvement? If substantial, why is it 'low' ?

Comment 2 Adam Litke 2016-03-09 15:46:43 UTC
(In reply to Yaniv Kaul from comment #1)
> Can you quantify the improvement? If substantial, why is it 'low' ?

The improvement is principally in user experience.  In cases where a VM has a lot of snapshots that need to be removed it can be labor intensive to "babysit" the merges and submit one after another.  I have not done tests to check the actual copy efficiency but we would be reducing a lot of round-trip calls back and forth between engine and vdsm, waiting for state transitions, etc.  I am sure qemu would copy more efficiently as well.

The severity is "low" because it currently works and this is not as important as block device watermark tracking during live merge.


Note You need to log in before you can comment on or make changes to this bug.