[libvirt] [PATCH] Libvirt Snapshot API

Philip Jameson pjameson at pjameson.us
Sun Jan 17 18:06:33 UTC 2010

Sorry, looks like I missed the message about the new error handling functions, and I messed up a regex. Attached are the changes.

Philip Jameson
-------------- next part --------------
A non-text attachment was scrubbed...
Name: snapshot-diff.txt.3
Type: application/octet-stream
Size: 2359 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100117/423e3df9/attachment-0001.obj>
-------------- next part --------------

On Jan 17, 2010, at 12:31 PM, Philip Jameson wrote:

> A little while ago I submitted a patch to add a snapshot API, but I was informed that it got somewhat lost in the list. I have attached a patch that implements this API at least for the qemu driver, and it also added another function for screenshots, just because it usually is nice to  get a view of the screen with the snapshot.
> The snapshot functions just take names, as I figure it should be up to libvirt to figure out where the snapshots go. e.g. for qemu, they just get stored with the drive using qemu's built in facilities. If someone were using LVM, then that could be managed by libvirt, just like it does for qemu monitors and such. The 'list snapshots' just returns a list of names for now, as that's all I needed, but I think it would probably be best to add a struct that would be name/time.
> I don't know if it's of any use at the moment, but I needed these functions implemented, so I figured I'd at least submit my changes
> Philip Jameson
> <snapshot-diff.txt.2>--
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list

More information about the libvir-list mailing list