2010/12/22 Justin Clift <jclift@xxxxxxxxxx>: > On 23/12/2010, at 8:13 AM, Eric Blake wrote: >> On 12/22/2010 01:56 PM, Matthias Bolte wrote: >>> Shorten qemuDomainSnapshotWriteSnapshotMetadata function name >>> and make it take a snapshot pointer instead of dealing with >>> the current snapshot. Update other functions accordingly. >>> >>> Add a qemuDomainSnapshotReparentChildren hash iterator to >>> reparent the children of a snapshot that is being deleted. Use >>> qemuDomainSnapshotWriteMetadata to write updated metadata >>> to disk. >>> >>> This fixes a problem where outdated parent information breaks >>> the snapshot tree and hinders the deletion of child snapshots. >>> >>> Reported by Philipp Hahn. >>> --- >>> src/qemu/qemu_driver.c | Â 92 +++++++++++++++++++++++++++++++++++++----------- >>> 1 files changed, 71 insertions(+), 21 deletions(-) >> >> ACK. > > Cool. ÂSounds like this is the fix for BZ #663586: > > Âhttps://bugzilla.redhat.com/show_bug.cgi?id=663586 I didn't know there is a BZ entry for it, but yes. Matthias -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list