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
Attachment:
snapshot-diff.txt.3
Description: Binary data
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@xxxxxxxxxx > https://www.redhat.com/mailman/listinfo/libvir-list
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list